aihuaxu commented on code in PR #12496:
URL: https://github.com/apache/iceberg/pull/12496#discussion_r1999085447


##########
.palantir/revapi.yml:
##########
@@ -1172,6 +1172,11 @@ acceptedBreaks:
         \ java.util.function.Consumer<T>)"
       justification: "Removing deprecated code"

Review Comment:
   To help me understand the scope of this PR: we will collect the metrics for 
Variant subcolumns but the storing them as Variant to Avro files and retrieving 
from Avro files will be separate since that depends on Avro Variant read/write 
implementation. And also the pruning with such metrics will be separate as 
well. 
   
   Is that correct?



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

Reply via email to