[Kde-pim] Review Request 111507: Add CHANGEDSINCE parameter to FETCH command
Christian Mollekopf
chrigi_1 at fastmail.fm
Mon Jul 15 09:56:46 BST 2013
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111507/#review35958
-----------------------------------------------------------
Regarding the feeder usage, how does this handle removals? It seems with this approach only new items would be detected and removals would be ignored?
The patch looks ok, but I agree with Kevin that this should be part of the fetchscope.
server/src/handlerhelper.cpp
<http://git.reviewboard.kde.org/r/111507/#comment26618>
space--
- Christian Mollekopf
On July 14, 2013, 9:55 p.m., Dan Vrátil wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111507/
> -----------------------------------------------------------
>
> (Updated July 14, 2013, 9:55 p.m.)
>
>
> Review request for Akonadi, Vishesh Handa and Volker Krause.
>
>
> Description
> -------
>
> This allows clients to fetch from the Akonadi server only those items that have been changed since given timestamp. The syntax is CHANGEDSINCE %unixTimestamp%.
>
> Usecase for this feature is mainly in the Nepomuk Feeder agent, which currently has to fetch all items from Akonadi and all items from Nepomuk on start up and compare them, to make sure the Nepomuk database is in sync with Akonadi. It causes high IO and high CPU and memory consumption during start, slowing down the start up. This feature will allow the Feeder to request only items that have changed since last time, which in most cases should mean no items at all.
>
>
> Diffs
> -----
>
> libs/protocol_p.h e94bf94
> server/src/akonadiconnection.cpp b8bcca3
> server/src/handler/fetchhelper.h 7f0f2b7
> server/src/handler/fetchhelper.cpp 78ea6ed
> server/src/handlerhelper.h 294b128
> server/src/handlerhelper.cpp e6d2f51
>
> Diff: http://git.reviewboard.kde.org/r/111507/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Dan Vrátil
>
>
_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/
More information about the kde-pim
mailing list