D28258: [TimeEstimator] Correct batch size, remove config reference
Stefan BrĂ¼ns
noreply at phabricator.kde.org
Tue Mar 24 20:41:38 GMT 2020
bruns created this revision.
bruns added reviewers: Baloo, ngraham.
Herald added projects: Frameworks, Baloo.
Herald added a subscriber: kde-frameworks-devel.
bruns requested review of this revision.
REVISION SUMMARY
Using a fixed batch size is wrong, as the contentIndexer may process
smaller batches, e.g after content indexing failed, or more typically
when the indexer is idle and only has to update single/some files.
Instead of storing the total batch time in the history store the per
file average time.
As there is no longer any config dependency remove it.
REPOSITORY
R293 Baloo
BRANCH
submit
REVISION DETAIL
https://phabricator.kde.org/D28258
AFFECTED FILES
src/file/filecontentindexer.cpp
src/file/filecontentindexer.h
src/file/fileindexscheduler.cpp
src/file/timeestimator.cpp
src/file/timeestimator.h
To: bruns, #baloo, ngraham
Cc: kde-frameworks-devel, hurikhan77, lots0logs, LeGast00n, cblack, fbampaloukas, GB_2, domson, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, abrahams
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200324/e03b2c2c/attachment.html>
More information about the Kde-frameworks-devel
mailing list