Skip to content

Commit 0a00bd3

Browse files
committed
fix: disable LZO compression option and update tests to reflect its unavailability
1 parent 5cb1c0e commit 0a00bd3

File tree

2 files changed

+7
-1
lines changed

2 files changed

+7
-1
lines changed

python/datafusion/dataframe.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,10 @@ class Compression(Enum):
5757
GZIP = "gzip"
5858
BROTLI = "brotli"
5959
LZ4 = "lz4"
60-
LZO = "lzo"
60+
# TODO
61+
# lzo is not implemented yet
62+
# https://github.com/apache/arrow-rs/issues/6970
63+
# LZO = "lzo"
6164
ZSTD = "zstd"
6265
LZ4_RAW = "lz4_raw"
6366

python/tests/test_dataframe.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1113,6 +1113,9 @@ def test_write_compressed_parquet_invalid_compression(df, tmp_path, compression)
11131113
df.write_parquet(str(path), compression=compression)
11141114

11151115

1116+
# TODO
1117+
# not testing lzo because it it not implemented yet
1118+
# https://github.com/apache/arrow-rs/issues/6970
11161119
@pytest.mark.parametrize("compression", ["zstd", "brotli", "gzip"])
11171120
def test_write_compressed_parquet_default_compression_level(df, tmp_path, compression):
11181121
# Test write_parquet with zstd, brotli, gzip default compression level,

0 commit comments

Comments
 (0)