FrankYang0529 commented on code in PR #19856:
URL: https://github.com/apache/kafka/pull/19856#discussion_r2126838780
##########
clients/src/main/java/org/apache/kafka/clients/GroupRebalanceConfig.java:
##########
@@ -53,8 +54,14 @@ public GroupRebalanceConfig(AbstractConfig config,
ProtocolType protocolType) {
// Consumer and Connect use different config names for defining
rebalance timeout
if ((protocolType == ProtocolType.CONSUMER) || (protocolType ==
ProtocolType.SHARE)) {
this.rebalanceTimeoutMs =
config.getInt(CommonClientConfigs.MAX_POLL_INTERVAL_MS_CONFIG);
+ // Connect doesn't support rack id.
+ // The GroupCoordinatorService throws error if the rackId is
empty. The default value of client.rack is empty string.
+ // Skip empty rackId to avoid InvalidRequestException.
Review Comment:
Removed it. Thanks.
--
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
