Alexey Serbin has submitted this change and it was merged. (
http://gerrit.cloudera.org:8080/23145 )
Change subject: KUDU-1261 update CFile {en,de}coders for array data type
......................................................................
KUDU-1261 update CFile {en,de}coders for array data type
This changelist introduces a new 'CFileFooterPB.is_type_array' field
to distinguish CFiles containing array data blocks [1]. When writing
a CFile with array data blocks, CFile writer implementations must also
set the corresponding bit in the 'CFileFooterPB.incompatible_features'
field (the new bit will be introduced in a follow-up changelist)
along with setting the new 'is_type_array' field to 'true'.
This changelist also updates the encoder/decoder registry logic
to register and look up encoders/decoders for array data blocks.
[1] https://gerrit.cloudera.org/#/c/22058
Change-Id: Ie84debb6dd8f81ce013cb3330b07c6abe2534c0b
Reviewed-on: http://gerrit.cloudera.org:8080/23145
Tested-by: Alexey Serbin <[email protected]>
Reviewed-by: Abhishek Chennaka <[email protected]>
---
M src/kudu/cfile/cfile.proto
M src/kudu/cfile/cfile_reader.cc
M src/kudu/cfile/encoding-test.cc
M src/kudu/cfile/type_encodings.cc
M src/kudu/common/types.cc
M src/kudu/common/types.h
6 files changed, 120 insertions(+), 9 deletions(-)
Approvals:
Alexey Serbin: Verified
Abhishek Chennaka: Looks good to me, approved
--
To view, visit http://gerrit.cloudera.org:8080/23145
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ie84debb6dd8f81ce013cb3330b07c6abe2534c0b
Gerrit-Change-Number: 23145
Gerrit-PatchSet: 11
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
