Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/23486 )
Change subject: KUDU-1261: move Serialize into kudu::serdes namespace ...................................................................... KUDU-1261: move Serialize into kudu::serdes namespace This patch moves all the serialization methods from array_type_serdes.h into the 'kudu::serdes' namespace. They were in the 'kudu' namespace before. Also, there is an update in ArrayDataGenerator::Build() to reflect on various representations of 'empty' Slice instances that might be used as serialized form of an empty or null array cell. The code in ArrayCellMetadataView::Init() has been updated accordingly. Change-Id: Ic0fe86c2ff7a9ed0839ac3d9008295d3901b209f Reviewed-on: http://gerrit.cloudera.org:8080/23486 Reviewed-by: Abhishek Chennaka <[email protected]> Tested-by: Alexey Serbin <[email protected]> --- M src/kudu/cfile/cfile-test-base.h M src/kudu/cfile/cfile_reader.cc M src/kudu/common/array_cell_view.h M src/kudu/common/array_type_serdes-test.cc M src/kudu/common/array_type_serdes.h M src/kudu/common/partial_row.cc M src/kudu/common/types-test.cc 7 files changed, 25 insertions(+), 12 deletions(-) Approvals: Abhishek Chennaka: Looks good to me, approved Alexey Serbin: Verified -- To view, visit http://gerrit.cloudera.org:8080/23486 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ic0fe86c2ff7a9ed0839ac3d9008295d3901b209f Gerrit-Change-Number: 23486 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Riza Suminto <[email protected]>
