maedhroz commented on a change in pull request #1146:
URL: https://github.com/apache/cassandra/pull/1146#discussion_r693101683
##########
File path: src/java/org/apache/cassandra/net/InboundConnectionInitiator.java
##########
@@ -82,6 +84,9 @@
@Override
public void initChannel(SocketChannel channel) throws Exception
{
+ // if any of the handlers added fail they will send the error to
the "head", so this needs to be first
+ channel.pipeline().addFirst("Internode Error Exclussions", new
InternodeErrorExclussionHandler());
Review comment:
nit: Factor out a constant for this name?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
