gharris1727 commented on PR #13036: URL: https://github.com/apache/kafka/pull/13036#issuecomment-1363112472
> after your change, it failed other tests: Thanks, fixed. I was running only the first test locally and didn't check to see what the timeouts in the other test were. > After all, the producerID expiration doesn't cause any test failure here, we shouldn't change it. WDYT? Yes, the CI failures I was seeing were all about the race condition between the test and transactionId expiration. However, while I was running these tests under a local CPU slowdown to 25-30%, I was seeing race conditions happening between the test and the producerId expiration. The test would poll for the producer state, only for it to have already been expired by the broker. I'm lengthening both here so that I don't have to follow up when this test flakes out in the future :) -- 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
