abhioncbr commented on PR #10814: URL: https://github.com/apache/pinot/pull/10814#issuecomment-1589611356
> this looks good to me. let's wait for the test to complete I don't think integration test failure is related to these PR changes ``` Error: Failures: Error: MultiNodesOfflineClusterIntegrationTest.testServerHardFailure:161 Failed to meet condition in 1000ms after 0 errors, error message: Failed to include the restarted server into the routing. Other tests may be affected [INFO] Error: Tests run: 115, Failures: 1, Errors: 0, Skipped: 0 ``` I can possibly merge the branch with master and give it a try. -- 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: commits-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org