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

Reply via email to