[GitHub] [arrow] wesm commented on a change in pull request #7789: PARQUET-1878: [C++] lz4 codec is not compatible with Hadoop Lz4Codec

2020-09-22 Thread GitBox
wesm commented on a change in pull request #7789: URL: https://github.com/apache/arrow/pull/7789#discussion_r492965387 ## File path: cpp/src/arrow/util/compression.h ## @@ -30,7 +30,18 @@ namespace arrow { struct Compression { /// \brief Compression algorithm - enum type

[GitHub] [arrow] wesm commented on a change in pull request #7789: PARQUET-1878: [C++] lz4 codec is not compatible with Hadoop Lz4Codec

2020-09-22 Thread GitBox
wesm commented on a change in pull request #7789: URL: https://github.com/apache/arrow/pull/7789#discussion_r492965387 ## File path: cpp/src/arrow/util/compression.h ## @@ -30,7 +30,18 @@ namespace arrow { struct Compression { /// \brief Compression algorithm - enum type

[GitHub] [arrow] wesm commented on a change in pull request #7789: PARQUET-1878: [C++] lz4 codec is not compatible with Hadoop Lz4Codec

2020-08-04 Thread GitBox
wesm commented on a change in pull request #7789: URL: https://github.com/apache/arrow/pull/7789#discussion_r465310364 ## File path: cpp/src/arrow/util/compression.cc ## @@ -131,7 +131,7 @@ Result> Codec::Create(Compression::type codec_type, if (compression_level_set) {

[GitHub] [arrow] wesm commented on a change in pull request #7789: PARQUET-1878: [C++] lz4 codec is not compatible with Hadoop Lz4Codec

2020-07-31 Thread GitBox
wesm commented on a change in pull request #7789: URL: https://github.com/apache/arrow/pull/7789#discussion_r463842568 ## File path: cpp/src/arrow/util/compression.cc ## @@ -131,7 +131,7 @@ Result> Codec::Create(Compression::type codec_type, if (compression_level_set) {