Re: [PR] Remove deprecation warnings in test [iceberg-python]

2025-01-03 Thread via GitHub
Fokko merged PR #1416: URL: https://github.com/apache/iceberg-python/pull/1416 -- 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...@iceber

Re: [PR] Remove deprecation warnings in test [iceberg-python]

2025-01-03 Thread via GitHub
Fokko commented on code in PR #1416: URL: https://github.com/apache/iceberg-python/pull/1416#discussion_r1902135055 ## tests/expressions/test_parser.py: ## @@ -70,7 +70,6 @@ def test_equals_false() -> None: def test_is_null() -> None: assert IsNull("foo") == parser.parse("

Re: [PR] Remove deprecation warnings in test [iceberg-python]

2024-12-20 Thread via GitHub
kevinjqliu commented on code in PR #1416: URL: https://github.com/apache/iceberg-python/pull/1416#discussion_r1894101745 ## tests/expressions/test_parser.py: ## @@ -70,7 +70,6 @@ def test_equals_false() -> None: def test_is_null() -> None: assert IsNull("foo") == parser.pa

Re: [PR] Remove deprecation warnings in test [iceberg-python]

2024-12-20 Thread via GitHub
Fokko commented on code in PR #1416: URL: https://github.com/apache/iceberg-python/pull/1416#discussion_r1894094791 ## tests/expressions/test_parser.py: ## @@ -70,7 +70,6 @@ def test_equals_false() -> None: def test_is_null() -> None: assert IsNull("foo") == parser.parse("

Re: [PR] Remove deprecation warnings in test [iceberg-python]

2024-12-20 Thread via GitHub
kevinjqliu commented on PR #1416: URL: https://github.com/apache/iceberg-python/pull/1416#issuecomment-2557233390 done! @Fokko can you take another look? ran locally no warnings in `make-test` -- This is an automated message from the Apache Git Service. To respond to the message, please l

Re: [PR] Remove deprecation warnings in test [iceberg-python]

2024-12-20 Thread via GitHub
Fokko commented on PR #1416: URL: https://github.com/apache/iceberg-python/pull/1416#issuecomment-2557111091 @kevinjqliu can you rebase this one? :) -- 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 t

Re: [PR] Remove deprecation warnings in test [iceberg-python]

2024-12-07 Thread via GitHub
kevinjqliu commented on code in PR #1416: URL: https://github.com/apache/iceberg-python/pull/1416#discussion_r1874539222 ## pyiceberg/utils/deprecated.py: ## @@ -56,7 +56,6 @@ def deprecation_message(deprecated_in: str, removed_in: str, help_message: Optio def _deprecation_w

[PR] Remove deprecation warnings in test [iceberg-python]

2024-12-07 Thread via GitHub
kevinjqliu opened a new pull request, #1416: URL: https://github.com/apache/iceberg-python/pull/1416 Fix these errors in `make-test`: * tests/table/test_name_mapping.py::test_mapping_lookup_by_name `Call to find, deprecated in 0.8.0, will be removed in 0.9.0. Please use `apply_name_map