Re: [PR] add PARTITION_SUMMARY_PROP [iceberg-python]

2025-07-14 Thread via GitHub
kevinjqliu commented on PR #2202: URL: https://github.com/apache/iceberg-python/pull/2202#issuecomment-3071743878 Thanks for the PR @gtrettenero and @bryanck for the review :) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub a

Re: [PR] add PARTITION_SUMMARY_PROP [iceberg-python]

2025-07-14 Thread via GitHub
kevinjqliu merged PR #2202: URL: https://github.com/apache/iceberg-python/pull/2202 -- 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...@i

Re: [PR] add PARTITION_SUMMARY_PROP [iceberg-python]

2025-07-14 Thread via GitHub
kevinjqliu commented on code in PR #2202: URL: https://github.com/apache/iceberg-python/pull/2202#discussion_r2206169714 ## pyiceberg/table/snapshots.py: ## @@ -306,6 +307,8 @@ def build(self) -> Dict[str, str]: changed_partitions_size = len(self.partition_metrics)

[PR] add PARTITION_SUMMARY_PROP [iceberg-python]

2025-07-11 Thread via GitHub
gtrettenero opened a new pull request, #2202: URL: https://github.com/apache/iceberg-python/pull/2202 # Rationale for this change # Are these changes tested? # Are there any user-facing changes? -- This is an automated message from the Apache G