[GitHub] [hbase] Apache9 commented on a diff in pull request #5370: HBASE-28038 Add TLS settings to ZooKeeper client

2023-09-03 Thread via GitHub
Apache9 commented on code in PR #5370: URL: https://github.com/apache/hbase/pull/5370#discussion_r1314265464 ## hbase-common/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKConfig.java: ## @@ -330,4 +341,27 @@ public static String getClientZKQuorumServersString(Configuration

[GitHub] [hbase] Apache9 commented on a diff in pull request #5370: HBASE-28038 Add TLS settings to ZooKeeper client

2023-08-27 Thread via GitHub
Apache9 commented on code in PR #5370: URL: https://github.com/apache/hbase/pull/5370#discussion_r1306614354 ## hbase-common/src/main/java/org/apache/hadoop/hbase/HConstants.java: ## @@ -219,6 +222,12 @@ public enum OperationStatusCode { public static final String CLIENT_ZOOK