Hello Kudu Jenkins, Abhishek Chennaka,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/23001

to look at the new patch set (#7).

Change subject: KUDU-1261 support 1D arrays in TypeInfo
......................................................................

KUDU-1261 support 1D arrays in TypeInfo

This changelist updates TypeInfo to support one-dimensional arrays
of Kudu scalar types.  With this patch, the run-time registry of data
types is now populated with necessary TypeInfo objects for 1D arrays
for every registered Kudu scalar data type.

This changelist also adds basic tests scenarios to verify the contents
of the TypeInfo registry once the necessary array types have been added.

The following methods aren't implemented yet as a part of this changelist
and will be implemented in a follow-up changelist after the introduction
of the ArrayCellMetadataView class on top of [1]:
  * ArrayTypeTraits::Compare()
  * ArrayTypeTraits::AreConsecutive()
  * ArrayTypeTraits::AppendDebugStringForValue()

[1] https://gerrit.cloudera.org/#/c/23002/

Change-Id: I4215a86c1989d5bc37a6980312fa7fc82f136d60
---
M src/kudu/common/types-test.cc
M src/kudu/common/types.cc
M src/kudu/common/types.h
3 files changed, 376 insertions(+), 13 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/01/23001/7
--
To view, visit http://gerrit.cloudera.org:8080/23001
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I4215a86c1989d5bc37a6980312fa7fc82f136d60
Gerrit-Change-Number: 23001
Gerrit-PatchSet: 7
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)

Reply via email to