Re: [PR] Allow for missing operation [iceberg-python]

2024-10-31 Thread via GitHub
Fokko merged PR #1263: URL: https://github.com/apache/iceberg-python/pull/1263 -- 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...@iceber

Re: [PR] Allow for missing operation [iceberg-python]

2024-10-29 Thread via GitHub
omkenge commented on PR #1263: URL: https://github.com/apache/iceberg-python/pull/1263#issuecomment-2444621663 Hello @Fokko Here in inspect.py "You can see that we’ve already set the operation value to None when it’s not present in the summary. Can we remove this operation field here to

Re: [PR] Allow for missing operation [iceberg-python]

2024-10-29 Thread via GitHub
omkenge commented on PR #1263: URL: https://github.com/apache/iceberg-python/pull/1263#issuecomment-2444593734 Hello @Fokko "You can see that we’ve already set the operation value to None when it’s not present in the summary. Can we remove this operation field here to avoid conflict?"

[PR] Allow for missing operation [iceberg-python]

2024-10-29 Thread via GitHub
Fokko opened a new pull request, #1263: URL: https://github.com/apache/iceberg-python/pull/1263 And then default to `overwrite`, see: https://lists.apache.org/thread/h9qmrmlgxh91ol0y2v8olt90b9q6p9xr -- This is an automated message from the Apache Git Service. To respond to the message,