niket-goel commented on code in PR #12206:
URL: https://github.com/apache/kafka/pull/12206#discussion_r881044720
##########
clients/src/main/java/org/apache/kafka/clients/admin/KafkaAdminClient.java:
##########
@@ -318,6 +325,11 @@ public class KafkaAdminClient extends AdminClient {
private final Logger log;
private final LogContext logContext;
+ /**
+ * The name of the internal raft metadata topic
+ */
+ private static final String METADATA_TOPIC_NAME = "__cluster_metadata";
Review Comment:
This is a great point! I think it makes sense to give ourselves wiggle room
and call this `MetadataQuorum`.
Re: moving the constant -- I was actually looking for a common place to put
it so that both the clients and the server code can use the same value. Is
there any class that they share? Right now the name for the topic on the server
side is defined in `KafkaRaftServer`.
--
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
