[GitHub] [flink] zentol commented on a diff in pull request #21127: [FLINK-29707][cli] Fix possible comparator violation for "flink list"

2022-10-21 Thread GitBox
zentol commented on code in PR #21127: URL: https://github.com/apache/flink/pull/21127#discussion_r1001671017 ## flink-clients/src/test/java/org/apache/flink/client/cli/CliFrontendListTest.java: ## @@ -28,21 +28,100 @@ import org.junit.jupiter.api.AfterAll; import org.junit.ju

[GitHub] [flink] zentol commented on a diff in pull request #21127: [FLINK-29707][cli] Fix possible comparator violation for "flink list"

2022-10-21 Thread GitBox
zentol commented on code in PR #21127: URL: https://github.com/apache/flink/pull/21127#discussion_r1001669758 ## flink-clients/src/test/java/org/apache/flink/client/cli/CliFrontendListTest.java: ## @@ -28,21 +28,100 @@ import org.junit.jupiter.api.AfterAll; import org.junit.ju

[GitHub] [flink] zentol commented on a diff in pull request #21127: [FLINK-29707][cli] Fix possible comparator violation for "flink list"

2022-10-20 Thread GitBox
zentol commented on code in PR #21127: URL: https://github.com/apache/flink/pull/21127#discussion_r1001199344 ## flink-clients/src/test/java/org/apache/flink/client/cli/CliFrontendListTest.java: ## @@ -28,21 +28,100 @@ import org.junit.jupiter.api.AfterAll; import org.junit.ju

[GitHub] [flink] zentol commented on a diff in pull request #21127: [FLINK-29707][cli] Fix possible comparator violation for "flink list"

2022-10-20 Thread GitBox
zentol commented on code in PR #21127: URL: https://github.com/apache/flink/pull/21127#discussion_r1000760604 ## flink-clients/src/test/java/org/apache/flink/client/cli/CliFrontendListTest.java: ## @@ -28,21 +28,100 @@ import org.junit.jupiter.api.AfterAll; import org.junit.ju

[GitHub] [flink] zentol commented on a diff in pull request #21127: [FLINK-29707][cli] Fix possible comparator violation for "flink list"

2022-10-20 Thread GitBox
zentol commented on code in PR #21127: URL: https://github.com/apache/flink/pull/21127#discussion_r1000760604 ## flink-clients/src/test/java/org/apache/flink/client/cli/CliFrontendListTest.java: ## @@ -28,21 +28,100 @@ import org.junit.jupiter.api.AfterAll; import org.junit.ju

[GitHub] [flink] zentol commented on a diff in pull request #21127: [FLINK-29707][cli] Fix possible comparator violation for "flink list"

2022-10-20 Thread GitBox
zentol commented on code in PR #21127: URL: https://github.com/apache/flink/pull/21127#discussion_r1000760604 ## flink-clients/src/test/java/org/apache/flink/client/cli/CliFrontendListTest.java: ## @@ -28,21 +28,100 @@ import org.junit.jupiter.api.AfterAll; import org.junit.ju