Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/24059 )
Change subject: IMPALA-14700: Add support for Parquet LZ4_RAW compression ...................................................................... Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/24059/4/be/src/exec/parquet/parquet-common.cc File be/src/exec/parquet/parquet-common.cc: http://gerrit.cloudera.org:8080/#/c/24059/4/be/src/exec/parquet/parquet-common.cc@48 PS4, Line 48: parquet::CompressionCodec::LZ4_RAW, // LZ4 This changes the write codepath so that compression_codec=lz4 is LZ4_RAW when it used to be regular LZ4. Did we intend to change the write side? Changing the write side creates some compatibility concerns about other components / older Impalas. -- To view, visit http://gerrit.cloudera.org:8080/24059 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I22ee4e5bf9abec37be941c1dca8019a563343d34 Gerrit-Change-Number: 24059 Gerrit-PatchSet: 4 Gerrit-Owner: Balazs Hevele <[email protected]> Gerrit-Reviewer: Balazs Hevele <[email protected]> Gerrit-Reviewer: Daniel Vanko <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Surya Hebbar <[email protected]> Gerrit-Comment-Date: Sat, 21 Mar 2026 01:21:53 +0000 Gerrit-HasComments: Yes
