Fix GroupRebalanceConfig to take in rack id as an optional arg

[Aditya Addepalli (aaddepalli7)]

AK changed the definition of this config - https://issues.apache.org/jira/browse/KAFKA-17747
https://github.com/apache/kafka/blame/trunk/clients/src/main/java/org/apache/kafka/clients/GroupRebalanceConfig.java#L74

[confluent-cla-assistant]

🎉 All Contributor License Agreements have been signed. Ready to merge.
_{Please push an empty commit if you would like to re-run the checks to verify CLA status for all contributors.}

[Copilot]

Pull Request Overview

This PR updates the SchemaRegistryCoordinator to pass the new optional rack ID argument in line with the Kafka GroupRebalanceConfig API change.

• Adds a second Optional.empty() to the constructor call to fill the new rack ID parameter.
• Aligns the coordinator instantiation with the updated Kafka client config signature.

Comments suppressed due to low confidence (1)

core/src/main/java/io/confluent/kafka/schemaregistry/leaderelector/kafka/SchemaRegistryCoordinator.java:84

• There are no unit tests covering the new rack ID parameter behavior. Consider adding tests to ensure the coordinator handles both present and absent rack IDs correctly.

Optional.empty(),

[Copilot]

[nitpick] The duplicated Optional.empty() calls are ambiguous. Consider using named constants or inline comments to clarify which Optional.empty() corresponds to the group instance ID and which to the new rack ID parameter.

Suggested change

	Optional.empty(),
	Optional.empty(),
	GROUP_INSTANCE_ID_EMPTY,
	RACK_ID_EMPTY,

[sonarqube-confluent]

Analysis Details

0 Issues

• 0 Bugs
• 0 Vulnerabilities
• 0 Code Smells

Coverage and Duplications

• 0.00% Coverage (0.00% Estimated after merge)
• No duplication information (2.00% Estimated after merge)

Project ID: schema-registry

View in SonarQube