[GitHub] [iceberg] kbendick edited a comment on pull request #3284: Spec - Add -1 to Manifest Entry's data_file.record_count to indicate unknown

[GitBox <gi...@apache.org>]

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