[GitHub] [kafka] jolshan commented on a change in pull request #10286: KAFKA-12318: system tests need to fetch Topic IDs via Admin Client instead of via ZooKeeper

[GitBox]

jolshan commented on a change in pull request #10286:
URL: https://github.com/apache/kafka/pull/10286#discussion_r590798792



##########
File path: tests/kafkatest/version.py
##########
@@ -101,7 +101,7 @@ def kafka_configs_command_uses_bootstrap_server_scram(self):
         # User SCRAM Credentials (KIP-554)
         return self >= V_2_7_0
 
-    def supports_topic_ids(self):
+    def zk_supports_topic_ids(self):

Review comment:
       I was having trouble naming this haha. This is a better name.




----------------------------------------------------------------
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.

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