[GitHub] [kafka] dengziming commented on pull request #12112: MINOR: Fix flaky testDescribeUnderReplicatedPartitions

2022-05-04 Thread GitBox
dengziming commented on PR #12112: URL: https://github.com/apache/kafka/pull/12112#issuecomment-1118110191 @showuon I seem to see it failed but I can't find it anymore, I will close this after I can't see it fail again in a few days. @divijvaidya I would fix your comment if this test real

[GitHub] [kafka] dengziming commented on pull request #12112: MINOR: Fix flaky testDescribeUnderReplicatedPartitions

2022-05-05 Thread GitBox
dengziming commented on PR #12112: URL: https://github.com/apache/kafka/pull/12112#issuecomment-1119266873 @showuon I have updated it, though @divijvaidya provided a good suggestion, we still have a more generic solution for this kind of flakiness. [see details here](https://github.com/apac

[GitHub] [kafka] dengziming commented on pull request #12112: MINOR: Fix flaky testDescribeUnderReplicatedPartitions

2022-05-06 Thread GitBox
dengziming commented on PR #12112: URL: https://github.com/apache/kafka/pull/12112#issuecomment-1120105686 Let's hold on to this before [#12108](https://github.com/apache/kafka/pull/12108) was approved. -- This is an automated message from the Apache Git Service. To respond to the message

[GitHub] [kafka] dengziming commented on pull request #12112: MINOR: Fix flaky testDescribeUnderReplicatedPartitions

2022-05-10 Thread GitBox
dengziming commented on PR #12112: URL: https://github.com/apache/kafka/pull/12112#issuecomment-1123114260 Hello @showuon @divijvaidya, #12108 has been merged, we added `TestUtils.ensureConsistentKRaftMetadata` to fix all this type of flakiness, PTAL. -- This is an automated message from