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 (#7).

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
4 files changed, 109 insertions(+), 9 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/45/23145/7
--
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: 7
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)

Reply via email to