[kudu-CR](branch-1.5.x) KUDU-2416: Fix PartialRow.setMin and add a unit test
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10190 ) Change subject: KUDU-2416: Fix PartialRow.setMin and add a unit test .. KUDU-2416: Fix PartialRow.setMin and add a unit test * Fixes INTEGER.MIN_VALUE being used for LONG colums that could lead to incorrect partition prunning. * Adds a unit test for all types. Change-Id: I510743385cc567a8443efc8b0c9f5b641cf04c31 Reviewed-on: http://gerrit.cloudera.org:8080/10190 Reviewed-by: Dan BurkertTested-by: Grant Henke --- 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, 20 insertions(+), 1 deletion(-) Approvals: Dan Burkert: Looks good to me, approved Grant Henke: Verified -- To view, visit http://gerrit.cloudera.org:8080/10190 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.5.x Gerrit-MessageType: merged Gerrit-Change-Id: I510743385cc567a8443efc8b0c9f5b641cf04c31 Gerrit-Change-Number: 10190 Gerrit-PatchSet: 2 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins
[kudu-CR](branch-1.5.x) KUDU-2416: Fix PartialRow.setMin and add a unit test
Grant Henke has removed a vote on this change. Change subject: KUDU-2416: Fix PartialRow.setMin and add a unit test .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/10190 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.5.x Gerrit-MessageType: deleteVote Gerrit-Change-Id: I510743385cc567a8443efc8b0c9f5b641cf04c31 Gerrit-Change-Number: 10190 Gerrit-PatchSet: 1 Gerrit-Owner: Grant HenkeGerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Kudu Jenkins
[kudu-CR](branch-1.5.x) KUDU-2416: Fix PartialRow.setMin and add a unit test
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/10190 ) Change subject: KUDU-2416: Fix PartialRow.setMin and add a unit test .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/10190 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.5.x Gerrit-MessageType: comment Gerrit-Change-Id: I510743385cc567a8443efc8b0c9f5b641cf04c31 Gerrit-Change-Number: 10190 Gerrit-PatchSet: 1 Gerrit-Owner: Grant HenkeGerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins Gerrit-Comment-Date: Thu, 26 Apr 2018 01:43:29 + Gerrit-HasComments: No
[kudu-CR](branch-1.5.x) KUDU-2416: Fix PartialRow.setMin and add a unit test
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/10190 ) Change subject: KUDU-2416: Fix PartialRow.setMin and add a unit test .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/10190 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.5.x Gerrit-MessageType: comment Gerrit-Change-Id: I510743385cc567a8443efc8b0c9f5b641cf04c31 Gerrit-Change-Number: 10190 Gerrit-PatchSet: 1 Gerrit-Owner: Grant HenkeGerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Kudu Jenkins Gerrit-Comment-Date: Wed, 25 Apr 2018 19:47:11 + Gerrit-HasComments: No
[kudu-CR](branch-1.5.x) KUDU-2416: Fix PartialRow.setMin and add a unit test
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10190 Change subject: KUDU-2416: Fix PartialRow.setMin and add a unit test .. KUDU-2416: Fix PartialRow.setMin and add a unit test * Fixes INTEGER.MIN_VALUE being used for LONG colums that could lead to incorrect partition prunning. * Adds a unit test for all types. Change-Id: I510743385cc567a8443efc8b0c9f5b641cf04c31 --- 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, 20 insertions(+), 1 deletion(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/90/10190/1 -- To view, visit http://gerrit.cloudera.org:8080/10190 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.5.x Gerrit-MessageType: newchange Gerrit-Change-Id: I510743385cc567a8443efc8b0c9f5b641cf04c31 Gerrit-Change-Number: 10190 Gerrit-PatchSet: 1 Gerrit-Owner: Grant Henke