rdblue commented on code in PR #12496:
URL: https://github.com/apache/iceberg/pull/12496#discussion_r1999837148
##########
.palantir/revapi.yml:
##########
@@ -1172,6 +1172,11 @@ acceptedBreaks:
\ java.util.function.Consumer<T>)"
justification: "Removing deprecated code"
Review Comment:
Yes, this PR translates Parquet metrics for shredded fields to a Variant
object that is accepted by `InclusiveMetricsEvaluator`. For Avro, we would need
to come up with a strategy for producing metrics. I'm not sure that it makes
sense to because there are no shredded fields.
--
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]
