goutamadwant commented on PR #19178: URL: https://github.com/apache/pinot/pull/19178#issuecomment-5708707928
@Jackie-Jiang I rebased this onto the current master and addressed every review thread. The update now uses a dedicated broker routing endpoint, withholds acknowledgement until the full instance-config routing update completes, and replaces request-path broker I/O with a single bounded background checker. It also adds strict HTTPS/auth handling, redirect protection, and non-blocking shutdown cleanup. The focused broker, server, and HTTP-client regression suites pass, and all threads are replied to and resolved. I do not have permission to apply labels; because this adds server configuration options, the PR template calls for the `release-notes` label. Could you add that label and take another look when convenient? -- 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]
