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

[Vishesh Handa]

> On May 2, 2013, 4:35 p.m., Christian Mollekopf wrote:
> > 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).

But the FindUnindexedItemsJob is very expensive and causes HUGE amounts of CPU load. I want to avoid calling it unless absolutely necessary.

Right now if we go offline the changerecording will go online and record stuff. Wait, is switching off the agent the same as going offline? Cause then I guess we should just trigger the FindUnindexedItemsJob.


- Vishesh


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


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/