Bug 260865

Hej Andre,

with 'go offline' you mean shutdown the network connection or select 'Work offline' in the context actions?

Ciao,
Tobias

Shutdown the network connection.

correct OS.

Windows CE version 20110112 old akonadi agent server.
Don't see the address completion popup in the composer receiver tab.
Entered some letters. Nothing happens. The mobile was online.

I am reopening this bug as i belive it is still valid and the test from Comment #4 that caused this bug to be set to resolved did not test the original scenario.

The steps to reproduce from this bugs description are no longer valid since the online/offline status is now known correctly by the application. But i expect this problem still to exist under bad connection scenarios. (Where the device is technically online but has lots of packet loss and a very high latency)

I've assigned this bug to me to give it a retest.

Still valid, if the connection to the server is bad, it appears that the addresselineedit freezes up until it gets an answer.

In my current test the device completly froze i was not even able to turn it off by hitting the shutdown button. Here is the debug output surrunding that:


004768/1//kmail-mobile(198485930)/ void __cdecl KPIM::AddresseeLineEdit::Private::slotAkonadiSearchResult(class KJob *): Found 0 groups 
004769/1//kmail-mobile(198485930)/ class QString __cdecl KuitSemanticsPrivate::semanticToVisualText(const class QString &,Kuit::Fmt::Var,Kuit::Fmt::Var) const: "Markup error in message {<kuil><interface>An:</interface><nl/>&nbsp;&nbsp;aheinecke@intevation.de<br/></k[...]
004769/1//ð):
004772/3//search done "strigisearch5447" 
004772/1//ø):
004775/1//ì):
004775/1//ü):
004775/1//kmail-mobile(198485930)/ class QString __cdecl KUrl::path(enum KUrl::AdjustPathOption) const:  "dc=demo,dc=kolab,dc=org" 
004775/1//kmail-mobile(198485930)/ class QString __cdecl KUrl::path(enum KUrl::AdjustPathOption) const:  "dc=demo,dc=kolab,dc=org" 
004775/1//kmail-mobile(198485930)/ void __cdecl KLDAP::LdapUrl::updateQuery(void): LDAP URL updateQuery(): "ldap://demo.kolab.org:389dc=demo,dc=kolab,dc=org?cn,mail,givenname,sn,objectClass?sub?%28%26%28%7C%28objectclass%3Dperson%29%28objectclass%3DgroupOfNames%29%2[...]
004775/1//ø):
004775/1//kmail-mobile(198485930)/ void __cdecl KLDAP::LdapClient::startQuery(const class QString &): LdapClient: Doing query: "ldap://demo.kolab.org:389dc=demo,dc=kolab,dc=org?cn,mail,givenname,sn,objectClass?sub?%28%26%28%7C%28objectclass%3Dperson%29%28objectclass[...]
004775/1//ô):ü):
004775/1//kmail-mobile(198485930)/ void __cdecl KLDAP::LdapClientSearch::startSearch(const class QString &): LdapClientSearch::startSearch() "&(|(objectclass=person)(objectclass=groupOfNames)(mail=*))(|(cn=aheinecke@intevation.de*)(mail=aheinecke@intevation.de*)(mai[...]
004775/1//kmail-mobile(198485930)/ void __cdecl KLDAP::LdapQueryJob::start(void): 
004775/1//ì):

And another one, this one is a bit more clear in my opionion:
Waiting -1 msecs for result?

04/1//Èi
010906/1//kmail-mobile(1250103102)/ class QString __cdecl KuitSemanticsPrivate::semanticToVisualText(const class QString &,Kuit::Fmt::Var,Kuit::Fmt::Var) const: "Markup error in message {<kuil><interface>An:</interface><nl/>&nbsp;&nbsp;aheine<br/></kuil>}: Entity 'n[...]
010906/1//ð)=
010906/1//kmail-mobile(1250103102)/ void __cdecl KPIM::AddresseeLineEdit::Private::akonadiHandlePending(void): Pending items:  0 
010912/1//ì)=
010912/1//kmail-mobile(1250103102)/ void __cdecl KLDAP::LdapUrl::updateQuery(void): LDAP URL updateQuery(): "ldap://demo.kolab.org:389dc=demo,dc=kolab,dc=org??base" 
010912/1//kmail-mobile(1250103102)/ class QString __cdecl KUrl::path(enum KUrl::AdjustPathOption) const:  "dc=demo,dc=kolab,dc=org" 
010912/1//kmail-mobile(1250103102)/ void __cdecl KLDAP::LdapUrl::updateQuery(void): LDAP URL updateQuery(): "ldap://demo.kolab.org:389dc=demo,dc=kolab,dc=org?" 
010912/1//kmail-mobile(1250103102)/ class QString __cdecl KUrl::path(enum KUrl::AdjustPathOption) const:  "dc=demo,dc=kolab,dc=org" 
010912/1//kmail-mobile(1250103102)/ void __cdecl KLDAP::LdapUrl::updateQuery(void): LDAP URL updateQuery(): "ldap://demo.kolab.org:389dc=demo,dc=kolab,dc=org?cn,mail,givenname,sn,objectClass" 
010912/1//kmail-mobile(1250103102)/ class QString __cdecl KUrl::path(enum KUrl::AdjustPathOption) const:  "dc=demo,dc=kolab,dc=org" 
010912/1//kmail-mobile(1250103102)/ void __cdecl KLDAP::LdapUrl::updateQuery(void): LDAP URL updateQuery(): "ldap://demo.kolab.org:389dc=demo,dc=kolab,dc=org?cn,mail,givenname,sn,objectClass?sub" 
010912/1//kmail-mobile(1250103102)/ class QString __cdecl KUrl::path(enum KUrl::AdjustPathOption) const:  "dc=demo,dc=kolab,dc=org" 
010912/1//kmail-mobile(1250103102)/ void __cdecl KLDAP::LdapUrl::updateQuery(void): LDAP URL updateQuery(): "ldap://demo.kolab.org:389dc=demo,dc=kolab,dc=org?cn,mail,givenname,sn,objectClass?sub?%28%26%28%7C%28objectclass%3Dperson%29%28objectclass%3DgroupOfNames%29%[...]
010912/1//kmail-mobile(1250103102)/ class QString __cdecl KUrl::path(enum KUrl::AdjustPathOption) const:  "dc=demo,dc=kolab,dc=org" 
010912/1//kmail-mobile(1250103102)/ void __cdecl KLDAP::LdapClient::startQuery(const class QString &): LdapClient: Doing query: "ldap://demo.kolab.org:389dc=demo,dc=kolab,dc=org?cn,mail,givenname,sn,objectClass?sub?%28%26%28%7C%28objectclass%3Dperson%29%28objectclas[...]
010912/1//kmail-mobile(1250103102)/ class QString __cdecl KUrl::path(enum KUrl::AdjustPathOption) const:  "dc=demo,dc=kolab,dc=org" 
010912/1//ü)=
010912/1//kmail-mobile(1250103102)/ void __cdecl KLDAP::LdapClientSearch::startSearch(const class QString &): LdapClientSearch::startSearch() "&(|(objectclass=person)(objectclass=groupOfNames)(mail=*))(|(cn=aheine*)(mail=aheine*)(mail=*@aheine*)(givenName=aheine*)(s[...]
010912/1//kmail-mobile(1250103102)/ void __cdecl KLDAP::LdapQueryJob::start(void): 
010912/1//kmail-mobile(1250103102)/ class QString __cdecl KUrl::path(enum KUrl::AdjustPathOption) const:  "dc=demo,dc=kolab,dc=org" 
010912/1//kmail-mobile(1250103102)/ int __cdecl KLDAP::LdapOperation::search(const class KLDAP::LdapDN &,KLDAP::LdapUrl::Scope,const class QString &,const class QStringList &): asyncSearch() base=" "dc=demo,dc=kolab,dc=org" " scope= 2 filter=" "(&(|(objectclass=pers[...]
010913/1//kmail-mobile(1250103102)/ int __cdecl KLDAP::LdapOperation::waitForResult(int,int): ( 8 , -1 ): Waiting -1 msecs for result. Attempt # 1

The LDAP query code should be executed in its own thread, so even if it blocks in KLDAP::LdapOperation::waitForResult, it shouldn't block the complete application.

I agree that having it in a thread is neccessary so that latencies do not freeze the application, but for the blocking i do not know if this will help.

Since it does not block only our application but in fact the complete device (Can only be unblocked by taking out the battery), i wonder if this is even solved by putting it in a thread.

It might be that because we run our frontends with above normal priority that we run into this fill block. It might as well be that we are running into a problem with the ldap implemenation in Windows CE.

Git commit 570542e8f85101ade75a7cad80007fd048dd8585 by Tobias Koenig.
Committed on 28/02/2011 at 11:06.
Pushed by tokoe into branch 'komo3'.

Fix blocking LDAP search queries

Make sure the LDAP search queries are really executed in their
own thread, so that weak network connections or slow servers to
not block the UI.

BUG: 260865

M  +5    -0    libkdepim/ldap/ldapqueryjob.cpp     
M  +3    -0    libkdepim/ldap/ldapqueryjob.h     
M  +1    -1    libkdepim/ldap/ldapsession.cpp     

http://commits.kde.org/kdepim/570542e8f85101ade75a7cad80007fd048dd8585

Retested works now on connection loss and even with good connection scenarios you can fluently type in the addresseelinedit.