<html>

 <body>

  <div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">

   <table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">

    <tr>

     <td>

      This is an automatically generated e-mail. To reply, visit:

      <a href="https://git.reviewboard.kde.org/r/123183/">https://git.reviewboard.kde.org/r/123183/</a>

     </td>

    </tr>

   </table>

   <br />

<table bgcolor="#e0e0e0" width="100%" cellpadding="12" style="border: 1px gray solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">

 <tr>

  <td>

   <h1 style="margin: 0; padding: 0; font-size: 10pt;">This change has been marked as submitted.</h1>

  </td>

 </tr>

</table>

<br />

<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">

 <tr>

  <td>

<div>Review request for Akonadi and Daniel Vrátil.</div>

<div>By Christian Mollekopf.</div>

<p style="color: grey;"><i>Updated Feb. 4, 2017, 11:24 a.m.</i></p>

<div style="margin-top: 1.5em;">

 <b style="color: #575012; font-size: 10pt;">Repository: </b>

akonadi

</div>

<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>

 <table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">

 <tr>

  <td>

   <pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Merged the kolab branch into master.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I tried fixing all coding style issues and all tests are passing again (except some of the link tests that are disabled).</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I currently can't push the branch due to too many commits, I'll follow up on that.</p>

<hr style="text-rendering: inherit;margin: 0;padding: 0;white-space: normal;border: 1px solid #ddd;line-height: inherit;" />

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">dbinitializer: Also insert items when Q_ASSERT is disabled.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Merge branch 'master' into mergeKolabIntegration</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Merge branch 'kolab/integration/1.12.0' into mergeKolabIntegration</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Conflicts:

    CMakeLists.txt

    autotests/private/imapparserbenchmark.cpp

    autotests/private/notificationmessagev2test.cpp

    autotests/private/notificationmessagev2test.h

    autotests/server/CMakeLists.txt

    autotests/server/collectionreferencetest.cpp

    autotests/server/dbtest_data/dbinit_mysql

    autotests/server/dbtest_data/dbinit_mysql_incremental

    autotests/server/dbtest_data/dbinit_psql

    autotests/server/dbtest_data/dbinit_psql_incremental

    autotests/server/dbtest_data/dbinit_sqlite

    autotests/server/dbtest_data/dbinit_sqlite_incremental

    autotests/server/dbtest_data/dbtest_data.qrc

    autotests/server/dbtest_data/unittest_dbupdate.xml

    autotests/server/dbtest_data/unittest_schema.xml

    autotests/server/handlerhelpertest.cpp

    cmake/modules/FindSqlite.cmake

    server/src/dbusconnectionpool.cpp

    server/src/dbusconnectionpool.h

    server/src/handler/move.cpp

    server/src/nepomuk/dbusconnectionpool.cpp

    server/src/nepomuk/dbusconnectionpool.h

    server/src/nepomuk/queryserviceclient.cpp

    server/tests/unittest/dbinitializertest_data/dbinit_mysql

    server/tests/unittest/dbinitializertest_data/dbinit_mysql_incremental

    server/tests/unittest/dbinitializertest_data/dbinit_psql

    server/tests/unittest/dbinitializertest_data/dbinit_psql_incremental

    server/tests/unittest/dbinitializertest_data/dbinit_sqlite

    server/tests/unittest/dbinitializertest_data/dbinit_sqlite_incremental

    server/tests/unittest/dbinitializertest_data/dbinitializertestdata.qrc

    server/tests/unittest/dbinitializertest_data/unittest_dbupdate.xml

    server/tests/unittest/dbinitializertest_data/unittest_schema.xml

    server/tests/unittest/dbtest_data/dbinit_mysql

    server/tests/unittest/dbtest_data/dbinit_mysql_incremental

    server/tests/unittest/dbtest_data/dbinit_psql

    server/tests/unittest/dbtest_data/dbinit_psql_incremental

    server/tests/unittest/dbtest_data/dbinit_sqlite

    server/tests/unittest/dbtest_data/dbinit_sqlite_incremental

    server/tests/unittest/dbtest_data/dbtest_data.qrc

    server/tests/unittest/dbtest_data/unittest_dbupdate.xml

    server/tests/unittest/dbtest_data/unittest_schema.xml

    src/private/notificationmessagev2.cpp

    src/private/notificationmessagev2_p.h

    src/private/notificationmessagev2_p_p.h

    src/private/notificationmessagev3.cpp

    src/private/notificationmessagev3_p.h

    src/private/protocol_p.h

    src/private/xdgbasedirs.cpp

    src/server/CMakeLists.txt

    src/server/akonadi.cpp

    src/server/akonadi.h

    src/server/cachecleaner.cpp

    src/server/cachecleaner.h

    src/server/clientcapabilities.cpp

    src/server/collectionscheduler.cpp

    src/server/connection.cpp

    src/server/connection.h

    src/server/dbusconnectionpool.cpp

    src/server/dbusconnectionpool.h

    src/server/handler.cpp

    src/server/handler/akappend.cpp

    src/server/handler/akappend.h

    src/server/handler/append.cpp

    src/server/handler/capability.cpp

    src/server/handler/colmove.cpp

    src/server/handler/create.cpp

    src/server/handler/fetch.cpp

    src/server/handler/fetchhelper.cpp

    src/server/handler/fetchhelper.h

    src/server/handler/fetchscope.cpp

    src/server/handler/fetchscope.h

    src/server/handler/list.h

    src/server/handler/modify.cpp

    src/server/handler/remove.cpp

    src/server/handler/scope.cpp

    src/server/handler/scope.h

    src/server/handler/store.cpp

    src/server/handler/store.h

    src/server/handler/subscribe.cpp

    src/server/handler/tagappend.cpp

    src/server/handler/tagfetchhelper.cpp

    src/server/handler/tagfetchhelper.h

    src/server/handler/tagremove.cpp

    src/server/handler/tagstore.cpp

    src/server/handlerhelper.cpp

    src/server/handlerhelper.h

    src/server/imapstreamparser.cpp

    src/server/imapstreamparser.h

    src/server/intervalcheck.cpp

    src/server/main.cpp

    src/server/notificationmanager.cpp

    src/server/notificationsource.cpp

    src/server/notificationsource.h

    src/server/preprocessormanager.cpp

    src/server/response.h

    src/server/search/agentsearchinstance.cpp

    src/server/search/searchmanager.cpp

    src/server/search/searchmanager.h

    src/server/search/searchtaskmanager.cpp

    src/server/search/searchtaskmanager.h

    src/server/storage/datastore.cpp

    src/server/storage/datastore.h

    src/server/storage/dbconfigpostgresql.cpp

    src/server/storage/dbconfigsqlite.cpp

    src/server/storage/dbinitializer.cpp

    src/server/storage/dbupdater.cpp

    src/server/storage/entity.h

    src/server/storage/itemqueryhelper.cpp

    src/server/storage/itemretrievalmanager.cpp

    src/server/storage/notificationcollector.cpp

    src/server/storage/notificationcollector.h

    src/server/storage/parthelper.cpp

    src/server/storage/parthelper.h

    src/server/storage/parttypehelper.cpp

    src/server/storage/querybuilder.cpp

    src/server/storage/querybuilder.h

    src/server/storage/storagedebugger.cpp

    src/server/storage/storagedebugger.h

    src/server/storagejanitor.cpp

    src/shared/akapplication.cpp

    src/shared/aktest.h</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Create task to other resources makes kmail unresponsible</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">The problem is that relations are only resoucewise allowed. At the

moment nobody can explain me, why this is needed. So I created a

workaround in disabling that check for now.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">KOLAB: 4513</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Prepare 1.12.0.1 release.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Fixed typo</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Try harder to find the user's home directory and the config therein on Windows.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">We need the definition of Collection, not just a forward declaration.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Fix build on Windows where we need to check for unistd.h</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Fix build at least on Windows by bringing in the Akonadi namespace.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Filter items by resource when fetching from a resource.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">A resource should never sync tag members from another resource.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Only emit a notification if somethin changed on the tag.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This avoids bouncing notifications back and forth betwenn resources

that update the remoteId.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Merge remote-tracking branch 'kolab/dev/fix_tagremote' into kolab/integration/1.12.0</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Do not cleanup all entries for a resouce if one tag is updated.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">If we delete all entries for one resource, than we loose the remote id

for the tags. Only delete the specific entry, that is inserted/updated

afterwards.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">KOLAB: 3541</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">NotificationSource: Send notifications to referenced collections.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">We no longer rely on referenced collections being monitored.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">NotificationSource: set the session to filter referenced notifications.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">LIST: filter referenced collections also with an invalid toplevel collection</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Otherwise we get unwanted collections from a toplevel list if another session

referenced the collection.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Notify about dereferenced collections.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Otherwise zanshin won't be notified about dereferenced collections.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Include Tag RID in Tag removal notification</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">When notification is removed, we need to pass along Tag RID, so that resource

can sync the change to server. However, Tags have per-resource RIDs, and regular

clients are not supposed to be aware of Tag RIDs at all. This means that we

need to emit a dedicated notification for each resource, that provided tag RID,

and one additional notification without RID for clients.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">The NotificationSource then filters out the notifications, so that notification

with a RID is delivered only to the resource that owns this particular RIDs.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Since NotificationSource is not aware which resource it belongs to, we use an

ugly hack and rely on the fact, that each ResourceBase ignores it's own session,

and that session is always called the same as the resource.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">TAGSTORE: Read the command until end, even when we are going to delete the tag</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">We must read the command until end, otherwise processing of subsequent

commands gets stuck.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">TAGSTORE: Remove tag when its last RID is unset by TAGSTORE command</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Resources use TAGSTORE to tell the Akonadi that tag has been removed on

the remote server. They simply unset RID of that tag (because tags have

per-resource RID). To emulate the situation, when all remote references

to a tag have been removed, we need to remove the tag in Akonadi server

and untag all items when the very last RID is removed by a resource.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Disabled the imapparser benchmark.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">It takes forever and tells us nothing without values to compare to.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">ImapStreamParser: Properly parse empty sequence sets.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">It used to insert -1. We require this to clear tags using an empty set.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Avoid trying to set negative timer intervals.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">resolveTags: support empty set of tags</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">We need this to clear tags.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Since we removed the notification compression, we also can't test it any longer.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Made linkhandlertest pass.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">RID operations are broken, but we don't currently need them.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">NotificationSource: Don't drop unsubscribe or enabled notifications</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Since the collection is already disabled we require an exception for this.

Otherwise the monitortest doesn't pass.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">NotificationSource: Make it possible to setExclusive</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Don't dispatch notifications about disabled collections</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">The only exception are exclusive subscribers, which expect even notifications

about disabled collections.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">For this we add a little hack: a flag that is set by NotificationCollector to

the NotificationMessage that says that the collection in this notification is

disabled. NotificationCollector knows, because it is passed the actual Collection

from caller. The presence of the flag is then evaluated by NotificationSource

when deciding whether to accept the notification or not. This is because

NotificationSource only works with what's in the notification, and querying

database for each Collection to check, whether it's enabled or not would be

very expensive.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">When Collection is disabled, the notification is dropped, unless the current

subscriber is exclusive or is monitoring the Collection explicitly, or the

Collection is referenced.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">CollectionScheduler: only try to schedule enabled or referenced collections in initScheduler()</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Otherwise we end up iterating over a growing QMap of already-scheduled collections

over and over again, which causes Akonadi server to take ages before it starts.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">NotificationCompression: Limit search to last 10 items, and remove modify after add.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Removing the modify after add avoids that each collection is initially indexed twice.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Reduced notification compression.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This are three squashed commits:</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Remove notification compression algorithm</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">The compression does not have much sense nowadays, as we rarely end up in

situation where merging notifications would result in performance improvement

(mostly because the most intensive operations now use batch notifications).

The only actual remaining use for the compression algorithm is in case of removal:</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">If there is a notification in the queue for a certain Item, and a removal

notification for the same Item is appended to the queue, it means that the Item

no longer exists in Akonadi and at time the first notification is delivered to

Monitor, EntityCache will no longer be able to retrieve the Item, which renders

the notification useless and we need to remove it to prevent EntityCache from

getting Item/CollectionFetchJob error.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">However when massive removal operation takes place (in case of Collections that

is, removal of Items is again delivered as Batch notification), the queue has

to be iterated over and over again - and as the queue grows, the algorithm

stops being an optimization and instead becomes a bottleneck.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">For that reason we dicided to remove the compression algorithm completely and

instead make sure, that Monitor (and EntityCache) on the client side can

properly handle the case of Item/Collection no longer existing in Akonadi at

the time the Notification is delivered.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Bring back notification compression - at lest partially</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Turned out that adding a new Collection generates one ADD and several

(usually 3 in my case) MODIFY notifications - since the MODIFY is

expensive and slows down initial sync, we compress it ito one notification again.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">All other notifications are dispatched directly without any compression.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Notification compression: further optimization to minimalize worst-case scenarios</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">We are now iterating over the list of pending notifications in reverse order,

because it's more probably that compressable notifications are at the end of

the queue rather at the beginning.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Iterating from the end also means that once we reach Add notification for the

same collection, we can break immediatelly and don't have to iterate through the

rest of the list, since there can be no other notifications for this Collection

before "Add"</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Preserve local-only flags during MERGE</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Some flags are never synced to server (because the server does not

support it for instance), so the resource cannot know about them when

syncing changes in the Item (another flags change, payload change,

whatever) and since ItemSync always crates non-incremental flags merging,

we would lost these local-only flags.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This hack ensures, that during MERGE when we have non-incremental flags

changes, certain local flags ($ATTACHMENT, $WATCHED, $INVITATION, ...)

are preserved.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">make Akonadi::Server::Connection to build for 12.04</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">handler::list: CollectionAttributs values are QByteArray</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">We should read/write them as QByteArray and not as QString</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Akonadi::Server: add a benchmark fetch test for a popolated collection</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">QueryBuilder::setForwardOnly: use ForwardOnly mode for sql query</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">* less RAM</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">LIST: comments and simplifications.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">LIST: Retrieve attributes also for collections that only have been included because they are part of the tree.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">LIST: The base collection listing should also succeed if the filter doesn't match.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Otherwise an ETM with a list filter cannot fetch collections that are part of the tree,

but don't match the tree. In particular it would also not receive updates from the

monitor for said collections.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Catch invalid parents (happendend to me once), so better make sure we catch this in non-debug builds.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">LIST: Retrieve attributes like mimetypes in a parallel running query.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">We still have to load attributes for ancestors into memory, but at least the

rest we can stream directly.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">LIST: list mimetypes directly instead of loading them into memory.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">It's faster and saves memory.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">LIST: Improved performance</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Instead of issuing at least 2 queries per collection (attributes and mimetype),

we collect all required data in queries for all collections. This improves

the list performance by a factor of ~3. Some more memory is used as a tradeoff,

but this currently doesn't seem to bad.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">FakeClient: Support for ignoring lines.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Useful for benchmarks with thousands of responses.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Support for writing StorageDebugger content to a file.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Handler time reporting</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">TAGSTORE: allow clearing the rid</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">FETCH-Tests and made collectioncontext non-persistent.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">FetchHandlerTest</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">TAGSTORE: Actually working tagupdates + test.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">With this it becomes possible to update the rid.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Relations support</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Conflicts:

    libs/notificationmessagev2.cpp

    libs/notificationmessagev2_p.h

    libs/notificationmessagev2_p_p.h

    libs/notificationmessagev3.cpp

    server/src/handler/fetchhelper.cpp

    server/src/handler/fetchscope.cpp

    server/src/handler/fetchscope.h

    server/src/notificationsource.cpp

    server/src/storage/datastore.cpp

    server/src/storage/notificationcollector.cpp

    server/src/storage/notificationcollector.h</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">LIST: Collection ancestor fetch scope</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">parentLookup is only used locally.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">LIST: Fix ancestor retrieval</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">My previous fix has introduced a regression that has broken ancestor retrieval,

this commit fixes that.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Conflicts:

    server/src/handler/list.cpp</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">adjusted test</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Modify: Make sure the referenced state is stored before we trigger the sync.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Otherwise it's possible that we get a LIST that expectes the referenced state,

before it's stored.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Changed ancestors listing to use regular collection format and include name.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">LIST: Fetch ancestor attributes</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Conflicts:

    server/src/handlerhelper.cpp

    server/src/handlerhelper.h

    server/tests/unittest/handlerhelpertest.cpp</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">LIST: list referenced collections to resources.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Otherwise referenced collections are never synchronized resulting in no content.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Fix LIST handler listing irrelevant collections in mimetype filter</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">The hasChildCollections check would include every collection with at least

one children, regardless whether it passes the mimetype filter or not. This

patch changes child collection check to only evalute to true if at least one

collection in the subtree matches the mimetype filter.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Conflicts:

    server/tests/unittest/listhandlertest.cpp</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">LIST: Non-recursive listing</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Instead of recursively query for children,

we query for a list of all collections we're interested in,

and then assemble a tree.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This potentially results in a small overhead with few collections,

but should scale much better when we have many collections while most

are getting filtered by the initial query (i.e. disabled collections).</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Additionally this patch ensures enabled collections that are nested in

a disabled collection are correctly found.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">REVIEW: 119418</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Conflicts:

    server/src/handler/list.cpp

    server/src/handler/list.h

    server/tests/unittest/listhandlertest.cpp</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Test-DbInitializer: improve for listhandler test.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Share DbInitializer for other tests and make use in listhandlertest.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">use dbinitializer</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">remove the collections manually since sqlite doesn't deal with constraints.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Conflicts:

    server/tests/unittest/collectionreferencetest.cpp

    server/tests/unittest/listhandlertest.cpp</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">FETCH: Fetch tags with item</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">REVIEW: 119468</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Don't call CREATE DATABASE in FakeAkonadiServer</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Unit-tests always use SQLite, which does not support CREATE DATABASE. This

for some reason fails silently in Qt 4, but produces (a valid) error in Qt 5.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">TagFetchHelper: Fix SQL query for fetching tag RIDs</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">The original query was wrong and was returning a single tag multiple

times, if it had multiple remote IDs.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">CollectonReferenceTest: Fixed the tests after fixing the reference manager</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This test couldn't work because the referenced state is reset after

each scenario (due to the session resetting the reference count).

I adapated the test now accordingly.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Fix a bug in DataStore::virtualCollections()</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">The code was not correctly resolving all virtual collections that

given items belong too.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Fix race condition in StorageDebugger</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Thanks to David Faure for discovering this.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">TagAppend: fix repeated appends with merge enabled</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This was triggered by the knutresource which creates it's tags everytime

it's syncing collections. For updating the record we cant use the convenience

class, since we need to identify it using both id and resourceId, and not only id.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">REVIEW: 119320</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Use the right identifier to remove the session from the CollectionReferenceManager.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Fix HRID-based FETCH not searching in correct collection</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Fix compiler warning in MERGE handler</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">compile with Clang</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Qt 4.8 doesn't support Clang (and so doesn't have initializer lists with Clang)</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">STORE: Allow modifying items tags via Tag RID or GID</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Tags RID is of course allowed only to resources</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Conflicts:

    CMakeLists.txt</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Optimize the cacheOnly fetch scope override hack</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">With Baloo we introduced a hack that ignores CACHEONLY parameter to FETCH command

when all requested items are local (provided by maildir/mbox resources) so that

Baloo was able to index local mails despite cache expiration policy.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This (together with the broken CollectionScheduler) lead to local mails never

expiring from Akonadi Cache.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This change permits the CACHEONLY override only to the Baloo Indexer agent.

Unlike some other agents/clients, the Baloo agent is smart and does only incremental

checks, so it will not cause the entire maildirs to be loaded into Akonadi cache.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">BUG: 333514

FIXED-IN: 1.13.0</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Fix CollectionScheduler's timer not resuming when uninhibited</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Since CacheCleaner is inhibited during various operations after start, the

timer was never correctly resumed when the inhibitor was removed, and so

payload parts were never expired.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">CCBUG: 333514</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">PartStreamer: don't UPDATE Part twice</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">When using SQLite, always start transactions in IMMEDIATE mode</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">IMMEDIATE mode acquires RESERVED lock immediately, which seems to resolve

the database deadlock errors we've been seeing with SQLite.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Disable COPY and MOVE into virtual collection</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Although technically possible, it makes no sense for virtual collections to own

any items and we already prevent X-AKAPPEND to virtual collections anyway.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">MERGE: Only emit change notification if something has really changed</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">The MERGE request does not have to contain all data (like GID, for instance),

so instead of overwriting existing GID with an empty GID, we ignore this

"change". The same applies for flags: I noticed that when syncing a folder,

KMail was eating about 50% CPU, because ETM generated dataChanged() for every

item in the synced folder. It was because the Server assumed that non-incremental

flags change must always generate a change, which is not true of course.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Fix a bug in setToVector() conversion</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">QVector(int) constructor allocates n items, so appending set values to it creates

a vector of n zeros followed by the appended values, which luckily in our case had

no side-effect, but was confusing. Instead we use std::copy to copy the QSet to the

QVector.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Fix and optimize DataStore::virtualCollections()</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">The method now returns QMap<Entity::Id, QList<PimItem> > instead of being a

multimap, and the PimItem now correctly contains mime type.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Implement fetching Virtual References via FETCH command</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This adds a new parameter to fetch scope called VIRTREF. When listed in fetch

scope, the FETCH response will include list of virtual collections that the item

is linked to. This allows resources to link/unlink items incrementally, without

having to make a full listing of all collections to find exact inconsistency

between the cache and backend storage.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Conflicts:

    CMakeLists.txt</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">LINK: Restore resource context if it was unset</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Fix retrieving statistics for virtual collections</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">ModifyHandlerTest: Removed duplicate test.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Fixed listhandlertest.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Or is the order not guaranteed?</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Support for temporary referenced collections.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Conflicts:

    CMakeLists.txt</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Option to avoid db population in fakedatastore.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This allows tests to avoid the population by the xml data so they can

populate the store themselves.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Add benchmark for ImapParser::parseParenthesizedList.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">The test data comes from real-world data, with the actual email

addresses and names replaced by dummy data.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Explicitly set size of hiddenAttribute</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">When appending a hidden attribute, we must specify the size, because

otherwise PSQL will try to insert a NULL value, which is not permitted

in PartTable.datasize column</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Register the /SearchManager object on main thread connection</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Otherwise the /SearchManager path would never appear on our service.

This won't affect the crash in SearchTaskManager, as that is using

it's own per-thread connection.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">FETCH: Drop the requirement that GID-based fetch must have collection context</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">PartStreamer: Make sure the external part storage directory exists</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Bumped protocol version and server version.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Conflicts:

    CMakeLists.txt</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Migrated subscribed to enabled, fixed tests.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Only emit subscribed notification after modified notification, cleanup, fixed tests.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Only schedule collections for sync that are enabled or have a corresponding preference.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Convert Tristate to SMALLINT on PostgreSQL</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">TINYINT (1-byte int) is not part of SQL standard and is only supported by MySQL

and SQLite. In PostgreSQL, the smallest numerical type is SMALLINT, which has

2 bytes.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Make FakeAkonadiServer a subclass of AkonadiServer</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This way we make sure that parts that call AkonadiServer::instance() will in fact

get a pointer to FakeAkonadiServer.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Also make sure that all unit-tests that use FakeAkonadiServer call quit() at the

end to clean up the test environment.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">The cachecleaner and intervalcheck are optional and not available during unittests.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Made CacheCleaner and IntervalCheck non-singletons.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">AkonadiServer serves as the singleton to access the main instances,

IntervalCheck and CacheCleaner are in fact optional and the returned pointer

needs to be checked accordingly.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">LIST/MODIFY/CREATE: support for enabled state and sync/display/index preference.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This patch adds support for the new enabled state plus the local

preference for sync/display/index. The enabled state gives a preference

that can be synchronized accross devices, while the local preference

allows to override that setting individually for each purpose.

This mechanism should eventually replace the local subscription mechanism.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Use AKONADI_EXCEPTION_MAKE_INSTANCE for proper Exception realization in FakeAkonadiServerException.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Implement direct streaming of part data</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This implements a new optimization for item streaming, which improves performance

and memory efficiency when appending or modifying items, but allowing clients to

write the payload directly to a file instead of sending it to the server.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This effectively eliminates the need of transmitting (possibly large, but always

at least DbConfig::sizeTreshold() bytes-large) part data to the server, which has

to process it and write it to a file. This feature of course only works for external

parts, i.e. parts larger than the configured treshold. Smaller parts or non-literals

are still parsed as usually.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">The main change that clients have to cope with is that the continuation response

can now contain parseable data that they must check for:</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">+ STREAM [FILE external_file_name]</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">If client receives this response, it parses the file name and writes the data

directly into the file instead of sending them to the server. In case of error,

the client replies "* NO Error message", otherwise it just sends next payload part

header, or end of list.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">The server only verifies that the file size matches the advertised size of the part

and continues parsing next data.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Clients have to advertise that they support direct payload streaming by listing

DIRECTSTREAMING in CAPABILITY command. Clients without this capability will never

receive the STREAM continuation response and must provide any payload the usualy way.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Conflicts:

    CMakeLists.txt</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Make FakeAkonadiServer a singleton initialized early in AKTEST_FAKESERVER_MAIN macro</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This makes sure that all environment variables will be set before anything

else has a change to call XdgDataDirs or anything else that caches the paths.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">FakeAkonadiServer: don't overwrite user's config in ~/.config/akonadi</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Fix handler unit-tests failing because of protocol version mismatch</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Add more complex test cases for X-AKAPPEND handler</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Improve macros in aktest.h</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Add setWaitTimeout() to ImapStreamParser to configure timeout for unit-test</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Unlike in real world, the client will always send response to server immediately

so waiting for 30 seconds slows down the tests unneccesarilly when we want to test

whether server can cope with read timeout correctly.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Terminate FakeClient when a QTest::qVerify or QTest::qCompare fails</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This requires special version of QCOMPARE and QVERIFY that calls quit()

when the check fails. Otherwise the test got stuck after a failure, because

the FakeClient did not play the scenario further, but did not 'disconnect'

from FakeServer.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Backport some ImapStreamParser bug fixes from KIMAP</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Discovered by akappend handler test</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Base X-AKAPPEND handler unit-test</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">AkTest: more comfortable test helpers</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Add a nasty hack to remove custom cmdline arguments before calling QTest::qExec</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">QTest's QApplication will abort on 'unkown argument', if we pass it some of

ours, so we process our arguments first, then remove them from argv and pass

the stripped argv to QTest::qExec.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Rework the Fake testing infrastructure to be more like server-client</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This is inspired by FakeServer from KIMAP, where server (a thread) is given a

scenario and it compares whether requests from client match the scenario and

sends back replies from the scenario, except here the roles are switched and

the scenario is played by client, which verifies whether server responds

correctly to it's commands.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Add FakeSearchManager and instantiate an inactive PreprocessorManager from FakeServer</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">FakeSearchManager is needed to make SearchManager::instance() return a valid

pointer (which is expected by all callers) but not really do anything when

called, as we don't want the search code to interfere with our tests.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Same applies for the PreprocessorManager, but that can be simply turned of and

really does nothing when there are no preprocessor registered.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Don't emit notify() when there's no notification in NotificationCollector's queue</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This prevents 'ghost' notifications appearing on some places and makes the

code easier to unit-test.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Extend LinkHandlerTest by tests for UNLINK command</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Implement a unittest for LINK handler and fix a bug it has found</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Add empty constructor to Scope so that it can be a metatype</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Add QDebug support to NotificationMessageV3 and few handy methods for unit-testing</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Introduce some Fake classes to allow unit-testing of internal parts</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Namely we have FakeDataStore, which records all calls and calls the actual

implementation, FakeConnection, which allows injecting specific handlers and

specifying commands to process as a QByteArray (so no need to use socket) and

finally FakeAkonadiServer, which when started sets up the entire environment

in /tmp, intializes and populates database and cleans up everything again when

destroyed.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Make Connection a QObject and create ConnectionThread that owns Connection</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This solves some problems with QObject ownership (Connection lives in a thread in which

it's executed) and allows unit-tests to be single-threaded.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Add a XML file that describes content the database will be populated with</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">The XML file is transformed into a C++ class called DbPopulator using XSL transformation.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Due to my experience in writing XSL stylesheets being non-existent, the XSL templates are

rather messy and the format and content of the input XML file are limited, but should be

sufficient to populate the database with some testing data that unit-tests can refer to.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Fix handling of tags in AK-APPEND and MERGE commands</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">The handlers are now able to process tags identified by GID or RID

and will create a new tag, if no such tag exists.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">RTag (identifying tag by RID) requires resource context.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Manually rollback transaction when SQLITE reports deadlock</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Unlike other databases, SQLITE won't rollback the transaction for us.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Improve detection whether SQLite is built with unlock_notify API</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Support for tag types + and fixed tag fetch by RID.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">The type is stored a separate table and always returned when fetching a tag.

Additionally we're now picking the right result of the query as rid.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">REVIEW: 117769</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Conflicts:

    CMakeLists.txt</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Remove space from an argument passed to postgres server</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Some users reported problem with starting PostgreSQL when there's a space

between name and value of an argument passed to postgres. Removing the

space fixes problem for them (withouth breaking PostgreSQL for those who

didn't have a problem with the space)</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Thanks SergTruf for the patch.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">BUG: 332988

FIXED-IN: 1.12.2</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Fix PostgreSQL start when postmaster.pid is not removed after non-clean shutdown</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">When PostgreSQL is not terminated nicely, a pidfile is left behind in db_data,

which will prevent pg_ctl from starting a new PostgreSQL server.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">We check for postmaster.pid file and verify that postgres server with PID

specified in the pidfile is not running anymore, then delete the pidfile to

allow pg_ctl to start a new server. If the postgres server is still running

(possibly after Akonadi server crash), we try to connect to it right away.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">BUG: 286826

FIXED-IN: 1.12.2</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Fix PostgreSQL startup when akonadiserverrc contains empty values</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">When there are empty configuration options in akonadiserverrc (like Host),

we would still pick them up and use them as valid values, which is wrong,

because then we work with empty socket path etc.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Instead we always replace non-existent and empty config values with the

default ones, so that Akonadi with PostgreSQL is always able to start.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Added an index for the remoteId column.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">The MERGE does a lot of lookups by rid and therefore is a lot faster with the index.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Incremental changes for MERGE and tag support for MERGE and APPEND.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Incremental changes make it possible to i.e. only update the flags.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Tag support allows clients to create a new item with tags specified via

\Tag['tagRemoteId'] flag (the flag can be repeated with different

remote IDs to assign multiple tags. The tags MUST exist.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">When SILENT is specified on MERGE, the resulting item won't be sent in the response

of MERGE, instead just an UID is sent.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Daniel Vratil is of course responsible for writing the patch ;-)</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Update Doxygen documentation</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Fix links and add information about some new features. This is still far from

complete or perfect.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Include <numeric> for std::accumulate.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This fixes the build with at least libc++ after f212406e.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Fix build when sqlite is not installed</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Check whether Sqlite is compiled with SQLITE_ENABLE_UNLOCK_NOTIFY</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">The QSQLITE3 driver now requires that SQLite is compiled with

SQLITE_ENABLE_UNLOCK_NOTIFY, which enables the unlock_notify API.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">It turns out that some distributions ship SQlite compiled without

this option and Akonadi then fails to compile.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">To prevent this, we detect whether SQlite is compiled with unlock_notify

API at configuration time by compiling a simple check in FindSqlite.cmake.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">REMOVE: Parse in-command context</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Parse collection and/or tag context specified within command.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Tag context should not be persistent accross commands</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Unlike with SELECT for collections, we have no way to change tag context

or unset it, so if one command sets the context, all subsequent commands

would have to override it.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Instead we make tag context non-persistent (it's reset before every new

handler is started).</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Once (if) we kill SELECT in KF5 and specify the context in each command

instead for collections too, we can make the entire context to be per-

command instead of per-connection.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Fix change detection when storing an updated attribute part</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">DataStore: sqllite does not support the QuerySize feature, so we can't rely on it.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">MERGE: Only emit itemChanged if the item is actually modified by the merge</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This adds a new optional argument to PartHelper::storeStreamedParts() that will

be set to true if the part data have changed, or a new part was created. The

code always compares data sizes first and will fall back to comparing the actual

data only when the 'changed' argument is set by caller and when part sizes are

the same.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Implement MERGE command handler</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">MERGE command allows merging new item into an existing one based on

GID and/or RID match. If no such item exists a new one is created.

If it exists, the new item is merged into the existing one, overwriting

conflicting flags, and parts.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Merging is restricted to parent collection and mime type, to minimize

potential RID/GID conflicts, since they are not guaranteed to be unique.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Syntax of MERGE command is equivalent to syntax of X-AKAPPEND command,

except for the command ("MERGE") is followed by parenthesized list of

attributes to merge by (currently supported values are "REMOTEID" and

"GID").</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Implementation-wise, Merge handler is a subclass of AkAppend handler,

which allows reusing all the existing code for insertion of a new item

(in case no candidate for merging is found) and only implements merging.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Conflicts:

    CMakeLists.txt</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Fix retrieving of GID from SQL query result in FetchHelper</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This has been broken since the day one, but nobody noticed. I guess

we were lucky enough to always query other arguments, so that

ItemQueryPimItemGidColumn actually matched indexed of the GID column

in query.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Another reason why we need proper unit-tests on the server...</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Remove the invalid GID part from PartTable before starting PartTable migration</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">More people than we expected have invalid 'GID' part in their PartTable,

which breaks migration to schema 25, because it expects all part types

to have a valid name.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">To work around this fact, we DELETE all parts with name 'GID' from PartTable

before starting the actual migration. This will not fix the migration for

people with other invalid parts, but I haven't heard of any such. To make

this completely bullet-proof, we would need to iterate through all entries,

which would be massively slower than current INSERT INTO ... SELECT FROM approach.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Distributions, this is a good choice for backporting into 1.12.1 ;-)</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">BUG: 331867

FIXED-IN: 1.12.2</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Use per-thread QDBusConnections</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This moves DBusConnectionPool from Nepomuk search code to Akonadi and makes

use of it in search infrastructure and couple other classes that interact

with DBus from non-main thread.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">QDBusConnection is not thread-safe in Qt 4, so we need to workaround it by

having a connection for each thread. Qt 5 should be OK, so we can remove this

in Frameworks.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This should fix random crashes I've been seeing when SearchTaskManager::addTask()

was called from multiple threads simultaneously.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">With the Baloo port completed, don't require Soprano by default.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Approved by Dan.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Merge branch 'master' into kolab/integration/1.12.0</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Conflicts:

    CMakeLists.txt</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Appended kolab version number.</p></pre>

  </td>

 </tr>

</table>

<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>

<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">

 <tr>

  <td>

   <pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">All tests pass, I'll test it with kdepimlibs once that merge is done as well.</p></pre>

  </td>

 </tr>

</table>

<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>

<ul style="margin-left: 3em; padding-left: 0;">

 <li>autotests/private/CMakeLists.txt <span style="color: grey">(374238693422a60e8f558e1c34ce14ed37067778)</span></li>

 <li>autotests/private/notificationmessagev2test.h <span style="color: grey">(09a950a3238880e366ef8b989eddc2792252c8bf)</span></li>

 <li>autotests/private/notificationmessagev2test.cpp <span style="color: grey">(1b4e4b4043ce1c9d2fba9188d4294fb2986a1dd7)</span></li>

 <li>autotests/server/CMakeLists.txt <span style="color: grey">(6849af98fba6bfe2242bb0f6d371ea8dd9e2be5f)</span></li>

 <li>autotests/server/dbinitializer.h <span style="color: grey">(ad910aeaf454862b7cf4f06e2695cc039fb2991d)</span></li>

 <li>autotests/server/dbinitializer.cpp <span style="color: grey">(31d94ea293cb2260b7c603cae710cc8852e8bdd2)</span></li>

 <li>autotests/server/fakeclient.cpp <span style="color: grey">(5b9d65862367ea3c6e78f9c0d6eb8e35d1b9f253)</span></li>

 <li>autotests/server/fetchhandlertest.cpp <span style="color: grey">(PRE-CREATION)</span></li>

 <li>autotests/server/linkhandlertest.cpp <span style="color: grey">(ede047aedf34238e5792b909f8ef847aaa64ab82)</span></li>

 <li>autotests/server/listhandlertest.cpp <span style="color: grey">(0edae82f2b5a1a7fba48d1d6fbaa77628a99986e)</span></li>

 <li>autotests/server/notificationmanagertest.cpp <span style="color: grey">(9b46648127da3c42bf835602b73019cb71865abc)</span></li>

 <li>autotests/server/relationhandlertest.cpp <span style="color: grey">(PRE-CREATION)</span></li>

 <li>autotests/server/taghandlertest.cpp <span style="color: grey">(PRE-CREATION)</span></li>

 <li>src/interfaces/org.freedesktop.Akonadi.NotificationSource.xml <span style="color: grey">(ef660147c2146bad6d0d069023fe8f2833f30a02)</span></li>

 <li>src/private/notificationmessagev2.cpp <span style="color: grey">(b8ddc2442af2d2c773d67fca32cb21c6cd65ab71)</span></li>

 <li>src/private/notificationmessagev2_p.h <span style="color: grey">(335a49ea0cfd9b6619bc89582c7a44da62412d14)</span></li>

 <li>src/private/notificationmessagev2_p_p.h <span style="color: grey">(2d200abcc31cd9ed8a2e8f711a6dbed24a500c3d)</span></li>

 <li>src/private/protocol_p.h <span style="color: grey">(2ec2a2e27edbd350cb9fad2320b916ef125da952)</span></li>

 <li>src/private/xdgbasedirs.cpp <span style="color: grey">(f88d7427d77d8404c3a92590994a49eda2cad05a)</span></li>

 <li>src/server/CMakeLists.txt <span style="color: grey">(795b0c6e1e8c850e4eda233c1e59b691d8a99f33)</span></li>

 <li>src/server/collectionscheduler.cpp <span style="color: grey">(59658c900714bcac9781d1bfa17911504e38d8f1)</span></li>

 <li>src/server/connection.h <span style="color: grey">(6a4b9637943710e452a329fcb3f742a8b785f7ac)</span></li>

 <li>src/server/connection.cpp <span style="color: grey">(4b0d45a55c35453ef55a9d1b8fb38d9763060201)</span></li>

 <li>src/server/handler.cpp <span style="color: grey">(21363429f4d7362ba095e506c65ddb968d6178a2)</span></li>

 <li>src/server/handler/fetchhelper.h <span style="color: grey">(c64f1cc2403399b553c6edc6797442d3ede561bf)</span></li>

 <li>src/server/handler/fetchhelper.cpp <span style="color: grey">(5179e1466ccd67947bd5508087f3ccbed1f1d5f9)</span></li>

 <li>src/server/handler/fetchscope.h <span style="color: grey">(1e2a9dc69b7f30930864fad4ca98632c1a28b5fc)</span></li>

 <li>src/server/handler/fetchscope.cpp <span style="color: grey">(f7e988f42ae0c8a597e4dfabea9077a19457a310)</span></li>

 <li>src/server/handler/list.h <span style="color: grey">(addb8096a2238e9829218e28f76b5bdc508b76bd)</span></li>

 <li>src/server/handler/list.cpp <span style="color: grey">(10774875ed2fddc15b2a5f882ae3661851e6e3a5)</span></li>

 <li>src/server/handler/merge.h <span style="color: grey">(b2bf984099b7879035081466ecc45beb901d98ff)</span></li>

 <li>src/server/handler/merge.cpp <span style="color: grey">(f86219face1874e84c62566b8f25d9f833fa2305)</span></li>

 <li>src/server/handler/modify.cpp <span style="color: grey">(8d2bd696d305304268054b313060afde0eee1b7e)</span></li>

 <li>src/server/handler/relationfetch.h <span style="color: grey">(PRE-CREATION)</span></li>

 <li>src/server/handler/relationfetch.cpp <span style="color: grey">(PRE-CREATION)</span></li>

 <li>src/server/handler/relationremove.h <span style="color: grey">(PRE-CREATION)</span></li>

 <li>src/server/handler/relationremove.cpp <span style="color: grey">(PRE-CREATION)</span></li>

 <li>src/server/handler/relationstore.h <span style="color: grey">(PRE-CREATION)</span></li>

 <li>src/server/handler/relationstore.cpp <span style="color: grey">(PRE-CREATION)</span></li>

 <li>src/server/handler/tagremove.cpp <span style="color: grey">(99b61dc5daf4398a2d7d9e2f2e08c975f449b006)</span></li>

 <li>src/server/handler/tagstore.cpp <span style="color: grey">(56c695a59d3dc4d19899780409e45a89088349dd)</span></li>

 <li>src/server/handlerhelper.h <span style="color: grey">(fc7d0a0b1344abf333d56a618e69a453bee670ff)</span></li>

 <li>src/server/handlerhelper.cpp <span style="color: grey">(67994a6267c04ba29a7e35687a68d8b6175f3b67)</span></li>

 <li>src/server/imapstreamparser.cpp <span style="color: grey">(34884beaf17aefd8abfaec3b03a3437bbf5edc1e)</span></li>

 <li>src/server/notificationmanager.cpp <span style="color: grey">(234ea5d9ff189dd89c2ffffceaf262231b3f0cdd)</span></li>

 <li>src/server/notificationsource.h <span style="color: grey">(bad148cd95c41502f752516f0f3d1aa34484e498)</span></li>

 <li>src/server/notificationsource.cpp <span style="color: grey">(3aaddf2f72b56cb750224daeaf8ddc47630a77fd)</span></li>

 <li>src/server/storage/akonadidb.xml <span style="color: grey">(4ab487bba74a492673b440cada69db691d38a4f3)</span></li>

 <li>src/server/storage/datastore.h <span style="color: grey">(7122667ca441dd6422de628f70f331a07fcefaa1)</span></li>

 <li>src/server/storage/datastore.cpp <span style="color: grey">(c89a4e69629f5e96052dd7853f69dc869fddb263)</span></li>

 <li>src/server/storage/dbconfigpostgresql.cpp <span style="color: grey">(15cf84990241c27df0395ad9eeaf64f9a4891ba9)</span></li>

 <li>src/server/storage/itemqueryhelper.cpp <span style="color: grey">(529077739acf7cedf512c0b95baf9421af6fbe98)</span></li>

 <li>src/server/storage/notificationcollector.h <span style="color: grey">(14f95365702f84297943e8424d6d294b7bb735e5)</span></li>

 <li>src/server/storage/notificationcollector.cpp <span style="color: grey">(142ba204eb872352d12a7271dbf0d024bc583d94)</span></li>

 <li>src/server/storage/partstreamer.cpp <span style="color: grey">(1d0919cc25d90691593f6429b71a848d337d11ee)</span></li>

 <li>src/server/storage/querybuilder.h <span style="color: grey">(14af3a787974f44bb0c8078223ebb3c5528f59ee)</span></li>

 <li>src/server/storage/querybuilder.cpp <span style="color: grey">(4d65c35507e3a518a23a017039f435324279c12d)</span></li>

 <li>src/server/storage/storagedebugger.h <span style="color: grey">(c02583f8fd31aa7e5d8a5c23663a75470a2d6d5c)</span></li>

 <li>src/server/storage/storagedebugger.cpp <span style="color: grey">(6264f410fbad55939dfadde185b9bc204e977198)</span></li>

</ul>

<p><a href="https://git.reviewboard.kde.org/r/123183/diff/" style="margin-left: 3em;">View Diff</a></p>

  </td>

 </tr>

</table>

  </div>

 </body>

</html>