xiangfu0 commented on PR #19311: URL: https://github.com/apache/pinot/pull/19311#issuecomment-5461714495
Addressed the process follow-ups on head `b612fab43e`: - Expanded the PR description with the end-to-end design/atomicity rationale, rollback boundary, exact supported/unsupported modes, resource-limit precedence, and the required all-servers-first rolling-upgrade sequence. The `design-review` label remains so maintainer sign-off is still explicit before landing. - Linked the sender-side sort TODO to #19395. - Rewrote the feature commits to remove both AI `Co-Authored-By` trailers. - Replied to and resolved all 13 inline threads with the pushed fix and regression evidence. Local validation passed: the 342-test focused suite, a 32-test matcher/operator rerun after the final retained-state changes, 13/13 two-server integration cases, the full 63-module `test-compile` reactor, Spotless, checkstyle, license checks, and diff hygiene. Fresh CI for this rewritten head is running. -- 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]
