slfan1989 commented on code in PR #1267:
URL: https://github.com/apache/ratis/pull/1267#discussion_r2099929193


##########
ratis-test/src/test/java/org/apache/ratis/datastream/DataStreamTestUtils.java:
##########
@@ -154,7 +154,7 @@ class MultiDataStreamStateMachine extends BaseStateMachine {
     @Override
     public CompletableFuture<DataStream> stream(RaftClientRequest request) {
       final SingleDataStream s = new SingleDataStream(request);
-      LOG.info("XXX {} put {}, {}", this, ClientInvocationId.valueOf(request), 
s);

Review Comment:
   I optimized the logs by removing the `XXX` prefix. I understand it was 
likely added for debugging purposes, but such prefixes are not appropriate in a 
production environment.
   



-- 
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]

Reply via email to