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

    https://github.com/apache/trafodion/pull/1369#discussion_r159730607
  
    --- Diff: dcs/src/main/java/org/trafodion/dcs/master/ServerManager.java ---
    @@ -240,15 +240,14 @@ public ScriptContext call() throws Exception {
                             }
                         }
                     } else {
    -                    if (LOG.isDebugEnabled())
    -                        LOG.debug("No restart for "
    -                                + znodePath
    -                                + "\nbecause DcsServer start time ["
    -                                + DateFormat.getDateTimeInstance().format(
    -                                        new Date(serverStartTimestamp))
    -                                + "] was before DcsMaster start time ["
    -                                + DateFormat.getDateTimeInstance().format(
    -                                        new Date(startupTimestamp)) + "]");
    +                    LOG.info("No restart for "
    --- End diff --
    
    Looks like there is another reason for no restart, namely 
(master.isFollower() && runningServers.size() < configuredServers.size()) ?


---

Reply via email to