You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kudu.apache.org by "Jean-Daniel Cryans (Code Review)" <ge...@ip-10-146-233-104.ec2.internal> on 2016/02/02 20:45:42 UTC

[kudu-CR] KUDU-842. Implement Operation.toString() in Java client

Hello Dan Burkert, Kudu Jenkins,

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

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

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

Change subject: KUDU-842. Implement Operation.toString() in Java client
......................................................................

KUDU-842. Implement Operation.toString() in Java client

This patch adds a new method to stringify row keys and uses it in
the new Operation.toString().

This patch also cleans up our use of booleans in row keys, which
aren't supported.

Change-Id: Ia258bc9bd4140ae0085bebe51935b465dac43db9
---
M java/kudu-client/src/main/java/org/kududb/client/KeyEncoder.java
M java/kudu-client/src/main/java/org/kududb/client/Operation.java
M java/kudu-client/src/main/java/org/kududb/client/PartialRow.java
M java/kudu-client/src/test/java/org/kududb/client/TestKeyEncoding.java
M java/kudu-client/src/test/java/org/kududb/client/TestOperation.java
5 files changed, 113 insertions(+), 28 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/45/1945/4
-- 
To view, visit http://gerrit.cloudera.org:8080/1945
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ia258bc9bd4140ae0085bebe51935b465dac43db9
Gerrit-PatchSet: 4
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Jean-Daniel Cryans
Gerrit-Reviewer: Dan Burkert <da...@cloudera.com>
Gerrit-Reviewer: Jean-Daniel Cryans
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>