[negezor]

Modern ClickHouse instances use ZSTD by default (network_compression_method = 'zstd'), which previously caused the client to crash with a protocol error. Now, it dynamically detects whether the incoming packet is LZ4 or ZSTD and decodes it on the fly. I also fixed some minor double-allocation errors in compress_data.

SET network_compression_method = 'zstd';

Run any heavy SELECT query. It should fetch data successfully instead of throwing ProtocolError / UnexpectedEof.

P.S. Ran into an unpleasant issue in production today, which is why I’m here :)
klickhouse::client: clickhouse client failed: Io(Custom { kind: UnexpectedEof, error: ProtocolError("unexpected compression algorithm identifier: '90', expected 82 (LZ4)") })

P.S.S. I think we should also remove the CompressionMethod requirement for 'None'. Let it automatically check all ClickHouse packet headers so it can determine on its own whether compression is needed or not

[negezor]

@Protryon The Cargo.toml specifies edition = "2024", which requires Rust version 1.85 or higher. However, the minimum versions set in the CI for both nightly and stable are currently 1.75.

[Protryon]

Thanks, I'll do a CI refresh and re-release today.