Re: [PR] Fix support for writing to nested field partition [iceberg-python]

2025-07-15 Thread via GitHub
kevinjqliu merged PR #2204: URL: https://github.com/apache/iceberg-python/pull/2204 -- 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] Fix support for writing to nested field partition [iceberg-python]

2025-07-14 Thread via GitHub
geruh commented on code in PR #2204: URL: https://github.com/apache/iceberg-python/pull/2204#discussion_r2206242808 ## tests/io/test_pyarrow.py: ## @@ -2350,6 +2350,72 @@ def test_partition_for_demo() -> None: ) +def test_partition_for_nested_field() -> None: +schem

Re: [PR] Fix support for writing to nested field partition [iceberg-python]

2025-07-14 Thread via GitHub
kevinjqliu commented on code in PR #2204: URL: https://github.com/apache/iceberg-python/pull/2204#discussion_r2206215236 ## pyiceberg/io/pyarrow.py: ## @@ -2765,3 +2767,22 @@ def _determine_partitions(spec: PartitionSpec, schema: Schema, arrow_table: pa.T ) retu

Re: [PR] Fix support for writing to nested field partition [iceberg-python]

2025-07-14 Thread via GitHub
kevinjqliu commented on code in PR #2204: URL: https://github.com/apache/iceberg-python/pull/2204#discussion_r2206212298 ## pyiceberg/io/pyarrow.py: ## @@ -2765,3 +2767,22 @@ def _determine_partitions(spec: PartitionSpec, schema: Schema, arrow_table: pa.T ) retu