smiklosovic commented on code in PR #2649:
URL: https://github.com/apache/cassandra/pull/2649#discussion_r1311222958
##########
src/java/org/apache/cassandra/streaming/StreamDeserializingTask.java:
##########
@@ -109,6 +111,12 @@ public StreamSession deriveSession(StreamMessage message)
// Attach this channel to the session: this only happens upon
receiving the first init message as a follower;
// in all other cases, no new control channel will be added, as the
proper control channel will be already attached.
streamSession.attachInbound(channel);
+
+ if (streamSession.getStreamOperation() == StreamOperation.BULK_LOAD &&
!Guardrails.bulkLoadEnabled.isEnabled(null))
Review Comment:
We tend to omit braces when bodies of `if`s contain just one statement. In
this case you do not need braces here.
--
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]
