Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10185 )
Change subject: KUDU-2416: Fix PartialRow.setMin and add a unit test ...................................................................... KUDU-2416: Fix PartialRow.setMin and add a unit test * Fixes a missing break after the decimal type that could lead to IllegalArgumentExceptions. * 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/10185 Reviewed-by: Todd Lipcon <t...@apache.org> Reviewed-by: Dan Burkert <danburk...@apache.org> Tested-by: Grant Henke <granthe...@apache.org> --- 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, 22 insertions(+), 1 deletion(-) Approvals: Todd Lipcon: Looks good to me, approved Dan Burkert: Looks good to me, approved Grant Henke: Verified -- To view, visit http://gerrit.cloudera.org:8080/10185 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I510743385cc567a8443efc8b0c9f5b641cf04c31 Gerrit-Change-Number: 10185 Gerrit-PatchSet: 2 Gerrit-Owner: Grant Henke <granthe...@apache.org> Gerrit-Reviewer: Dan Burkert <danburk...@apache.org> Gerrit-Reviewer: Grant Henke <granthe...@apache.org> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
