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


##########
streams/src/main/java/org/apache/kafka/streams/StreamsConfig.java:
##########
@@ -1076,7 +1082,12 @@ public class StreamsConfig extends AbstractConfig {
                     Type.LONG,
                     null,
                     Importance.LOW,
-                    WINDOW_SIZE_MS_DOC);
+                    WINDOW_SIZE_MS_DOC)
+            .define(DEFAULT_CLIENT_SUPPLIER_CONFIG,

Review Comment:
   nit: we keep the configs grouped by priority and order alphabetically within 
each group. Can we more this accordingly?



##########
streams/src/main/java/org/apache/kafka/streams/StreamsConfig.java:
##########
@@ -1727,6 +1738,15 @@ public static Set<String> 
verifyTopologyOptimizationConfigs(final String config)
         return verifiedConfigs;
     }
 
+    /**
+     * Return configured KafkaClientSupplier
+     * @return Configured KafkaClientSupplier
+     */
+    public KafkaClientSupplier getKafkaClientSupplier() {

Review Comment:
   Thanks. Can you send a follow up email to the vote thread, calling out the 
change?
   
   We don't need to re-vote but if anybody would have an objection, we need to 
give them a chance to race 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: jira-unsubscr...@kafka.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to