xiangfu0 commented on PR #17357: URL: https://github.com/apache/pinot/pull/17357#issuecomment-4154194853
Pushed `e053d09b52`. I reproduced the previous failing path locally with `OfflineClusterIntegrationTest#testShowTablesMatchesPinotAdminClient`. The root cause was broker-side header forwarding: metadata requests were replaying inbound transport headers such as `Host`/`Connection` to the controller admin endpoints, which could make the controller return HTTP 400. The broker now filters non-forwardable headers before broker-to-controller metadata calls, and the full local validation now passes, including the exact offline-cluster repro plus the required `spotless`, `checkstyle`, and `license` checks on the affected modules. GitHub has not attached a fresh check suite to the new PR head yet, so the old December failures are still the only runs visible right now. -- 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]
