Hello Kudu Jenkins, Abhishek Chennaka,

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

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

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

Change subject: KUDU-1261 instantiate SetArray templates in C++ client library
......................................................................

KUDU-1261 instantiate SetArray templates in C++ client library

Since Kudu C++ client API still officially supports C++98, it's
necessary to have template instantiations for KuduPartialRow::SetArray,
KuduPartialRow::Set used in non-array setters.  Otherwise, there might
be linker errors: see [1] for details.

This is a follow-up to [2].

[1] https://isocpp.org/wiki/faq/templates#separate-template-fn-defn-from-decl
[2] https://github.com/apache/kudu/commit/5db5af1de

Change-Id: I26b923c57213ca3bb6ae77fab4b8ca0e45c8e0e3
---
M src/kudu/common/partial_row.cc
1 file changed, 140 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/84/23484/2
--
To view, visit http://gerrit.cloudera.org:8080/23484
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

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

Reply via email to