After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 785393 - [ESourceRefresh] Let zero Refresh interval mean 'No periodic update'
[ESourceRefresh] Let zero Refresh interval mean 'No periodic update'
Status: RESOLVED FIXED
Product: evolution-data-server
Classification: Platform
Component: Contacts
3.22.x (obsolete)
Other Linux
: Normal normal
: ---
Assigned To: evolution-addressbook-maintainers
Evolution QA team
Depends on:
Blocks:
 
 
Reported: 2017-07-25 10:42 UTC by Andrea Vai
Modified: 2017-07-26 13:47 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Andrea Vai 2017-07-25 10:42:17 UTC
If I enter "0" as the refresh interval value in an address book of type "Google", the evolution-addressbook-factory process (-subprocess, actually) unlimitedly raises memory occupation because tries to update continuously (I can reproduce and monitor it with 

top -p`pidof evolution-addressbook-factory` -p`pidof evolution-addressbook-factory-subprocess`

).

This behavior should be avoided by not accepting the 0 value as the refresh interval, or giving 0 the special meaning (as in other softwares) "do never refresh".

Personally, I would prefer the latter "solution", because it would enhance evo with a new feature :-)

Reference: https://mail.gnome.org/archives/evolution-list/2017-July/msg00107.html
Comment 1 Milan Crha 2017-07-26 13:47:05 UTC
Thanks for a bug report. I made the zero mean 'No periodic update' with:

Created commit 5c7055b in eds master (3.25.90+)
Created commit 2d692bc in eds gnome-3-24 (3.24.5+)