[kudu-CR] KUDU-1424. Add getters to PartialRow
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: KUDU-1424. Add getters to PartialRow .. KUDU-1424. Add getters to PartialRow - Adds getters for each type to PartialRow - Adds public isNull and isSet methods - Adds unit tests for PartialRow - Fixes javadoc in PartialRow and RowResult Change-Id: I7c751eda9e8d6da5dd6ddd2ec798259bc037fb7d Reviewed-on: http://gerrit.cloudera.org:8080/6554 Tested-by: Kudu Jenkins Reviewed-by: Jean-Daniel Cryans--- M java/kudu-client/src/main/java/org/apache/kudu/client/PartialRow.java M java/kudu-client/src/main/java/org/apache/kudu/client/RowResult.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestPartialRow.java 3 files changed, 713 insertions(+), 89 deletions(-) Approvals: Jean-Daniel Cryans: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/6554 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7c751eda9e8d6da5dd6ddd2ec798259bc037fb7d Gerrit-PatchSet: 3 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Grant Henke Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon
[kudu-CR] KUDU-1424. Add getters to PartialRow
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1424. Add getters to PartialRow .. Patch Set 2: Code-Review+2 -- 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: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Grant HenkeGerrit-Reviewer: Grant Henke Gerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon Gerrit-HasComments: No
[kudu-CR] KUDU-1424. Add getters to PartialRow
Grant Henke 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: Line 149:* @return A boolean > nit: here and everywhere below, start with a lower case. Done 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. Done -- 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 HenkeGerrit-Reviewer: Grant Henke Gerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon Gerrit-HasComments: Yes
[kudu-CR] KUDU-1424. Add getters to PartialRow
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 HenkeGerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon Gerrit-HasComments: Yes