924060929 opened a new pull request, #69830: URL: https://github.com/apache/doris/pull/69830
### What problem does this PR solve? Problem Summary: FE still carries two execution coordinators: the legacy `Coordinator` assigns scan ranges and fragment instances itself, while `NereidsCoordinator` executes Nereids distributed plans but inherits the legacy coordinator's state and initialization. `CloudCoordinator` adds another legacy-only factory branch. This duplicates scheduling and lifecycle ownership and retains a fallback that newer execution features do not support. Replace the legacy implementation with the standalone Nereids implementation, renamed to `Coordinator`, and remove `NereidsCoordinator` and `CloudCoordinator`. SQL, INSERT and background load creation now use the same factory. Cloud compute-group selection, privilege validation and backend membership remain in `BackendDistributedPlanWorkerManager` during distributed planning. Preserve the remaining shared capabilities explicitly: HTTP Stream Load generates parameters with `ThriftPlansBuilder`; Arrow Flight retains scans that must outlive dispatch; streaming INSERT retains scan backend IDs; cancellation uses the runtime task state, without treating remote-catalog workers as local backends. Group-commit backend selection remains in distributed planning. Retire `enable_nereids_distribute_planner` using the existing removed-variable handling and remove its legacy-only optimizer branches. Migrate relevant unit coverage and remove regression assertions that specifically required the deleted legacy scheduling path; keep their result checks. ### Release note Use Nereids distributed planning for all queries and loads. `enable_nereids_distribute_planner` no longer selects a planner; existing SET statements for this removed variable remain accepted. ### Check List (For Author) - Test: `./build.sh --fe --exclude-obs-dependencies --exclude-cos-dependencies`; FE Checkstyle (0 violations); 205 focused FE unit cases across 19 classes, including ordinary and group-commit HTTP stream planning. Corrected test fixtures were rerun separately. OBS/COS plugin packaging and full BE-backed regression were not run locally. - Behavior changed: Yes. Remove the legacy scheduling fallback and always use Nereids distributed planning. - Does this need documentation: No; the retired variable is hidden by the existing removed-variable mechanism. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
