Hello Brian McDevitt, Kudu Jenkins, Adar Dembo, Grant Henke, Todd Lipcon,

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

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

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

Change subject: KUDU-2689: Made PartialRow setters use a fluent-style.
......................................................................

KUDU-2689: Made PartialRow setters use a fluent-style.

-Changed PartialRow return types from void to PartialRow so that the method 
calls can be chained.
-Changed '@return a PartialRow' to '@return this PartialRow' to clearly 
indicate that it's the current row that is being mutated rather than a new 
PartialRow.
-Removed redundant indentation.

Change-Id: I3476fb3c0a28812e2732e6c6268f31a393928206
---
M java/kudu-client/src/main/java/org/apache/kudu/client/PartialRow.java
M java/kudu-client/src/test/java/org/apache/kudu/client/TestPartialRow.java
2 files changed, 132 insertions(+), 86 deletions(-)


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I3476fb3c0a28812e2732e6c6268f31a393928206
Gerrit-Change-Number: 12949
Gerrit-PatchSet: 2
Gerrit-Owner: Anonymous Coward <raym...@phdata.io>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Brian McDevitt <br...@phdata.io>
Gerrit-Reviewer: Grant Henke <granthe...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>

Reply via email to