Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14736 )
Change subject: [Java] Fixed Type.getTypeForName method. ...................................................................... [Java] Fixed Type.getTypeForName method. Original version uses only name() method to check Type name. Output from name() (inerrithed from Enum) method was different from output from Type.getName() method (ie INT32 vs int32), so the check fails. Now the method uses both getName() and name() to check Type name. Change-Id: Ibd0f4f614125d630d128c36fce05f22e19c60100 Reviewed-on: http://gerrit.cloudera.org:8080/14736 Tested-by: Kudu Jenkins Reviewed-by: Grant Henke <[email protected]> --- M java/kudu-client/src/main/java/org/apache/kudu/Type.java A java/kudu-client/src/test/java/org/apache/kudu/TestType.java 2 files changed, 53 insertions(+), 1 deletion(-) Approvals: Kudu Jenkins: Verified Grant Henke: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/14736 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ibd0f4f614125d630d128c36fce05f22e19c60100 Gerrit-Change-Number: 14736 Gerrit-PatchSet: 8 Gerrit-Owner: Michele Milesi <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Michele Milesi <[email protected]>
