[GitHub] [kafka] clolov commented on a diff in pull request #13122: KAFKA-14594: Move LogDirsCommand to tools module

2023-05-03 Thread via GitHub
clolov commented on code in PR #13122: URL: https://github.com/apache/kafka/pull/13122#discussion_r1183882730 ## tools/src/test/java/org/apache/kafka/tools/LogDirsCommandTest.java: ## @@ -0,0 +1,119 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more +

[GitHub] [kafka] clolov commented on a diff in pull request #13122: KAFKA-14594: Move LogDirsCommand to tools module

2023-05-03 Thread via GitHub
clolov commented on code in PR #13122: URL: https://github.com/apache/kafka/pull/13122#discussion_r1183881591 ## tools/src/test/java/org/apache/kafka/tools/LogDirsCommandTest.java: ## @@ -0,0 +1,119 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more +

[GitHub] [kafka] clolov commented on a diff in pull request #13122: KAFKA-14594: Move LogDirsCommand to tools module

2023-05-02 Thread via GitHub
clolov commented on code in PR #13122: URL: https://github.com/apache/kafka/pull/13122#discussion_r1182519045 ## tools/src/test/java/org/apache/kafka/tools/LogDirsCommandTest.java: ## @@ -0,0 +1,83 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + *

[GitHub] [kafka] clolov commented on a diff in pull request #13122: KAFKA-14594: Move LogDirsCommand to tools module

2023-05-02 Thread via GitHub
clolov commented on code in PR #13122: URL: https://github.com/apache/kafka/pull/13122#discussion_r1182518605 ## tools/src/test/java/org/apache/kafka/tools/LogDirsCommandTest.java: ## @@ -0,0 +1,83 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + *

[GitHub] [kafka] clolov commented on a diff in pull request #13122: KAFKA-14594: Move LogDirsCommand to tools module

2023-02-06 Thread via GitHub
clolov commented on code in PR #13122: URL: https://github.com/apache/kafka/pull/13122#discussion_r1097662809 ## clients/src/test/java/org/apache/kafka/clients/admin/MockAdminClient.java: ## @@ -870,7 +870,37 @@ synchronized public AlterReplicaLogDirsResult alterReplicaLogDirs(

[GitHub] [kafka] clolov commented on a diff in pull request #13122: KAFKA-14594: Move LogDirsCommand to tools module

2023-01-17 Thread GitBox
clolov commented on code in PR #13122: URL: https://github.com/apache/kafka/pull/13122#discussion_r1072326336 ## tools/src/main/java/org/apache/kafka/tools/LogDirsCommand.java: ## @@ -0,0 +1,158 @@ +/** + * Licensed to the Apache Software Foundation (ASF) under one or more + * c

[GitHub] [kafka] clolov commented on a diff in pull request #13122: KAFKA-14594: Move LogDirsCommand to tools module

2023-01-17 Thread GitBox
clolov commented on code in PR #13122: URL: https://github.com/apache/kafka/pull/13122#discussion_r1072271288 ## tools/src/main/java/org/apache/kafka/tools/LogDirsCommand.java: ## @@ -0,0 +1,158 @@ +/** + * Licensed to the Apache Software Foundation (ASF) under one or more + * c

[GitHub] [kafka] clolov commented on a diff in pull request #13122: KAFKA-14594: Move LogDirsCommand to tools module

2023-01-17 Thread GitBox
clolov commented on code in PR #13122: URL: https://github.com/apache/kafka/pull/13122#discussion_r1072271288 ## tools/src/main/java/org/apache/kafka/tools/LogDirsCommand.java: ## @@ -0,0 +1,158 @@ +/** + * Licensed to the Apache Software Foundation (ASF) under one or more + * c

[GitHub] [kafka] clolov commented on a diff in pull request #13122: KAFKA-14594: Move LogDirsCommand to tools module

2023-01-17 Thread GitBox
clolov commented on code in PR #13122: URL: https://github.com/apache/kafka/pull/13122#discussion_r1072024066 ## tools/src/main/java/org/apache/kafka/tools/LogDirsCommand.java: ## @@ -0,0 +1,158 @@ +/** + * Licensed to the Apache Software Foundation (ASF) under one or more + * c

[GitHub] [kafka] clolov commented on a diff in pull request #13122: KAFKA-14594: Move LogDirsCommand to tools module

2023-01-17 Thread GitBox
clolov commented on code in PR #13122: URL: https://github.com/apache/kafka/pull/13122#discussion_r1072022574 ## tools/src/main/java/org/apache/kafka/tools/LogDirsCommand.java: ## @@ -0,0 +1,158 @@ +/** + * Licensed to the Apache Software Foundation (ASF) under one or more + * c

[GitHub] [kafka] clolov commented on a diff in pull request #13122: KAFKA-14594: Move LogDirsCommand to tools module

2023-01-17 Thread GitBox
clolov commented on code in PR #13122: URL: https://github.com/apache/kafka/pull/13122#discussion_r1072022248 ## tools/src/main/java/org/apache/kafka/tools/LogDirsCommand.java: ## @@ -0,0 +1,158 @@ +/** + * Licensed to the Apache Software Foundation (ASF) under one or more + * c