[TimeEstimator] Correct batch size, remove config reference
ClosedPublic

Authored by bruns on Mar 24 2020, 8:41 PM.

Details

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.

Diff Detail

Repository
R293 Baloo
Branch
submit
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 24189
Build 24207: arc lint + arc unit
bruns created this revision.Mar 24 2020, 8:41 PM
Restricted Application added projects: Frameworks, Baloo. · View Herald TranscriptMar 24 2020, 8:41 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
bruns requested review of this revision.Mar 24 2020, 8:41 PM
bruns updated this revision to Diff 78418.Mar 25 2020, 12:14 AM

whitespace

ngraham accepted this revision.Mar 25 2020, 4:13 PM
This revision is now accepted and ready to land.Mar 25 2020, 4:13 PM
This revision was automatically updated to reflect the committed changes.