mjsax commented on code in PR #23322:
URL: https://github.com/apache/kafka/pull/23322#discussion_r3907127559


##########
group-coordinator/src/main/java/org/apache/kafka/coordinator/group/GroupCoordinatorConfig.java:
##########
@@ -433,6 +435,10 @@ public class GroupCoordinatorConfig {
     public static final Class<?> 
STREAMS_GROUP_TOPOLOGY_DESCRIPTION_PLUGIN_CLASS_DEFAULT = null;
     public static final String 
STREAMS_GROUP_TOPOLOGY_DESCRIPTION_PLUGIN_CLASS_DOC = "The fully qualified 
class name of a StreamsGroupTopologyDescriptionPlugin implementation. When not 
set, the feature is disabled.";
 
+    public static final String STREAMS_GROUP_ASSIGNMENT_REFINER_CLASS_CONFIG = 
"group.streams.assignment.refiner.class";
+    public static final Class<?> 
STREAMS_GROUP_ASSIGNMENT_REFINER_CLASS_DEFAULT = null;

Review Comment:
   I actually think yes -- did notice the same before I pushed, but was "too 
lazy" to change -- but this config should never be `null`, so it makes sense to 
have a proper default.



-- 
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]

Reply via email to