dajac commented on code in PR #16945:
URL: https://github.com/apache/kafka/pull/16945#discussion_r1730706726
##########
group-coordinator/src/main/java/org/apache/kafka/coordinator/group/GroupCoordinatorConfig.java:
##########
@@ -71,7 +71,7 @@ public class GroupCoordinatorConfig {
/** New group coordinator configs */
public static final String NEW_GROUP_COORDINATOR_ENABLE_CONFIG =
"group.coordinator.new.enable";
public static final String NEW_GROUP_COORDINATOR_ENABLE_DOC = "Enable the
new group coordinator.";
- public static final boolean NEW_GROUP_COORDINATOR_ENABLE_DEFAULT = false;
+ public static final boolean NEW_GROUP_COORDINATOR_ENABLE_DEFAULT = true;
Review Comment:
I will remove all the `kraft+kip848` and that condition in a subsequent PR.
We really want all the existing tests to use the new coordinator now. If
someone wants to use the old coordinator for some tests, he will have to
explicitly set the configuration to disable it.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
