dino2895 opened a new pull request, #11093:
URL: https://github.com/apache/ozone/pull/11093

   ## What changes were proposed in this pull request?
   
   `MockDatanodeStorage` currently stores full block data as `String`, which 
requires binary chunk data to be decoded as UTF-8. This conversion is lossy for 
arbitrary binary data, including EC parity data.
   
   This change:
   
   * Stores `fullBlockData` values as `ByteString`.
   * Reuses the defensive copy created when chunk data is stored.
   * Removes UTF-8 conversions when accumulating full block data.
   * Updates `TestOzoneECClient` assertions to compare binary data exactly.
   
   ## What is the link to the Apache JIRA
   
   https://issues.apache.org/jira/browse/HDDS-10478
   
   ## How was this patch tested?
   
   * `mvn -pl :ozone-client test -Dtest=TestOzoneECClient`
     * All 34 tests passed.
   * `./hadoop-ozone/dev-support/checks/checkstyle.sh`
     * Passed with no Checkstyle violations.
   


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