poboiko added inline comments.

INLINE COMMENTS

> bruns wrote in filecontentindexer.cpp:91
> Yes, its a binary search.
> Because we only have IDs here, and the progress reporting works on strings.

There is `Transaction::documentId(const QByteArray& path)`, which can resolve 
it using `DocumentUrlDB`.
I believe it still would be cheaper than reindexing several files multiple 
times.

REPOSITORY
  R293 Baloo

REVISION DETAIL
  https://phabricator.kde.org/D16266

To: bruns, #baloo, #frameworks, poboiko, ngraham
Cc: broulik, apol, kde-frameworks-devel, ashaposhnikov, michaelh, astippich, 
spoorun, ngraham, bruns, abrahams

Reply via email to