liukun4515 commented on code in PR #1855: URL: https://github.com/apache/arrow-rs/pull/1855#discussion_r903704042
########## arrow/Cargo.toml: ########## @@ -60,7 +63,7 @@ multiversion = { version = "0.6.1", default-features = false } bitflags = { version = "1.2.1", default-features = false } [features] -default = ["csv", "ipc", "test_utils"] +default = ["csv", "ipc", "test_utils", "zstd", "lz4"] Review Comment: Yes, include the dependency by default. buffer compression protocol is common protocol for all languages in the arrow ecosystem, if we use the compression/decompression as optional, we can read the file or stream from compressed side. 2.0.0 compression has been implemented in the Java, C++ and other languages, If your rust server receiver a message compressed by the protocol, we can read them by default. -- 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...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org