[GitHub] [kafka] showuon commented on a diff in pull request #14202: KAFKA-15329: Make default remote.log.metadata.manager.class.name as topic based RLMM

2023-08-15 Thread via GitHub
showuon commented on code in PR #14202: URL: https://github.com/apache/kafka/pull/14202#discussion_r1294388761 ## storage/src/test/java/org/apache/kafka/server/log/remote/storage/RemoteLogManagerConfigTest.java: ## @@ -32,21 +36,28 @@ public TestConfig(Map originals) {

[GitHub] [kafka] showuon commented on a diff in pull request #14202: KAFKA-15329: Make default remote.log.metadata.manager.class.name as topic based RLMM

2023-08-14 Thread via GitHub
showuon commented on code in PR #14202: URL: https://github.com/apache/kafka/pull/14202#discussion_r1294184526 ## storage/src/test/java/org/apache/kafka/server/log/remote/storage/RemoteLogManagerConfigTest.java: ## @@ -52,6 +54,28 @@ public void testValidConfigs() {