rondagostino commented on pull request #11238: URL: https://github.com/apache/kafka/pull/11238#issuecomment-904824877
> we can have ceil method to use, which will be easier Good suggestion! Done. Here's some system tests runs that show the new code is syntactically correct (and I observed the correct kill behavior). ``` test_id: kafkatest.sanity_checks.test_bounce.TestBounce.test_simple_run.metadata_quorum=COLOCATED_KRAFT.quorum_size=1 status: PASS run time: 1 minute 49.278 seconds -------------------------------------------------------------------------------- test_id: kafkatest.sanity_checks.test_bounce.TestBounce.test_simple_run.metadata_quorum=COLOCATED_KRAFT.quorum_size=3 status: PASS run time: 2 minutes 6.602 seconds ``` Thank for the review! -- 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: jira-unsubscr...@kafka.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
