[Kde-pim] Review Request 110268: Nepomuk Feeder: Simplify change recording logic

[Christian Mollekopf]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/110268/#review31897
-----------------------------------------------------------


Not sure if we should ever enable changeRecording. The point of those variables was exactly that we can disable the change recoding entirely if the feeder is disabled (set offline). Otherwise the changerecording queue will grow endlessly which will end up as a cpu and memory hog. Change recording could be ok if the feeder is disabled temporariliy (powersaving or whatnot), but it's imo a rather dangerous feature. I'd therefore rather avoid recording changes and rely on the FindUnindexedItemsJob (which could be run whenever the feeder is set online again).

- Christian Mollekopf


On May 2, 2013, 8:59 a.m., Vishesh Handa wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/110268/
> -----------------------------------------------------------
> 
> (Updated May 2, 2013, 8:59 a.m.)
> 
> 
> Review request for KDEPIM and Christian Mollekopf.
> 
> 
> Description
> -------
> 
>    Only record changes when we are not online. We get notified normally
>    otherwise because we are an Observer.
>     
>    Also removed the superflous variables which could be simplified into
>    isOnline()
> 
> 
> Diffs
> -----
> 
>   agents/nepomukfeeder/nepomukfeederagent.cpp 7380a32 
>   agents/nepomukfeeder/nepomukfeederagent.h eafe5cd 
> 
> Diff: http://git.reviewboard.kde.org/r/110268/diff/
> 
> 
> Testing
> -------
> 
> works. Changes are being recorded when we are not online.
> 
> 
> Thanks,
> 
> Vishesh Handa
> 
>

_______________________________________________
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/