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 120663 - Reduce config traffic for mailbox views
Reduce config traffic for mailbox views
Status: RESOLVED FIXED
Product: balsa
Classification: Other
Component: general
2.0.x
Other All
: Normal enhancement
: ---
Assigned To: Balsa Maintainers
Balsa Maintainers
Depends on:
Blocks:
 
 
Reported: 2003-08-25 12:57 UTC by Peter Bloomfield
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch against cvs--for discussion (29.48 KB, patch)
2003-08-25 12:58 UTC, Peter Bloomfield
none Details | Review
new patch against current cvs (38.44 KB, patch)
2003-08-28 13:31 UTC, Peter Bloomfield
none Details | Review

Description Peter Bloomfield 2003-08-25 12:57:58 UTC
See http://mail.gnome.org/archives/balsa-list/2003-August/msg00049.html and
the rest of the thread.
Comment 1 Peter Bloomfield 2003-08-25 12:58:51 UTC
Created attachment 19486 [details] [review]
Patch against cvs--for discussion
Comment 2 Peter Bloomfield 2003-08-28 13:30:41 UTC
The following patch implements deferred local scanning, in much the
same way as IMAP scanning is deferred. When installed, Balsa scans the
local folder tree only as far as visible mailboxes (and one more
level, to detect which have children). It scans further as the folder
tree is expanded in the GUI.

This patch doesn't support any user configuration options for
controlling the depth of local scanning; it simply picks up a value
from balsa_app.local_scan_depth, which is hard-coded as 1. User
control could easily be added, alongside the IMAP scan depth option.

Some timings for simply starting and closing Balsa (indicative--not
carefully controlled!):

Test situation:
3161 mailboxes
                                                                     
          
cvs first time:
real    0m13.615s
user    0m8.220s
sys     0m0.320s
                                                                     
          
cvs next time:
real    0m37.044s
user    0m29.570s
sys     0m0.450s
                                                                     
          
deferred scanning first time:
real    0m18.169s
user    0m12.270s
sys     0m0.130s
                                                                     
          
deferred scanning next time:
real    0m4.479s
user    0m0.730s
sys     0m0.090s

The `cvs first time' has no views in the config file, but adds them on
exit, so `cvs next time' has the startup delay. The `deferred scanning
first time' has compatibility code which loads the views, so it
*incurs* the startup delay, but saves only a few, so `deferred
scanning next time' has neither startup nor exit delay.
Comment 3 Peter Bloomfield 2003-08-28 13:31:34 UTC
Created attachment 19581 [details] [review]
new patch against current cvs
Comment 4 Pawel Salek 2004-05-25 21:00:23 UTC
Did it get in?
Comment 5 Peter Bloomfield 2004-05-25 21:48:09 UTC
No--the deferred scanning resulted in the horrible closing/reopening of all open
mailboxes, once per folder added to the tree--it was really ugly.

But...with the new scheme and no more balsa-mblist-repopulate, it should work
just fine.  The patch is probably now hopelessly out of date, though, so it'll
be a little work.
Comment 6 Peter Bloomfield 2004-11-16 13:54:33 UTC
Implemented.  Resolving as FIXED...