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