Re: [PR] fix: correct LZ0 to LZO in compression options [datafusion-python]

2025-01-14 Thread via GitHub
timsaucer merged PR #995: URL: https://github.com/apache/datafusion-python/pull/995 -- 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: github-unsubscr...@d

Re: [PR] fix: correct LZ0 to LZO in compression options [datafusion-python]

2025-01-13 Thread via GitHub
kosiew commented on PR #995: URL: https://github.com/apache/datafusion-python/pull/995#issuecomment-2588817592 Encountered weird CI build error ``` Run ruff check --output-format=github python/ Would reformat: python/tests/test_functions.py 1 file would be reformatted, 35 files a

Re: [PR] fix: correct LZ0 to LZO in compression options [datafusion-python]

2025-01-13 Thread via GitHub
kosiew commented on code in PR #995: URL: https://github.com/apache/datafusion-python/pull/995#discussion_r1914165106 ## python/tests/test_dataframe.py: ## @@ -1115,6 +1115,9 @@ def test_write_compressed_parquet_invalid_compression(df, tmp_path, compression) df.write_p

Re: [PR] fix: correct LZ0 to LZO in compression options [datafusion-python]

2025-01-13 Thread via GitHub
kylebarron commented on code in PR #995: URL: https://github.com/apache/datafusion-python/pull/995#discussion_r1913388654 ## python/tests/test_dataframe.py: ## @@ -1115,6 +1115,9 @@ def test_write_compressed_parquet_invalid_compression(df, tmp_path, compression) df.wri

Re: [PR] fix: correct LZ0 to LZO in compression options [datafusion-python]

2025-01-12 Thread via GitHub
kosiew commented on PR #995: URL: https://github.com/apache/datafusion-python/pull/995#issuecomment-2586418645 Encountered weird CI build error ``` Run ruff check --output-format=github python/ Would reformat: python/tests/test_functions.py 1 file would be reformatted, 35 fil

[PR] fix: correct LZ0 to LZO in compression options [datafusion-python]

2025-01-12 Thread via GitHub
kosiew opened a new pull request, #995: URL: https://github.com/apache/datafusion-python/pull/995 # Which issue does this PR close? Closes #979. # Rationale for this change Correct confusion of lz0 for lzo # What changes are included in this PR?