FrankYang0529 commented on code in PR #19743:
URL: https://github.com/apache/kafka/pull/19743#discussion_r2100067823


##########
core/src/test/scala/integration/kafka/api/PlaintextAdminIntegrationTest.scala:
##########
@@ -3868,6 +3868,92 @@ class PlaintextAdminIntegrationTest extends 
BaseAdminIntegrationTest {
     } finally client.close(time.Duration.ZERO)
   }
 
+  @Test
+  def testListConfigResources(): Unit = {
+    client = createAdminClient
+
+    // Alter group and client metric config to add group and client metric 
config resource
+    val clientMetric = "client-metrics"
+    val group = "group"
+    val clientMetricResource = new 
ConfigResource(ConfigResource.Type.CLIENT_METRICS, clientMetric)
+    val groupResource = new ConfigResource(ConfigResource.Type.GROUP, group)
+    val alterResult = client.incrementalAlterConfigs(util.Map.of(

Review Comment:
   This is a scala file and I would like to avoid java/scala transfer, so I use 
java map here. Same as java set above.



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

Reply via email to