kbendick edited a comment on pull request #3284:
URL: https://github.com/apache/iceberg/pull/3284#issuecomment-941239952


   This is in reference to https://github.com/apache/iceberg/pull/3273, but the 
base implementation also does this and thus we should likely add this to the 
spec.
   
   **Note** - Marking this as a draft as we might want to also call out that 0 
indicates we should skip the file entirely (though that's perhaps overkill). 
Also, marking as draft as I'm open to formatting this differently.
   
   cc @RussellSpitzer @szehon-ho @aokolnychyi @rdblue 


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