sungwy commented on issue #1106:
URL: 
https://github.com/apache/iceberg-python/issues/1106#issuecomment-2315252871

   Hi @questsul - thank you for raising this issue, and for providing this 
analysis. The optional and required attributes in PyIceberg are based on the 
nullability of the objects as they are defined within the Rest Catalog Open API 
spec. Here, the operation field in summary is labeled to be a required 
attribute:
   
   
https://github.com/apache/iceberg/blob/8e2eb9ac2e33ce4bac8956d4e2f099444d03c0e3/open-api/rest-catalog-open-api.yaml#L2057-L2060
   
   I think there's a few takeaways here based on our findings:
   1. operation is a required field
   2. Java is interestingly more graceful in parsing the operation tag (and it 
probably should not be)
   3. The metadata files in your table are unfortunately not conformant to the 
REST API Spec. I think following up with the folks at Snowflake and seeing if 
there's a way to fix these metadata files (for example by making a copy and 
adding operation tags that are missing) may not be a bad idea


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