Bug 244503

This is a big problem, and likely to be a blocker for David and me.

Thomas, could you please try with the following patch and let me know how much faster it is for you then. It disabled Nepomuk access in the message list, which gives me a 3x speedup without having Nepomuk running at all, so likely more if there has been a working Nepomuk before. This is of course not the solution to this problem, but would indicate that I'm looking in the right direction to fix this, since I cannot confirm the numbers posted here on my system, loading 10k messages takes only ~2.5sec here.

Index: core/messageitem.cpp
===================================================================
--- core/messageitem.cpp        (revision 1149821)
+++ core/messageitem.cpp        (working copy)
@@ -232,6 +232,7 @@
   // TODO: The tag pointers here could be shared between all items, there really is no point in
   //       creating them for each item that has tags

+#if 0
   const Nepomuk::Resource resource( mNepomukResourceUri );
   const QList< Nepomuk::Tag > nepomukTagList = resource.tags();
   if ( !nepomukTagList.isEmpty() ) {
@@ -267,6 +268,7 @@
       mTagList->append( messageListTag );
     }
   }
+#endif
 }

 QList<MessageItem::Tag*> MessageItem::Private::getTagList() const
@@ -300,12 +302,14 @@
   if ( d->mAnnotationStateChecked )
     return d->mHasAnnotation;

+#if 0
   Nepomuk::Resource resource( d->mNepomukResourceUri );
   if ( resource.hasProperty( QUrl( Nepomuk::Resource::descriptionUri() ) ) ) {
     d->mHasAnnotation = !resource.description().isEmpty();
   } else {
     d->mHasAnnotation = false;
   }
+#endif

   d->mAnnotationStateChecked = true;
   return d->mHasAnnotation;

SVN commit 1150006 by tilladam:

Avoid initializing the tags list from nepomuk for sizehint calculation.

Since the font for each message can be overridden by a tag, and since
the font size needs to be known at row height calculation time, the
tags list ended up being populated for each item, during message list
creation, which, since it's a nepomuk roundtrip or two, was very slow.
This change makes the font only take the tag into account when the tags
list is already populated. Not perfect, as for non visible items the
height might be off, if they have font override tags, but I guess we
can live with the consequences. It's rather a fringe feature anyhow.

CCBUG: 244503

 M  +12 -0     messageitem.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1150006

Fixed according to Thomas. Ludwig, please reopen if it still happens for you.

*** Bug 243829 has been marked as a duplicate of this bug. ***