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

    https://github.com/apache/cassandra/pull/191#discussion_r167016420
  
    --- Diff: src/java/org/apache/cassandra/net/MessagingService.java ---
    @@ -254,6 +258,8 @@ public long getTimeout()
                     return DatabaseDescriptor.getRangeRpcTimeout();
                 }
             },
    +        PING(),
    --- End diff --
    
    Should this go this early in the enum before USUED_* and company? Does it 
make sense to ever insert anything into the middle of the enum?


---

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

Reply via email to