cassandra-pr mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From aweisberg <>
Subject [GitHub] cassandra pull request #278: Avoid running query to self through messaging s...
Date Tue, 09 Oct 2018 21:50:46 GMT
Github user aweisberg commented on a diff in the pull request:
    --- Diff: src/java/org/apache/cassandra/net/ ---
    @@ -1078,7 +1078,7 @@ public void sendOneWay(MessageOut message, int id, InetAddressAndPort
                 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 --
    This really needs to be NoSpamLogger? How many times a second might we do this in some
cases? What if new code comes that causes this to occur many times a second?
    In OSS debug logging is on all the time. That's really not a risk we should be taking.


To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message