Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/23145
to look at the new patch set (#6).
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
---
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/tools/kudu-tool-test.cc
5 files changed, 117 insertions(+), 17 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/45/23145/6
--
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: newpatchset
Gerrit-Change-Id: Ie84debb6dd8f81ce013cb3330b07c6abe2534c0b
Gerrit-Change-Number: 23145
Gerrit-PatchSet: 6
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
