nssalian commented on code in PR #2045: URL: https://github.com/apache/iceberg-go/pull/2045#discussion_r4077138421
########## website/src/feature-status.md: ########## @@ -49,8 +49,9 @@ All V1 features are supported. V1 is the format-version baseline. | Default values (`initial-default`, `write-default`) | Supported | | Row lineage (`_row_id`, `_last_updated_sequence_number`) | Supported | | Encryption keys in metadata | Supported | -| Variant type, non-shredded | Supported (PR [#932](https://github.com/apache/iceberg-go/pull/932); umbrella [#929](https://github.com/apache/iceberg-go/issues/929)) | -| Variant type, shredded reader / writer | In progress ([#986](https://github.com/apache/iceberg-go/issues/986), [#987](https://github.com/apache/iceberg-go/issues/987)) | +| Variant type, non-shredded | Supported | +| Variant type, shredded reader / writer | Supported | +| Variant predicate pushdown / field extract | Supported | Review Comment: So I cleaned this up a little. Extract terms prune at the data-file level via shredded variant bounds (`scanner.go:484-490` -> `evaluators.go` `boundFor`/`VariantBoundLiteral`), not just per-row. What's skipped is row-group stats and bloom, which the caveat now states explicitly. Let me know if this works for you. -- 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]
