captainzmc commented on pull request #2452:
URL: https://github.com/apache/ozone/pull/2452#issuecomment-885376538


   > Thanks @captainzmc for working on this. I think, we need define a config 
option to enable/disable the feature both on client and server.
   
   Thanks @bshashikant for the review and suggestions.
   
   On the server side, I think it's easy to distinguish. We added a stream 
method to ContainerStateMachine that follows the stream Init logic. This method 
is executed only if invoked actively by the client.  The new interface on the 
server has no impact on the existing logic.
   About the client. Can we write a key based on the size of the key and decide 
whether to use stream?  We can configure in putKeyHandler.java.  And if the 
user uses the API directly, they can call our new interface directly. We won't 
make any changes to the existing Key/Block OutPutStream. We will add some new 
interfaces as described in the previous design document values.
   
![image](https://user-images.githubusercontent.com/13825159/126734705-7f37384d-9917-4ff9-a199-15ce7d232668.png)
   


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