Github user kevinxu021 commented on a diff in the pull request:

    https://github.com/apache/trafodion/pull/1427#discussion_r172734228
  
    --- Diff: dcs/src/main/java/org/trafodion/dcs/master/DcsMaster.java ---
    @@ -203,8 +244,8 @@ public void run() {
             } catch (KeeperException.NodeExistsException e) {
                 // do nothing...some other server has created znodes
    --- End diff --
    
    it should be better to add warning for this exception that we could what 
happened. Any events happened should be reflected in log files.


---

Reply via email to