You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Abhishek Chennaka (Code Review)" <ge...@cloudera.org> on 2024/07/26 06:01:04 UTC

[kudu-CR] [WIP] KUDU-1261 C++ client complex type introduction

Hello Alexey Serbin, Kudu Jenkins, 

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

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

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

Change subject: [WIP] KUDU-1261 C++ client complex type introduction
......................................................................

[WIP] KUDU-1261 C++ client complex type introduction

This patch has a high level introduction idea of array datatype. There
are still things to be addressed with a bigger block being how to deal
with nested types.

The highlevel idea is to introduce a new class for each complex type
which is then used as a part of the schema. The exposure to the client
side is present in client-test.cc on which the feedback is requested.

Change-Id: I93b781aeff04a3edfe923a8339b84699ba8095e2
---
M src/kudu/client/client-test.cc
M src/kudu/client/schema-internal.h
M src/kudu/client/schema.cc
M src/kudu/client/schema.h
M src/kudu/common/schema.h
5 files changed, 47 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/74/20974/3
-- 
To view, visit http://gerrit.cloudera.org:8080/20974
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I93b781aeff04a3edfe923a8339b84699ba8095e2
Gerrit-Change-Number: 20974
Gerrit-PatchSet: 3
Gerrit-Owner: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)