[GitHub] [hbase] bharathv commented on a change in pull request #1945: HBASE-24603: Make Zookeeper sync() call synchronous

2020-06-25 Thread GitBox
bharathv commented on a change in pull request #1945: URL: https://github.com/apache/hbase/pull/1945#discussion_r445723003 ## File path: hbase-zookeeper/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKWatcher.java ## @@ -479,29 +502,15 @@ public ZNodePaths getZNodePaths() {

[GitHub] [hbase] bharathv commented on a change in pull request #1945: HBASE-24603: Make Zookeeper sync() call synchronous

2020-06-24 Thread GitBox
bharathv commented on a change in pull request #1945: URL: https://github.com/apache/hbase/pull/1945#discussion_r445276172 ## File path: hbase-zookeeper/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKWatcher.java ## @@ -595,9 +604,28 @@ private void

[GitHub] [hbase] bharathv commented on a change in pull request #1945: HBASE-24603: Make Zookeeper sync() call synchronous

2020-06-22 Thread GitBox
bharathv commented on a change in pull request #1945: URL: https://github.com/apache/hbase/pull/1945#discussion_r443737875 ## File path: hbase-zookeeper/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKWatcher.java ## @@ -595,9 +604,28 @@ private void