guohao-rosicky commented on code in PR #5791:
URL: https://github.com/apache/ozone/pull/5791#discussion_r1469494041


##########
hadoop-ozone/client/src/main/java/org/apache/hadoop/ozone/client/rpc/RpcClient.java:
##########
@@ -2358,7 +2366,7 @@ private KeyOutputStream.Builder createKeyOutputStream(
         .enableUnsafeByteBufferConversion(unsafeByteBufferConversion)
         .setConfig(clientConfig)
         .setAtomicKeyCreation(isS3GRequest.get())
-        .setClientMetrics(clientMetrics)
+        
.setBlockOutPutStreamResourceProvider(blockOutPutStreamResourceProvider)

Review Comment:
   Hi @adoroszlai @xichen01 , Thanks work on this, I think the reason for the 
current SingleThreadExecutor is because it uses asynchrony in an OutputStream 
and needs to guarantee the order of execution. If that's the case, is switching 
to a multi-threaded Executor still a guarantee?



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

Reply via email to