Hello Yida Wu, Joe McDonnell, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/20181
to look at the new patch set (#6).
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
---
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(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/81/20181/6
--
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: newpatchset
Gerrit-Change-Id: Id5ec1d0345ae35597f6aade9d8b9eef2257efeba
Gerrit-Change-Number: 20181
Gerrit-PatchSet: 6
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]>
