[GitHub] [kafka] showuon commented on pull request #12500: MINOR: Remove spurious sleep in ConsumerCoordinatorTest

2022-08-14 Thread GitBox
showuon commented on PR #12500: URL: https://github.com/apache/kafka/pull/12500#issuecomment-1214482042 Weird! Looks like the test still stuck. I'll take a look after my vacation. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [kafka] showuon commented on pull request #12500: MINOR: Remove spurious sleep in ConsumerCoordinatorTest

2022-08-14 Thread GitBox
showuon commented on PR #12500: URL: https://github.com/apache/kafka/pull/12500#issuecomment-1214369765 > Ah, sorry for the delay. I think time.sleep() here is needed here to make the pollTimer timeout. If we removed this, will hang in the coordinator.onJoinPrepare @aiquestion ,

[GitHub] [kafka] showuon commented on pull request #12500: MINOR: Remove spurious sleep in ConsumerCoordinatorTest

2022-08-11 Thread GitBox
showuon commented on PR #12500: URL: https://github.com/apache/kafka/pull/12500#issuecomment-1211653244 Triggering another jenkins build to make sure it passes the tests. https://ci-builds.apache.org/job/Kafka/job/kafka-pr/view/change-requests/job/PR-12500/2/ -- This is an automated