Github user mashengchen commented on a diff in the pull request:
https://github.com/apache/trafodion/pull/1369#discussion_r159793870
--- 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 --
yes . if backup-master take over from the failed master, then the
'isFollower' is true, the result will be the code does not run the if
conditions, and dcs server not restart
---
