chore(cleanup): remove SyncStrategy and simplify sync logic - #8405
Conversation
Eliminated the `SyncStrategy` enum and updated relevant code paths to default to serial execution, since it was hardcoded to serial for the BazelBuildSystem anyway.
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request removes the SyncStrategy configuration and parallel sync capabilities from the build system, simplifying target sharding and defaulting sync builds to run serially. The reviewer suggests further cleanups to eliminate redundant ternary operators and unused fields resulting from these hardcoded defaults, specifically by directly using singleThreadedExecutor in SyncPhaseCoordinator and defaultInvoker in BuildPhaseSyncTask.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
github-actions
Bot
added
product: CLion
github-project-automation
Bot
moved this from Untriaged
to Done
in Bazel IntelliJ Plugin
github-actions
Bot
removed
the
awaiting-review
2 participants
Eliminated the
SyncStrategyenum and updated relevant code paths to default to serial execution, since it was hardcoded to serial for the BazelBuildSystem anyway.