himanshug commented on pull request #10537: URL: https://github.com/apache/druid/pull/10537#issuecomment-721315754
@suneet-s for the specific change here, it does get covered by integration tests but scenarios like one of the leader overlord/coordinator node crash while some operation was happening, don't get tested in our build environments as all of them work with one overlord and coordinator nodes . Even in a real more complex env we would need to simulate node crashes to handle such scenarios to simulate different possible node failure models. does that answer your question ? ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org For additional commands, e-mail: commits-h...@druid.apache.org