aliehsaeedii commented on PR #20320: URL: https://github.com/apache/kafka/pull/20320#issuecomment-3244534530
@mjsax >What was the original purpose of the test? -- It's unclear to me, why creating a second KS instance help to stabilize the test. Can you elaborate? If I read the test correctly, the idea is to get a timeout exception for a describe call -- what is the conditions when we would get a timeout exception? Group does not exist? Or something else? I assumed in real-world scenarios, 1ms is too little for describing a group, but the flakiness proved that 1ms could be enough sometimes. I thought the group is existing, and we had a couple of other `describe group` calls inside the same test, that's why it does not throw in some runs. Thus, with a newly created group, with no former `describe group` call on it, it must always throw an exception if the timeout is 1ms. Although as I explained in the description, I am not sure if it really fixes the issue. If `1ms` is enough, it is simply enough. We can either remove this test or reduce it to a lower timeout? -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
