Re: [PR] Puffin: Add delete-vector-v1 blob type [iceberg]

["rdblue (via GitHub)" <gi...@apache.org>]

rdblue commented on code in PR #11238:
URL: https://github.com/apache/iceberg/pull/11238#discussion_r1794211865


##########
format/puffin-spec.md:
##########
@@ -123,6 +123,44 @@ The blob metadata for this blob may include following properties:
 
 - `ndv`: estimate of number of distinct values, derived from the sketch.
 
+#### `delete-vector-v1` blob type
+
+A serialized delete vector that represents the positions of rows in a file that
+are deleted.  A set bit at position P indicates that the row at position P is
+deleted.
+
+The bitmap supports positive 64-bit positions, but is optimized for cases where

Review Comment:
   Does it matter? These positions are row positions. There is never going to be a file with that many rows. Many, many other things would fail first.



-- 
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-unsubscribe@iceberg.apache.org

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


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