Re: [PR] Test: Add test to partition on field with a dot [iceberg-python]

2024-05-06 Thread via GitHub
HonahX merged PR #610: URL: https://github.com/apache/iceberg-python/pull/610 -- 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...@iceberg

Re: [PR] Test: Add test to partition on field with a dot [iceberg-python]

2024-04-25 Thread via GitHub
Fokko commented on code in PR #610: URL: https://github.com/apache/iceberg-python/pull/610#discussion_r1579050799 ## tests/integration/test_reads.py: ## @@ -473,6 +474,31 @@ def test_sanitize_character(catalog: Catalog) -> None: assert arrow_table.schema.names[0] == table_

Re: [PR] Test: Add test to partition on field with a dot [iceberg-python]

2024-04-16 Thread via GitHub
HonahX commented on code in PR #610: URL: https://github.com/apache/iceberg-python/pull/610#discussion_r1568179853 ## tests/integration/test_reads.py: ## @@ -473,6 +474,31 @@ def test_sanitize_character(catalog: Catalog) -> None: assert arrow_table.schema.names[0] == table

[PR] Test: Add test to partition on field with a dot [iceberg-python]

2024-04-16 Thread via GitHub
Fokko opened a new pull request, #610: URL: https://github.com/apache/iceberg-python/pull/610 This is not allowed in Avro, just checking if it works. -- 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