Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1424. Add getters to PartialRow ......................................................................
Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/6554/1/java/kudu-client/src/main/java/org/apache/kudu/client/PartialRow.java File java/kudu-client/src/main/java/org/apache/kudu/client/PartialRow.java: PS1, Line 149: A nit: here and everywhere below, start with a lower case. http://gerrit.cloudera.org:8080/#/c/6554/1/java/kudu-client/src/test/java/org/apache/kudu/client/TestPartialRow.java File java/kudu-client/src/test/java/org/apache/kudu/client/TestPartialRow.java: Line 82: // This is the expected exception nit: Here and elsewhere, inline comments should end with a period. -- To view, visit http://gerrit.cloudera.org:8080/6554 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I7c751eda9e8d6da5dd6ddd2ec798259bc037fb7d Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Grant Henke <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-HasComments: Yes
