[GitHub] [pulsar] aahmed-se commented on a change in pull request #10962: Support new topic format for broker admin healthcheck endpoint

2021-06-22 Thread GitBox
aahmed-se commented on a change in pull request #10962: URL: https://github.com/apache/pulsar/pull/10962#discussion_r655738605 ## File path: pulsar-client-admin-api/src/main/java/org/apache/pulsar/client/admin/TopicVersion.java ## @@ -0,0 +1,24 @@ +/** + * Licensed to the Apac

[GitHub] [pulsar] aahmed-se commented on a change in pull request #10962: Support new topic format for broker admin healthcheck endpoint

2021-06-21 Thread GitBox
aahmed-se commented on a change in pull request #10962: URL: https://github.com/apache/pulsar/pull/10962#discussion_r655913864 ## File path: pulsar-broker/src/test/java/org/apache/pulsar/broker/admin/AdminApiHealthCheckTest.java ## @@ -0,0 +1,83 @@ +/** + * Licensed to the Apa

[GitHub] [pulsar] aahmed-se commented on a change in pull request #10962: Support new topic format for broker admin healthcheck endpoint

2021-06-21 Thread GitBox
aahmed-se commented on a change in pull request #10962: URL: https://github.com/apache/pulsar/pull/10962#discussion_r655801162 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/namespace/NamespaceService.java ## @@ -142,6 +142,7 @@ public static final Pat

[GitHub] [pulsar] aahmed-se commented on a change in pull request #10962: Support new topic format for broker admin healthcheck endpoint

2021-06-21 Thread GitBox
aahmed-se commented on a change in pull request #10962: URL: https://github.com/apache/pulsar/pull/10962#discussion_r655801162 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/namespace/NamespaceService.java ## @@ -142,6 +142,7 @@ public static final Pat

[GitHub] [pulsar] aahmed-se commented on a change in pull request #10962: Support new topic format for broker admin healthcheck endpoint

2021-06-21 Thread GitBox
aahmed-se commented on a change in pull request #10962: URL: https://github.com/apache/pulsar/pull/10962#discussion_r655786267 ## File path: pulsar-broker/src/test/java/org/apache/pulsar/broker/admin/AdminApiHealthCheckTest.java ## @@ -0,0 +1,83 @@ +/** + * Licensed to the Apa

[GitHub] [pulsar] aahmed-se commented on a change in pull request #10962: Support new topic format for broker admin healthcheck endpoint

2021-06-21 Thread GitBox
aahmed-se commented on a change in pull request #10962: URL: https://github.com/apache/pulsar/pull/10962#discussion_r655774547 ## File path: pulsar-client-tools/src/main/java/org/apache/pulsar/client/cli/CmdConsume.java ## @@ -414,7 +414,7 @@ private int consumeFromWebSocket(S

[GitHub] [pulsar] aahmed-se commented on a change in pull request #10962: Support new topic format for broker admin healthcheck endpoint

2021-06-21 Thread GitBox
aahmed-se commented on a change in pull request #10962: URL: https://github.com/apache/pulsar/pull/10962#discussion_r655768064 ## File path: pulsar-broker/src/test/java/org/apache/pulsar/broker/admin/AdminApiHealthCheckTest.java ## @@ -0,0 +1,83 @@ +/** + * Licensed to the Apa

[GitHub] [pulsar] aahmed-se commented on a change in pull request #10962: Support new topic format for broker admin healthcheck endpoint

2021-06-21 Thread GitBox
aahmed-se commented on a change in pull request #10962: URL: https://github.com/apache/pulsar/pull/10962#discussion_r655738605 ## File path: pulsar-client-admin-api/src/main/java/org/apache/pulsar/client/admin/TopicVersion.java ## @@ -0,0 +1,24 @@ +/** + * Licensed to the Apac