3:21 p.m.
http://bugzilla.moblin.org/show_bug.cgi?id=5021
Summary: It takes 11 seconds to launch SyncUI service when EDS
has about 2500 records
Classification: Moblin Projects
Product: SyncEvolution
Version: upstream
Platform: Netbook
OS/Version: Moblin Linux
Status: NEW
Severity: major
Priority: Undecided
Component: GTK UI
AssignedTo: jku(a)linux.intel.com
ReportedBy: jingke.zhang(a)intel.com
QAContact: jingke.zhang(a)intel.com
CC: ross.burton(a)intel.com, yanshuang.zheng(a)intel.com,
shuang.wan(a)intel.com, syncevolution(a)lists.intel.com
Image: distro 20090803-001
Description:
=====================
This is found in our stress test.
After I synced about 2500 PIM Data (addressbook + events + tasks), the syncUI
launching time is much slow than before. It stays in "non service" status for
about 11 seconds (see my attachment), then the scheduleworld service can be
ready for use. The slowness can be also be met when switching services (with
thousands of PIM data).
If my data number is small (<100), the syncUI can be launched very quickly. Can
not imagine why this slowness relates to EDS. CC to Ross. Thanks!
Steps:
====================
1. Prepare 2500 data (contacts or tasks) in scheduleworld server.
2. Do "two-way" syncing with the service.
3. Reboot machine. Launch SyncUI again, you will find the launching time is
about 11 seconds longer.
--
Configure bugmail: http://bugzilla.moblin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching someone on the CC list of the bug.
3:41 p.m.
New subject: [Bug 5021] It takes 11 seconds to launch SyncUI service when EDS has about 2500 records
http://bugzilla.moblin.org/show_bug.cgi?id=5021
--- Comment #2 from shuangeeer <yanshuang.zheng(a)intel.com> 2009-08-03 23:41:03 ---
It happens to me that after modifying the configuration via command line,
sync-ui needs several seconds to launch and load configurations, sometimes even
failed to read configurations. reproduce 2 out of 6 times.
--
Configure bugmail: http://bugzilla.moblin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching someone on the CC list of the bug.
6:15 p.m.
New subject: [Bug 5021] [Stress] It takes 11 seconds to launch SyncUI service with 2500 PIM data
http://bugzilla.moblin.org/show_bug.cgi?id=5021
ZhangJingke <jingke.zhang(a)intel.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Summary|It takes 11 seconds to |[Stress] It takes 11
|launch SyncUI service when |seconds to launch SyncUI
|EDS has about 2500 records |service with 2500 PIM data
Severity|major |normal
--- Comment #3 from ZhangJingke <jingke.zhang(a)intel.com> 2009-08-04 02:15:48 ---
Add 'stress' to this title. I tried a common scenario: 300 data. With 300
contacts, the syncUI launching time is normal (within 1 second).
So download this priority to normal. Thanks!
--
Configure bugmail: http://bugzilla.moblin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching someone on the CC list of the bug.
2:02 a.m.
New subject: [Bug 5021] [Stress] It takes 11 seconds to launch SyncUI service with 2500 PIM data
http://bugzilla.moblin.org/show_bug.cgi?id=5021
--- Comment #4 from Ross Burton <ross.burton(a)intel.com> 2009-08-05 10:02:10 ---
Why does the sync UI need to read the entire addressbook before doing anything?
--
Configure bugmail: http://bugzilla.moblin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching someone on the CC list of the bug.
2:34 a.m.
New subject: [Bug 5021] [Stress] It takes 11 seconds to launch SyncUI service with 2500 PIM data
http://bugzilla.moblin.org/show_bug.cgi?id=5021
pohly <patrick.ohly(a)intel.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |patrick.ohly(a)intel.com
--- Comment #5 from pohly <patrick.ohly(a)intel.com> 2009-08-05 10:34:23 ---
(In reply to comment #4)
Why does the sync UI need to read the entire addressbook before doing
anything?
It shouldn't. I'm not even sure it does: I have done a quick grep of the
source, without finding what I was suspecting as the culprit. I was hoping that
Jussi (on vacation right now) would know more, so I haven't investigated
further.
--
Configure bugmail: http://bugzilla.moblin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching someone on the CC list of the bug.
2:33 a.m.
New subject: [Bug 5021] [Stress] It takes 11 seconds to launch SyncUI service with 2500 PIM data
http://bugzilla.moblin.org/show_bug.cgi?id=5021
--- Comment #6 from jku <jku(a)linux.intel.com> 2009-08-10 10:33:22 ---
I can reproduce this.
(In reply to comment #5)
(In reply to comment #4)
> Why does the sync UI need to read the entire addressbook before doing anything?
It shouldn't. I'm not even sure it does: I have done a quick grep of the
source, without finding what I was suspecting as the culprit. I was hoping that
Jussi (on vacation right now) would know more, so I haven't investigated
further.
Ah, but I do muck with sources... I'm guessing this is the problem: the client
asks dbus server for service configuration, and the server does this to ensure
the source is supported locally:
/* check whether we have support locally */
EvolutionSyncSourceParams params(name, config->getSyncSourceNodes(name),
"");
auto_ptr<EvolutionSyncSource>
syncSource(EvolutionSyncSource::createSource(params, false));
try {
local = FALSE;
if (syncSource.get()) {
syncSource->open();
local = TRUE;
}
}
Patrick, is this going to be slow with thousands of items? (I'll test tomorrow,
but I'm guessing you'll know without testing).
--
Configure bugmail: http://bugzilla.moblin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching someone on the CC list of the bug.
2:39 a.m.
New subject: [Bug 5021] [Stress] It takes 11 seconds to launch SyncUI service with 2500 PIM data
http://bugzilla.moblin.org/show_bug.cgi?id=5021
--- Comment #7 from jku <jku(a)linux.intel.com> 2009-08-10 10:39:26 ---
By the way, one reason for overall slowness of the system can be people panel,
it's not made to handle this many contacts (initially loading 2500 contacts
takes almost half a minute).
--
Configure bugmail: http://bugzilla.moblin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching someone on the CC list of the bug.
4:23 p.m.
New subject: [Bug 5021] [Stress] It takes 11 seconds to launch SyncUI service with 2500 PIM data
http://bugzilla.moblin.org/show_bug.cgi?id=5021
--- Comment #8 from pohly <patrick.ohly(a)intel.com> 2009-08-11 00:23:18 ---
(In reply to comment #6)
I can reproduce this.
(In reply to comment #5)
> (In reply to comment #4)
> > Why does the sync UI need to read the entire addressbook before doing
anything?
>
> It shouldn't. I'm not even sure it does: I have done a quick grep of the
> source, without finding what I was suspecting as the culprit. I was hoping that
> Jussi (on vacation right now) would know more, so I haven't investigated
> further.
Ah, but I do muck with sources... I'm guessing this is the problem: the client
asks dbus server for service configuration, and the server does this to ensure
the source is supported locally:
I was expecting something like this, but it shouldn't be slow. What I was
looking for was a call to checkStatus(), because that really has to read all
items (to get the REV string of them - we need a better API...).
/* check whether we have support locally */
EvolutionSyncSourceParams params(name, config->getSyncSourceNodes(name),
"");
auto_ptr<EvolutionSyncSource>
syncSource(EvolutionSyncSource::createSource(params, false));
try {
local = FALSE;
if (syncSource.get()) {
syncSource->open();
local = TRUE;
}
}
The open() call translates into:
* e_book_get_addressbooks()
* e_book_new_system_addressbook()
* e_book_open()
Patrick, is this going to be slow with thousands of items? (I'll
test tomorrow,
but I'm guessing you'll know without testing).
Sorry, I don't know whether e_book_open() really is slow with many items.
--
Configure bugmail: http://bugzilla.moblin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching someone on the CC list of the bug.
4:35 p.m.
New subject: [Bug 5021] [Stress] It takes 11 seconds to launch SyncUI service with 2500 PIM data
http://bugzilla.moblin.org/show_bug.cgi?id=5021
--- Comment #9 from Ross Burton <ross.burton(a)intel.com> 2009-08-11 00:35:39 ---
e_book_open() will scale with the size of the database, but it shouldn't be
taking 11 seconds for 2500 contacts. That said, a minimal test case which just
opens the addressbook will prove or disprove this hypothesis.
--
Configure bugmail: http://bugzilla.moblin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching someone on the CC list of the bug.
5:46 p.m.
New subject: [Bug 5021] [Stress] It takes 11 seconds to launch SyncUI service with 2500 PIM data
http://bugzilla.moblin.org/show_bug.cgi?id=5021
--- Comment #10 from jku <jku(a)linux.intel.com> 2009-08-11 01:45:59 ---
(In reply to comment #9)
e_book_open() will scale with the size of the database, but it
shouldn't be
taking 11 seconds for 2500 contacts. That said, a minimal test case which just
opens the addressbook will prove or disprove this hypothesis.
Nope, I found the problem: I'm always doing a EvolutionSyncConfig.copy(), and I
think I should only do it when the config does not exist yet (to get data from
template).
So it's my bug. I won, thanks for playing.
--
Configure bugmail: http://bugzilla.moblin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching someone on the CC list of the bug.
6:36 p.m.
New subject: [Bug 5021] [Stress] It takes 11 seconds to launch SyncUI service with 2500 PIM data
http://bugzilla.moblin.org/show_bug.cgi?id=5021
pohly <patrick.ohly(a)intel.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution| |FIXED
--- Comment #11 from pohly <patrick.ohly(a)intel.com> 2009-08-12 02:36:32 ---
It wasn't related to EDS. The slowness was caused by (unnecessarily) copying a
large .ini file with the uid/revision data into itself. That code has to do
O(n^2) key comparisons to determine that the values haven't changed.
Optimizations are conceivable (bug #5377), but for this issue the right
solution was to not do the copying at all. Merged into master, will be in 0.9:
commit 93731226b975963bf62d4fb8dde8b76ff3e40476
Author: Jussi Kukkonen <jku(a)linux.intel.com>
Date: Tue Aug 11 22:33:13 2009 +0300
dbus server: copy template configuration in a sane way
All calls to syncevo_get_server_config() resulted in
EvolutionSyncConfig.copy() call, which can be expensive.
There is no reason to copy except when copying from a
template.
--
Configure bugmail: http://bugzilla.moblin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching someone on the CC list of the bug.
9:15 p.m.
New subject: [Bug 5021] [Stress] It takes 11 seconds to launch SyncUI service with 2500 PIM data
http://bugzilla.moblin.org/show_bug.cgi?id=5021
pohly <patrick.ohly(a)intel.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Resolution|FIXED |INTEGRATED
--- Comment #12 from pohly <patrick.ohly(a)intel.com> 2009-08-25 05:15:42 ---
These fixes were all in SyncEvolution 0.9, which was integrated into Moblin
Trunk and the 2.1 branch. They are not integrated into Moblin 2.0 yet.
--
Configure bugmail: http://bugzilla.moblin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching someone on the CC list of the bug.
6:36 p.m.
New subject: [Bug 5021] [Stress] It takes 11 seconds to launch SyncUI service with 2500 PIM data
http://bugzilla.moblin.org/show_bug.cgi?id=5021
ZhangJingke <jingke.zhang(a)intel.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|RESOLVED |VERIFIED
--- Comment #13 from ZhangJingke <jingke.zhang(a)intel.com> 2009-09-10 02:36:16 ---
(In reply to comment #12)
These fixes were all in SyncEvolution 0.9, which was integrated into
Moblin
Trunk and the 2.1 branch. They are not integrated into Moblin 2.0 yet.
With 2.1-alpha-20090909-001 image, even my PIM data number is about 2600. The
sync-ui launch speed and service changing speed are normal now. Thanks!
--
Configure bugmail: http://bugzilla.moblin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching someone on the CC list of the bug.
6:37 p.m.
New subject: [Bug 5021] [Stress] It takes 11 seconds to launch SyncUI service with 2500 PIM data
http://bugzilla.moblin.org/show_bug.cgi?id=5021
--- Comment #14 from ZhangJingke <jingke.zhang(a)intel.com> 2009-09-10 02:37:03 ---
Verify-this-before-2.1
--
Configure bugmail: http://bugzilla.moblin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching someone on the CC list of the bug.
4189
days inactive
4226
days old
syncevolution-issues@syncevolution.org
13 comments
1 participants
participants (1)
-
bugzilla＠moblin.org