Re: [PR] Add option to delete datafiles [iceberg-python]

2024-04-05 Thread via GitHub
Fokko commented on code in PR #569: URL: https://github.com/apache/iceberg-python/pull/569#discussion_r1553588970 ## pyiceberg/table/__init__.py: ## @@ -2726,6 +2731,112 @@ def _commit(self) -> UpdatesAndRequirements: ) +class DeleteFiles(_MergingSnapshotProducer):

Re: [PR] Add option to delete datafiles [iceberg-python]

2024-04-05 Thread via GitHub
Fokko commented on code in PR #569: URL: https://github.com/apache/iceberg-python/pull/569#discussion_r1553580296 ## pyiceberg/table/__init__.py: ## @@ -2726,6 +2731,112 @@ def _commit(self) -> UpdatesAndRequirements: ) +class DeleteFiles(_MergingSnapshotProducer):

Re: [PR] Add option to delete datafiles [iceberg-python]

2024-04-05 Thread via GitHub
Fokko commented on code in PR #569: URL: https://github.com/apache/iceberg-python/pull/569#discussion_r1553580296 ## pyiceberg/table/__init__.py: ## @@ -2726,6 +2731,112 @@ def _commit(self) -> UpdatesAndRequirements: ) +class DeleteFiles(_MergingSnapshotProducer):

Re: [PR] Add option to delete datafiles [iceberg-python]

2024-04-03 Thread via GitHub
jqin61 commented on code in PR #569: URL: https://github.com/apache/iceberg-python/pull/569#discussion_r1548568425 ## pyiceberg/table/__init__.py: ## @@ -2726,6 +2731,112 @@ def _commit(self) -> UpdatesAndRequirements: ) +class DeleteFiles(_MergingSnapshotProducer):

[PR] Add option to delete datafiles [iceberg-python]

2024-04-02 Thread via GitHub
Fokko opened a new pull request, #569: URL: https://github.com/apache/iceberg-python/pull/569 This is done through the Iceberg metadata, resulting in efficient deletes if the data is partitioned correctly -- This is an automated message from the Apache Git Service. To respond to the messa