You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kudu.apache.org by "Mike Percy (Code Review)" <ge...@cloudera.org> on 2016/03/26 02:40:37 UTC

[kudu-CR] KUDU-839. Add Status class to Java client for use in RowError

Hello Jean-Daniel Cryans,

I'd like you to do a code review.  Please visit

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

to review the following change.

Change subject: KUDU-839. Add Status class to Java client for use in RowError
......................................................................

KUDU-839. Add Status class to Java client for use in RowError

This class is the Java equivalent of the C++ kudu::Status class and
shares all of the same functionality except for the Clone...() methods.
The thought there is that we would primarily construct these on the
server side, not the client side.

Change-Id: I58e3c949cf441b7dd798c5001aeed747457a8807
---
A java/kudu-client/src/main/java/org/kududb/Status.java
M java/kudu-client/src/main/java/org/kududb/client/RowError.java
M java/kudu-client/src/test/java/org/kududb/client/TestErrorCollector.java
A java/kudu-client/src/test/java/org/kududb/util/TestStatus.java
M src/kudu/common/wire_protocol.proto
M src/kudu/util/status.h
6 files changed, 413 insertions(+), 17 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/40/2640/1
-- 
To view, visit http://gerrit.cloudera.org:8080/2640
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I58e3c949cf441b7dd798c5001aeed747457a8807
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Jean-Daniel Cryans