aihuaxu commented on PR #14259:
URL: https://github.com/apache/iceberg/pull/14259#issuecomment-3492502943

   @talatuyarer  Overall the logic makes sense to me in which we are relying on 
the existing Parquet variant reader/writer to handle most of the work including 
shredding and what we need is mostly to convert between Flink Variant and 
Iceberg Variant. 
   
   As I read, shredding should work automatically. Can we add some basic 
validation for shredding coverage to make sure the Parquet schema is as 
expected?


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