poboiko added a comment.

  In D15959#339523 <https://phabricator.kde.org/D15959#339523>, @bruns wrote:
  
  > This works for me:
  >
  >    void FileIndexScheduler::scheduleIndexing()
  >    {
  >   -    if (m_threadPool.activeThreadCount() || m_indexerState == Suspended) 
{
  >   +    if (m_indexerState == Suspended) {
  >            return;
  >        }
  >
  
  
  That might require some changes in the scheduling logic. Right now it will 
behave weirdly.
  If we do this change, there might be situations where bunch of i.e. 
`ModifiedFileIndexer` gets pushed to queue in a row. For example, we were 
inside `FirstRunIndexer`, and 10 files were modified.
  Even worse, if will have a single new file and then 10 modified files - it 
will push 10 `NewFileIndexer`s.

REPOSITORY
  R293 Baloo

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

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

Reply via email to