Re: [PR] fix: support MonthTransform for partitioning [iceberg-python]

2024-09-16 Thread via GitHub
kevinjqliu merged PR #1176: URL: https://github.com/apache/iceberg-python/pull/1176 -- 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 MonthTransform for partitioning [iceberg-python]

2024-09-14 Thread via GitHub
felixscherz commented on code in PR #1176: URL: https://github.com/apache/iceberg-python/pull/1176#discussion_r1759784424 ## tests/integration/test_partition_evolution.py: ## @@ -152,6 +201,36 @@ def test_multiple_adds(catalog: Catalog) -> None: ) +@pytest.mark.integrat

Re: [PR] fix: support MonthTransform for partitioning [iceberg-python]

2024-09-14 Thread via GitHub
kevinjqliu commented on code in PR #1176: URL: https://github.com/apache/iceberg-python/pull/1176#discussion_r1759766338 ## tests/integration/test_partition_evolution.py: ## @@ -152,6 +201,36 @@ def test_multiple_adds(catalog: Catalog) -> None: ) +@pytest.mark.integrati

Re: [PR] fix: support MonthTransform for partitioning [iceberg-python]

2024-09-14 Thread via GitHub
felixscherz commented on code in PR #1176: URL: https://github.com/apache/iceberg-python/pull/1176#discussion_r1759758412 ## tests/integration/test_partition_evolution.py: ## @@ -100,6 +100,13 @@ def test_add_month(catalog: Catalog) -> None: _validate_new_partition_fields(t

Re: [PR] fix: support MonthTransform for partitioning [iceberg-python]

2024-09-14 Thread via GitHub
kevinjqliu commented on code in PR #1176: URL: https://github.com/apache/iceberg-python/pull/1176#discussion_r1759758077 ## tests/integration/test_partition_evolution.py: ## @@ -100,6 +100,13 @@ def test_add_month(catalog: Catalog) -> None: _validate_new_partition_fields(ta

[PR] fix: support MonthTransform for partitioning [iceberg-python]

2024-09-14 Thread via GitHub
felixscherz opened a new pull request, #1176: URL: https://github.com/apache/iceberg-python/pull/1176 Hi, this is fixing a minor bug where partition evolution in combination with a `MonthTransform` could raise an exception: https://github.com/apache/iceberg-python/issues/1156 --