Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/2741#discussion_r198638777
--- Diff:
storm-client/src/jvm/org/apache/storm/pacemaker/PacemakerClient.java ---
@@ -153,13 +154,13 @@ public String secretKey() {
}
public HBMessage send(HBMessage m) throws
PacemakerConnectionException, InterruptedException {
- LOG.debug("Sending message: {}", m.toString());
+ LOG.debug("Sending pacemaker message to {}: {}", host,
m.toString());
--- End diff --
nit: can we drop the `.toString()` for m? The log command should do it for
you, if it is needed.
---
