matthias-Q commented on PR #2173:
URL: https://github.com/apache/iceberg-python/pull/2173#issuecomment-3039091062

   Some notes:
   1. UUID is also not 100% in line with the Avro Schema specification. But I 
saw, that there are other PRs pending that fixes UUID related issues. (#2007)
   
   3. Avro schema does not have a notion of `field-id` and `element-id`. I 
could add a helper function that would add these. I know this is not the core 
responsibility of this library. I was using this to create iceberg tables from 
Kafka topics, where the schema is stored in the schema registry. I think this 
is a viable use case and hence these helpers would add value.
   


-- 
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: issues-unsubscr...@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org

Reply via email to