Re: [PR] MINOR: enable kraft test for ReassignPartitionsIntegrationTest [kafka]

2024-04-25 Thread via GitHub
FrankYang0529 commented on PR #15675: URL: https://github.com/apache/kafka/pull/15675#issuecomment-2078515889 > btw, it would be great if you can rewrite it by new test infra. Yeah, but this class need to assign different config to different broker. I will wait for

Re: [PR] MINOR: enable kraft test for ReassignPartitionsIntegrationTest [kafka]

2024-04-25 Thread via GitHub
chia7712 commented on PR #15675: URL: https://github.com/apache/kafka/pull/15675#issuecomment-2078363083 btw, it would be great if you can rewrite it by new test infra. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

Re: [PR] MINOR: enable kraft test for ReassignPartitionsIntegrationTest [kafka]

2024-04-07 Thread via GitHub
FrankYang0529 commented on PR #15675: URL: https://github.com/apache/kafka/pull/15675#issuecomment-2041482907 > @FrankYang0529 Could we use `ClusterTestExtensions` to enable `kraft`? It seems to me that using the new test framework is the better way. Otherwise, we may rewrite it again in

Re: [PR] MINOR: enable kraft test for ReassignPartitionsIntegrationTest [kafka]

2024-04-07 Thread via GitHub
chia7712 commented on PR #15675: URL: https://github.com/apache/kafka/pull/15675#issuecomment-2041482458 @FrankYang0529 Could we use `ClusterTestExtensions` to enable `kraft`? It seems to me that using the new test framework is the better way. Otherwise, we may rewrite it again in the

[PR] MINOR: enable kraft test for ReassignPartitionsIntegrationTest [kafka]

2024-04-07 Thread via GitHub
FrankYang0529 opened a new pull request, #15675: URL: https://github.com/apache/kafka/pull/15675 Enable Kraft test for `ReassignPartitionsIntegrationTest`. All test cases can pass on my laptop. ### Committer Checklist (excluded from commit message) - [ ] Verify design and