Github user ifesdjeen commented on a diff in the pull request:
https://github.com/apache/cassandra/pull/278#discussion_r224060251
--- Diff: src/java/org/apache/cassandra/net/MessagingService.java ---
@@ -1078,7 +1078,7 @@ public void sendOneWay(MessageOut message, int id,
InetAddressAndPort to)
logger.trace("{} sending {} to {}@{}",
FBUtilities.getBroadcastAddressAndPort(), message.verb, id, to);
if (to.equals(FBUtilities.getBroadcastAddressAndPort()))
- logger.trace("Message-to-self {} going over MessagingService",
message);
+ logger.debug("Message-to-self {} going over MessagingService",
message);
--- End diff --
To be honest, I think relying on the logging might have been a bad idea.
I've left this statement with `trace` and have rewritten the tests instead.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscr...@cassandra.apache.org
For additional commands, e-mail: pr-h...@cassandra.apache.org
