Hello Xuebin Su, Kudu Jenkins, Abhishek Chennaka,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/22868
to look at the new patch set (#15).
Change subject: WIP KUDU-1261 writing/reading of array data blocks
......................................................................
WIP KUDU-1261 writing/reading of array data blocks
Wit this patch, writing/reading of array data blocks works for arrays
of all the existing scalar types except for INT128 and DECIMAL128.
WIP:
* write proper description
* brush up the patch
* add exportable flavor of array_cell_view.h for Kudu C++ client API
* address TODOs
* resolve issue with DICTIONARY encoding when going a bit over
the configured block size while
Change-Id: I5825c939cb40d350a23a78609115f26e62cca270
---
M src/kudu/cfile/binary_dict_block.cc
M src/kudu/cfile/binary_dict_block.h
M src/kudu/cfile/binary_plain_block.cc
M src/kudu/cfile/binary_plain_block.h
M src/kudu/cfile/binary_prefix_block.cc
M src/kudu/cfile/binary_prefix_block.h
M src/kudu/cfile/block_encodings.h
M src/kudu/cfile/bshuf_block.h
M src/kudu/cfile/cfile-test-base.h
M src/kudu/cfile/cfile-test.cc
M src/kudu/cfile/cfile_reader.cc
M src/kudu/cfile/cfile_reader.h
M src/kudu/cfile/cfile_util.h
M src/kudu/cfile/cfile_writer.cc
M src/kudu/cfile/cfile_writer.h
M src/kudu/cfile/plain_bitmap_block.h
M src/kudu/cfile/plain_block.h
M src/kudu/cfile/rle_block.h
A src/kudu/common/array_cell_view.h
A src/kudu/common/array_type_serdes.h
M src/kudu/common/columnblock-test-util.h
M src/kudu/common/columnblock.h
M src/kudu/common/rowblock.h
M src/kudu/common/rowblock_memory.h
M src/kudu/tablet/multi_column_writer.cc
M src/kudu/util/bitmap.cc
M src/kudu/util/bitmap.h
27 files changed, 2,088 insertions(+), 204 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/68/22868/15
--
To view, visit http://gerrit.cloudera.org:8080/22868
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I5825c939cb40d350a23a78609115f26e62cca270
Gerrit-Change-Number: 22868
Gerrit-PatchSet: 15
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Xuebin Su <[email protected]>
