FrankChen021 opened a new pull request, #20293: URL: https://github.com/apache/druid/pull/20293
### Description Publish the final controller report before notifying the completion listener, and serve it while the controller is still registered. Keep removal of the active entry and retention of its report under the same lock used by completed-report lookups. This closes two report-publication windows: stale live counters after result completion, and a missing report during registry transfer. [PR #20291 CI](https://github.com/apache/druid/actions/runs/34184621200/job/101930347957) observed missing stage-zero counters immediately after a successful Dart query. The integration test is unchanged. ### Validation - 22 controller lifecycle and registry tests passed, including two concurrent-reader tests. - Disabling final-report publication makes the completion-listener regression fail; restoring it passes. - Checkstyle passed. ### Release note Make final Dart query reports available when query results finish, without a temporary missing-report window during retention. ### Review - [x] Self-reviewed, including concurrency behavior. - [x] Added regression tests. -- 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]
