-
Notifications
You must be signed in to change notification settings - Fork 13.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
MINOR: reduce the test suits of consumer group tools #16155
MINOR: reduce the test suits of consumer group tools #16155
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@m1a2st thanks for this patch. I agree that we have too many test suits for those tools. BTW, could you share the elapsed time with/without this patch?
@@ -62,7 +63,7 @@ static List<ClusterConfig> forConsumerGroupCoordinator() { | |||
serverProperties.put(NEW_GROUP_COORDINATOR_ENABLE_CONFIG, "true"); | |||
|
|||
return Collections.singletonList(ClusterConfig.defaultBuilder() | |||
.setTypes(Stream.of(KRAFT, CO_KRAFT).collect(Collectors.toSet())) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you please add comments for this change?
@@ -75,6 +76,7 @@ static List<ClusterConfig> forClassicGroupCoordinator() { | |||
serverProperties.put(NEW_GROUP_COORDINATOR_ENABLE_CONFIG, "false"); | |||
|
|||
return Collections.singletonList(ClusterConfig.defaultBuilder() | |||
.setTypes(Stream.of(ZK, KRAFT).collect(Collectors.toSet())) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ditto
@m1a2st Could you please fix the conflicts? |
fa6df27
to
e862f7f
Compare
Ok, I resolved the conflict |
Reviewers: Chia-Ping Tsai <chia7712@gmail.com>
Reviewers: Chia-Ping Tsai <chia7712@gmail.com>
Reviewers: Chia-Ping Tsai <chia7712@gmail.com>
Because
ConsumerGroupOffsetTest
test for a long time, so I think that we should reduce the test time.Committer Checklist (excluded from commit message)