[GitHub] [kafka] showuon edited a comment on pull request #9629: KAFKA-10754: fix flaky tests by waiting kafka streams be in running state before assert

2020-11-23 Thread GitBox
showuon edited a comment on pull request #9629: URL: https://github.com/apache/kafka/pull/9629#issuecomment-732646473 @ableegoldman @wcarlson5 , thanks for the comments. Now I know why it sets the default treads to 2. So, to make the test more reliable, I'll do: 1. wait for the state

[GitHub] [kafka] showuon edited a comment on pull request #9629: KAFKA-10754: fix flaky tests by waiting kafka streams be in running state before assert

2020-11-21 Thread GitBox
showuon edited a comment on pull request #9629: URL: https://github.com/apache/kafka/pull/9629#issuecomment-731557695 All tests passed, Yeah! This is an automated message from the Apache Git Service. To respond to the

[GitHub] [kafka] showuon edited a comment on pull request #9629: KAFKA-10754: fix flaky tests by waiting kafka streams be in running state before assert

2020-11-20 Thread GitBox
showuon edited a comment on pull request #9629: URL: https://github.com/apache/kafka/pull/9629#issuecomment-731520316 @ableegoldman , thanks for pointing it out. After investigation, I found the test `StreamsUncaughtExceptionHandlerIntegrationTest.shouldShutdownThreadUsingOldHandler`

[GitHub] [kafka] showuon edited a comment on pull request #9629: KAFKA-10754: fix flaky tests by waiting kafka streams be in running state before assert

2020-11-20 Thread GitBox
showuon edited a comment on pull request #9629: URL: https://github.com/apache/kafka/pull/9629#issuecomment-731520316 @ableegoldman , thanks for pointing it out. After investigation, I found the test `StreamsUncaughtExceptionHandlerIntegrationTest.shouldShutdownThreadUsingOldHandler`

[GitHub] [kafka] showuon edited a comment on pull request #9629: KAFKA-10754: fix flaky tests by waiting kafka streams be in running state before assert

2020-11-20 Thread GitBox
showuon edited a comment on pull request #9629: URL: https://github.com/apache/kafka/pull/9629#issuecomment-731520316 @ableegoldman , thanks for pointing it out. After investigation, I found the test `StreamsUncaughtExceptionHandlerIntegrationTest.shouldShutdownThreadUsingOldHandler`