Michael Smith has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20181 )
Change subject: IMPALA-12275: Read files written with DeflateCodec ...................................................................... IMPALA-12275: Read files written with DeflateCodec DeflateCodec is an alias to DefaultCodec. Impala works with DefaultCodec. Fixes reading files written with DeflateCodec. DeflateCodec isn't an issue with text files because they don't include a codec header. Sequence files do, which we check on decompress. Moves TestTextInterop to a E2E test since it doesn't require any special startup options and refactors out test running to be format-agnostic. Updates text file test as IMPALA-8721 is fixed. Removes creating a table in Impala for Hive to read, as it didn't test anything new. Adds tests for sequence files; excludes reading zstd due to IMPALA-12276. Testing: - manual exhaustive run of updated tests Change-Id: Id5ec1d0345ae35597f6aade9d8b9eef2257efeba Reviewed-on: http://gerrit.cloudera.org:8080/20181 Reviewed-by: Joe McDonnell <[email protected]> Tested-by: Michael Smith <[email protected]> --- M be/src/util/codec.cc M be/src/util/codec.h D tests/custom_cluster/test_hive_text_codec_interop.py A tests/query_test/test_hive_codec_interop.py 4 files changed, 126 insertions(+), 115 deletions(-) Approvals: Joe McDonnell: Looks good to me, approved Michael Smith: Verified -- To view, visit http://gerrit.cloudera.org:8080/20181 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Id5ec1d0345ae35597f6aade9d8b9eef2257efeba Gerrit-Change-Number: 20181 Gerrit-PatchSet: 7 Gerrit-Owner: Michael Smith <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Yida Wu <[email protected]>
