Aihua Xu
Aihua Xu
@huaxingao Can you also take a look at this change?
> @aihuaxu When you have chance can you also review conceptually. Am I covering all cases of Iceberg's Variant ? Sorry that I missed this message. I will take a...
@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...
I'm attaching two simple test files for references. I named them .txt since I can't attach *.parquet directly. [primitive.parquet.txt](https://github.com/user-attachments/files/19639256/primitive.parquet.txt) The file contains id integer and var variant with variant being...
cc @rdblue, @RussellSpitzer and @flyrain
> This needs some notes in `Partition Transforms` , I think explicitly we should disallow identity > > For Appendix B - We should define something or state explicitly we...
> @aihuaxu, I think there are a couple of things missing: > > * The Avro appendix should be updated to state that a Variant is stored as a Record...
I believe I have addressed the comments and can we move forward to merge the PR? Let me know if I miss anything. cc @RussellSpitzer and @rdblue