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

    https://github.com/apache/cassandra/pull/278#discussion_r223396208
  
    --- 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.warn("Message-to-self {} going over MessagingService", 
message);
    --- End diff --
    
    Use NoSpamLogger and maybe for the key incorporate the stack trace so we 
rate limit every call site separately?


---

---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscr...@cassandra.apache.org
For additional commands, e-mail: pr-h...@cassandra.apache.org

Reply via email to