Re: [PR] partitioned write support [iceberg-python]

2025-02-11 Thread via GitHub
kevinjqliu commented on PR #353: URL: https://github.com/apache/iceberg-python/pull/353#issuecomment-2652609001 hey @sungwy @jqin61 just wanted to double check that this PR is no longer relevant. I believe all components of partitioned write support has been already merged -- This is an

Re: [PR] partitioned write support [iceberg-python]

2024-03-28 Thread via GitHub
jqin61 commented on PR #353: URL: https://github.com/apache/iceberg-python/pull/353#issuecomment-2025344888 As discussed in the monthly meeting, this will be broken down into 4 prs of: 1. Partitioned append with identity transform 2. Dynamic overwrite using delete + append, 2 snapshots

Re: [PR] partitioned write support [iceberg-python]

2024-02-05 Thread via GitHub
jqin61 commented on code in PR #353: URL: https://github.com/apache/iceberg-python/pull/353#discussion_r1478509127 ## pyiceberg/manifest.py: ## @@ -308,6 +308,7 @@ def data_file_with_partition(partition_type: StructType, format_version: Literal field_id=field.field

Re: [PR] partitioned write support [iceberg-python]

2024-02-05 Thread via GitHub
jqin61 commented on code in PR #353: URL: https://github.com/apache/iceberg-python/pull/353#discussion_r1478509127 ## pyiceberg/manifest.py: ## @@ -308,6 +308,7 @@ def data_file_with_partition(partition_type: StructType, format_version: Literal field_id=field.field

Re: [PR] partitioned write support [iceberg-python]

2024-02-02 Thread via GitHub
jqin61 commented on code in PR #353: URL: https://github.com/apache/iceberg-python/pull/353#discussion_r1476487793 ## pyiceberg/table/__init__.py: ## @@ -2467,3 +2462,131 @@ def commit(self) -> Snapshot: ) return snapshot + + +@dataclass(frozen=True) +cla

Re: [PR] partitioned write support [iceberg-python]

2024-02-02 Thread via GitHub
syun64 commented on code in PR #353: URL: https://github.com/apache/iceberg-python/pull/353#discussion_r1476291241 ## pyiceberg/table/__init__.py: ## @@ -2467,3 +2462,131 @@ def commit(self) -> Snapshot: ) return snapshot + + +@dataclass(frozen=True) +cla