GitLab

This covers cases the conversion tool didn't previously handle.

Instead of maintaining 12k lines of hand-written protocol code, we
specify the protocol in an XML and use a custom-written generator
that generates the code for us.

It's not only much easier to modify the protocol - we only need to
change a single thing in the XML instead of touching several places
of the implementation - but it's also much safer, as there's less
risk of accidentally introducing a bug in the code.

The major difference between the original hand-written code and the
generated code is that we no longer use QSharedDataPointer and virtual
methods in the Private classes, but instead all members are directly
in the command clas with most getters and setters inlined. This means
that copying commands is quite costly, so we pass them around as
QSharedPointers or const references. This should give us a tiny little
bit more performance.

One of the bottlenecks in the current design is the individual Item
retrieval via requestItemDelivery(). The Server does a request for
the first Item, then waits for the Resource to deliver before requesting
the next Item and so on. This patch changes the signature of the
requestItemDelivery() DBus method to support passing multiple Items
(Item IDs to be specific) at once. The patch also deprecates
ResourceBase::retrieveItem() in favor of a newly introduces
ResourceBase::retrieveItems() overload (which takes Item::List and
set of part names as arguments) allowing resource implementations to
process all the requested Items in a single batch and pass them back
to the Server. This should be much faster than querying the Items one
by one.

This change does not remove the retrieveItem() method, but only
deprecates it. When Resource only implements retrieveItem() and not
the new retrieveItems() then retrieveItems() will split the ResourceScheduler
Task into many single-Item Tasks and will call retrieveItem() for
each of them.

The next step is to get rid of DBus in here and use a dedicated
command bus where the Server could request the Items via the Protocol
and would be handed the Items back through the bus.

Once all resources are ported to the new API, the old retrieveItem()
method should be removed.

Use a delay mechanism instead to prevent Session deadlocks.

Nothing compiles yet due to everything including everything and not
everything being ported yet, but we have all Collection jobs and
most Items jobs (theoretically) done.

Disabled some code that does not compile or has dependency issues

REVIEWS: 116285 116286 116287 116288 116289 116290 116291 116292 116293 116294 116295 116296 116297 116298 116299 116300 116301 116302 116303 116304 116305 116306 116307 116308 116309 116310 116311 116312 116313 116314 116315 116316 116317 116318 116319 116320 116321 116322 116323

One can now configure how the ItemFetchJob should return the results -
in batches, individually, or via the getter functions.

This is especially useful when fetching very large collections for
indexing since we can keep indexing them and dropping them. Otherwise
they would all be stored internally in the ItemFetchJob and would
consume a lot more memory.

REVIEW: 114505

without causing a detaching of all the items.

Reviewed by krake.

svn path=/trunk/KDE/kdepimlibs/; revision=1175579

svn path=/trunk/KDE/kdepimlibs/; revision=1146635

svn path=/trunk/KDE/kdepimlibs/; revision=1108339

svn path=/trunk/KDE/kdepimlibs/; revision=1072342

svn path=/trunk/KDE/kdepimlibs/; revision=1024353

has support for that since some time already.

svn path=/trunk/KDE/kdepimlibs/; revision=1024352

Merged revisions 1021447,1021502,1021505-1021506,1021510-1021512,1021524,1021529,1021589 via svnmerge from 
https://vkrause@svn.kde.org/home/kde/trunk/KDE/kdepimlibs

................
  r1021447 | ervin | 2009-09-09 09:48:54 +0200 (Wed, 09 Sep 2009) | 7 lines
  
  Also trigger the CollectionMoveJob on drops. First step at having move
  support in ETM.
  
  Needs more work for a complete support, but I'm working on some other
  stuff now.
................
  r1021502 | tokoe | 2009-09-09 14:01:53 +0200 (Wed, 09 Sep 2009) | 2 lines
  
  Add missing @since 4.4 tags
................
  r1021505 | ervin | 2009-09-09 14:32:11 +0200 (Wed, 09 Sep 2009) | 3 lines
  
  Adding an improved collectionChanged() method to Observer2 so that we
  get a chance to know which parts of the collection changed.
................
  r1021506 | tokoe | 2009-09-09 14:36:09 +0200 (Wed, 09 Sep 2009) | 2 lines
  
  Extend apidocs by examples
................
  r1021510 | tmcguire | 2009-09-09 14:41:33 +0200 (Wed, 09 Sep 2009) | 163 lines
  
  Block cyclic merges.
  SVN_SILENT
  SVN_MERGE
  --
  Blocked revisions 1015922,1015925,1015927-1015928,1017872,1019403,1019453,1019804 via svnmerge
  
  ................
    r1015922 | tmcguire | 2009-08-26 18:11:38 +0200 (Wed, 26 Aug 2009) | 17 lines
    
    Block cyclic merge
    SVN_SILENT
    Blocked revisions 1013754 via svnmerge
    
    ........
      r1013754 | winterz | 2009-08-20 19:46:26 +0200 (Thu, 20 Aug 2009) | 10 lines
      
      Backport r1013753 by winterz from trunk to the 4.3 branch:
      
      merge SVN commit 1013751 from e4 kdepimlibs to trunk
      
      make sure to convert Attachment labels from Utf8
      
      Sorry Thomas, you'll need to record this for us.
      MERGE: 4.3
    ........
  ................
    r1015925 | tmcguire | 2009-08-26 18:13:23 +0200 (Wed, 26 Aug 2009) | 33 lines
    
    Merged revisions 1013328-1013329,1013331 via svnmerge from 
    svn+ssh://tmcguire@svn.kde.org/home/kde/branches/KDE/4.3/kdepimlibs
    
    ........
      r1013328 | tmcguire | 2009-08-19 17:25:59 +0200 (Wed, 19 Aug 2009) | 11 lines
      
      Backport r1012151 by tmcguire from trunk to the 4.3 branch:
      
      Don't remove my whitespace if I set the PreserceSpaces flag.
      Strange is that the code seems to do that intentionally, but it is so old that svn annotate doesn't say anything useful.
      
      
      Add small unit test for that.
      
      BUG :204101
    ........
      r1013329 | tmcguire | 2009-08-19 17:26:48 +0200 (Wed, 19 Aug 2009) | 7 lines
      
      Backport r1012907 by tmcguire from trunk to the 4.3 branch:
      
      Now I know why the code didn't simply replace all spaces with nbsps: That would break
      word-wrapping.
      Therefore, take this into account and adjust the tests for it.
    ........
      r1013331 | tmcguire | 2009-08-19 17:27:44 +0200 (Wed, 19 Aug 2009) | 7 lines
      
      Backport r1012912 by tmcguire from trunk to the 4.3 branch:
      
      Fix spaces one and for all, hopefully.
      I forgot that in a sequence of spaces, all must be non-breaking, otherwise the breaking one
      isn't taken into account.
    ........
  ................
    r1015927 | tmcguire | 2009-08-26 18:14:46 +0200 (Wed, 26 Aug 2009) | 13 lines
    
    Merged revisions 1013810 via svnmerge from 
    svn+ssh://tmcguire@svn.kde.org/home/kde/branches/KDE/4.3/kdepimlibs
    
    ........
      r1013810 | winterz | 2009-08-20 21:43:58 +0200 (Thu, 20 Aug 2009) | 7 lines
      
      Backport r1013809 by winterz from trunk to the 4.3 branch:
      
      don't crash if the calendar passed to the FreeBusy is zero.
      CCBUG: 186597
      MERGE: 4.3
    ........
  ................
    r1015928 | tmcguire | 2009-08-26 18:15:38 +0200 (Wed, 26 Aug 2009) | 15 lines
    
    Merged revisions 1015198 via svnmerge from 
    svn+ssh://tmcguire@svn.kde.org/home/kde/branches/KDE/4.3/kdepimlibs
    
    ........
      r1015198 | mkoller | 2009-08-24 23:22:31 +0200 (Mon, 24 Aug 2009) | 9 lines
      
      Backport r1015195 by mkoller from trunk to the 4.3 branch:
      
      CCBUG: 202445
      
      Try much better to discover a URL, even if it is enclosed with
      some sort of brackets/quotes and even interrupted by line breaks or white space.
      RFC3986 explains this in appendix C
    ........
  ................
    r1017872 | winterz | 2009-08-31 21:16:15 +0200 (Mon, 31 Aug 2009) | 26 lines
    
    Merged revisions 1017406,1017525,1017535 via svnmerge from 
    https://svn.kde.org/home/kde/branches/kdepim/enterprise/kdepim
    
    ........
      r1017406 | winterz | 2009-08-30 11:17:58 -0400 (Sun, 30 Aug 2009) | 5 lines
      
      follow recent changes in the trunk incidenceformatter for the displayViewer.
      part of the fixes for kolab/issue3618
      
      MERGE:e4
    ........
      r1017525 | winterz | 2009-08-30 16:31:30 -0400 (Sun, 30 Aug 2009) | 6 lines
      
      part 2: 
      follow recent changes in the trunk incidenceformatter for the displayViewer.
      part of the fixes for kolab/issue3618
      
      MERGE: e4
    ........
      r1017535 | winterz | 2009-08-30 16:57:03 -0400 (Sun, 30 Aug 2009) | 4 lines
      
      follow trunk for part3 of fixes for displayView formatting
      continuing improvements for kolab/issue3816
      MERGE: e4
    ........
  ................
    r1019403 | winterz | 2009-09-03 16:21:38 +0200 (Thu, 03 Sep 2009) | 14 lines
    
    Merged revisions 1019400 via svnmerge from 
    https://svn.kde.org/home/kde/branches/kdepim/enterprise/kdepim
    
    ........
      r1019400 | winterz | 2009-09-03 10:03:23 -0400 (Thu, 03 Sep 2009) | 7 lines
      
      backport SVN commit 1017820 by winterz from trunk to e35:
      
      Provide a new toolTipStr() and extensiveDisplayStr() that print the Incidence's calendar.
      Deprecate the old toolTipStr() and extensiveDisplayStr().
      
      MERGE: e4
    ........
  ................
    r1019453 | winterz | 2009-09-03 19:02:00 +0200 (Thu, 03 Sep 2009) | 16 lines
    
    Merged revisions 1019450 via svnmerge from 
    https://svn.kde.org/home/kde/branches/kdepim/enterprise/kdepim
    
    ........
      r1019450 | winterz | 2009-09-03 12:56:08 -0400 (Thu, 03 Sep 2009) | 10 lines
      
      backport SVN commit 1018577 by winterz from trunk to e35:
      
      for displayView:
      + also show non-uri attachments
      + move the Calendar field prints to displayViewFormat(Event|Todo|Journal)
        instead of doing that in displayViewFormatHeader, because that screws-up
        the table row prints
      MERGE: e4
    ........
  ................
    r1019804 | winterz | 2009-09-04 14:34:56 +0200 (Fri, 04 Sep 2009) | 5 lines
    
    merge SVN commit 1019799 by winterz from trunk to e4:
    
    Fix "kmail: Kiosk-based [$e] config variables are not evaluated for file emailidentities"
    Thanks for the patch Jörg!
  ................
................
  r1021511 | tmcguire | 2009-09-09 14:42:58 +0200 (Wed, 09 Sep 2009) | 11 lines
  
  SVN_MERGE
  --
  Merged revisions 1020958 via svnmerge from 
  svn+ssh://tmcguire@svn.kde.org/home/kde/branches/kdepim/enterprise4/kdepimlibs
  
  ........
    r1020958 | tmcguire | 2009-09-07 20:06:13 +0200 (Mon, 07 Sep 2009) | 1 line
    
    Fix typo
  ........
................
  r1021512 | ervin | 2009-09-09 14:45:16 +0200 (Wed, 09 Sep 2009) | 2 lines
  
  Of course forgot a @since 4.4... meh.
................
  r1021524 | ervin | 2009-09-09 15:12:32 +0200 (Wed, 09 Sep 2009) | 3 lines
  
  Get rid of the "collectionMoved() from Observer2 hides collectionMoved()
  from Observer" warning. It's perfectly intended here.
................
  r1021529 | ervin | 2009-09-09 15:19:12 +0200 (Wed, 09 Sep 2009) | 3 lines
  
  Another warning related to collectionChanged (AH! got it right now!)
  which goes away.
................
  r1021589 | djarvie | 2009-09-09 17:41:25 +0200 (Wed, 09 Sep 2009) | 2 lines
  
  Change licence to LGPL
................

svn path=/branches/work/akonadi-ports/kdepimlibs/; revision=1021825

svn path=/trunk/KDE/kdepimlibs/; revision=1021502

svn path=/branches/work/akonadi-ports/kdepimlibs/; revision=1021125

not return a retrieved item but the requested one.

svn path=/branches/work/akonadi-ports/kdepimlibs/; revision=1020912

Merged revisions 1018774,1018800,1018808,1018953,1018955,1018976,1018993,1019010 via svnmerge from 
https://vkrause@svn.kde.org/home/kde/trunk/KDE/kdepimlibs

........
  r1018774 | tokoe | 2009-09-02 09:20:32 +0200 (Wed, 02 Sep 2009) | 2 lines
  
  Relicense my code under LGPL
........
  r1018800 | tokoe | 2009-09-02 10:46:16 +0200 (Wed, 02 Sep 2009) | 3 lines
  
  Factor out the code for ItemFetchScope and result parsing
  from ItemFetchJob into ProtocolHelper
........
  r1018808 | tokoe | 2009-09-02 11:14:17 +0200 (Wed, 02 Sep 2009) | 2 lines
  
  fix documentation
........
  r1018953 | cgiboudeaux | 2009-09-02 13:30:41 +0200 (Wed, 02 Sep 2009) | 1 line
  
  Fix a linker issue with akonadi-contact when used outside of kdepimlibs
........
  r1018955 | cgiboudeaux | 2009-09-02 13:31:55 +0200 (Wed, 02 Sep 2009) | 1 line
  
  Bump version.
........
  r1018976 | weilbach | 2009-09-02 15:16:07 +0200 (Wed, 02 Sep 2009) | 2 lines
  
  kDebug() fixes; dateTime fixes; support for slug and moreText (summary); started to work around seperate category setting for MovableType;
........
  r1018993 | weilbach | 2009-09-02 16:46:42 +0200 (Wed, 02 Sep 2009) | 2 lines
  
  Fix category support for MovableType and WordpressBuggy. Just set the QStringList with BlogPost::setCategories() and you are done. The rest is worked around internally.
........
  r1019010 | weilbach | 2009-09-02 17:34:21 +0200 (Wed, 02 Sep 2009) | 2 lines
  
  Cleanup and implement the category updating for modifyPost and fetchPost as well. Should close #194357 btw. The official docs for metaweblog.createPost etc. don't define inline category definition, so we call mt.setPostCategories now in everything up from MovableType.
........

svn path=/branches/work/akonadi-ports/kdepimlibs/; revision=1019208

svn path=/trunk/KDE/kdepimlibs/; revision=1018808