Skip to content

pipeline: avoid missing final indexing progress end#993

Open
SuperDuckGOGOGO wants to merge 1 commit intoMaskRay:masterfrom
SuperDuckGOGOGO:fix-progress-end-lost-wakeup
Open

pipeline: avoid missing final indexing progress end#993
SuperDuckGOGOGO wants to merge 1 commit intoMaskRay:masterfrom
SuperDuckGOGOGO:fix-progress-end-lost-wakeup

Conversation

@SuperDuckGOGOGO
Copy link
Copy Markdown

Summary

  • wake the main loop after stats.completed advances so the final indexing completion is not silent
  • extend MultiQueueWaiter with predicate-aware wait helpers and use them for index progress tracking
  • prevent the last completed == enqueued transition from being missed, which can otherwise suppress the final $/progress end

Test plan

  • Build ccls locally with Clang CMake packages available
  • Reproduce indexing on a large workspace and confirm the final $/progress end arrives
  • Inspect the pipeline.cc and threaded_queue.hh control flow for the lost-wakeup race

Wake the main loop when indexing completion advances and include progress changes in the wait predicate so the final completed==enqueued transition cannot be lost.

Made-with: Cursor
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant