stevie9868 opened a new pull request, #2193:
URL: https://github.com/apache/iceberg-python/pull/2193

   <!--
   Thanks for opening a pull request!
   -->
   
   <!-- In the case this PR will resolve an issue, please replace 
${GITHUB_ISSUE_ID} below with the actual Github issue id. -->
   <!-- Closes #${GITHUB_ISSUE_ID} -->
   
   # Rationale for this change
   This is the first change to support deletion vector write in pyiceberg, 
where we build an abstraction on top of the Bitmap for various operations.
   
   Jave implementation for 
[reference](https://github.com/apache/iceberg/blob/main/core/src/main/java/org/apache/iceberg/deletes/RoaringPositionBitmap.java)
   
   # Are these changes tested?
   Yes, I have added [all tests from 
Java](https://github.com/apache/iceberg/blob/main/core/src/test/java/org/apache/iceberg/deletes/TestRoaringPositionBitmap.java)
 except one that validates the ByteOrder as python bytes does not support Order 
like Java ByteBuffer
   
   # Are there any user-facing changes?
   No, this is a new class that is currently not used in any existing code path.
   
   <!-- In the case of user-facing changes, please add the changelog label. -->
   


-- 
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: issues-unsubscr...@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org

Reply via email to