dongnuo123 commented on code in PR #19904: URL: https://github.com/apache/kafka/pull/19904#discussion_r2135951958
########## group-coordinator/src/main/java/org/apache/kafka/coordinator/group/GroupCoordinatorConfig.java: ########## @@ -204,6 +205,11 @@ public class GroupCoordinatorConfig { ConsumerGroupMigrationPolicy.DOWNGRADE + ": only downgrade from consumer group to classic group is enabled, " + ConsumerGroupMigrationPolicy.DISABLED + ": neither upgrade nor downgrade is enabled."; + public static final String CONSUMER_GROUP_REGEX_BATCH_REFRESH_MAX_INTERVAL_MS_CONFIG = "group.consumer.regex.batch.refresh.max.interval.ms"; + public static final String CONSUMER_GROUP_REGEX_BATCH_REFRESH_MAX_INTERVAL_MS_DOC = "The interval at which the group coordinator will refresh " + + "the topics matching the group subscribed regexes. This is only applicable to consumer groups using the consumer group protocol. "; + public static final int CONSUMER_GROUP_REGEX_BATCH_REFRESH_MAX_INTERVAL_MS_DEFAULT = 10 * 60 * 1000; // 10 minutes + Review Comment: Yes it makes sense. I wanted to differentiate it with `REGEX_BATCH_REFRESH_MIN_INTERVAL_MS` so regex.refresh.interval.ms also works -- 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