ckj996 opened a new pull request #2495:
URL: https://github.com/apache/ozone/pull/2495


   ## What changes were proposed in this pull request?
   
   **Request for Comments**
   This is our streaming client implementation. It’s still in the early stages, 
but at least it works.
   This might be a good start to discuss the design, comments are welcomed.
   
   New DataStreamOutput were added, compared with OutputStream, there are some 
outstanding changes:
   
   * `write(ByteBuf)` methods were added, which can be used for zero buffer 
copy later.
   * `write(byte[])` methods were kept, so these DataStreamOutput still extends 
OutputStream for compatibility.
   * `write(ByteBuf)` is perferred over `write(byte[])`, the latter is 
implemented by calling the former.
   * Zero buffer copy is not implemented yet, there is still buffer copy in 
`BlockDataStreamOutput#write`.
   * OzoneDataStreamOutput only supports KeyDataStreamOutput for now, Crypto 
and Cipher were dropped.
   
   Also, we have modified the ozone shell to use the streaming api for putting 
large keys (length >= chunkSize).
   
   ## What is the link to the Apache JIRA
   
   https://issues.apache.org/jira/browse/HDDS-5488
   
   ## How was this patch tested?
   
   If this design is ok, unit tests should be added.
   


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