Github user DaveBirdsall commented on a diff in the pull request:
https://github.com/apache/trafodion/pull/1369#discussion_r159806194
--- 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 --
RIght... My point was that the text in the logging seems to need updating,
because the 'if' conditions have changed. What do you think?
---
