You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by jd...@apache.org on 2015/01/19 21:04:49 UTC
svn commit: r1653099 - in
/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic:
GenericUDFLower.java GenericUDFUpper.java
Author: jdere
Date: Mon Jan 19 20:04:49 2015
New Revision: 1653099
URL: http://svn.apache.org/r1653099
Log:
HIVE-9369: fix arguments length checking in Upper and Lower UDF (Alexander Pivovarov via Jason Dere)
Modified:
hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFLower.java
hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFUpper.java
Modified: hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFLower.java
URL: http://svn.apache.org/viewvc/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFLower.java?rev=1653099&r1=1653098&r2=1653099&view=diff
==============================================================================
--- hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFLower.java (original)
+++ hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFLower.java Mon Jan 19 20:04:49 2015
@@ -51,7 +51,7 @@ public class GenericUDFLower extends Gen
@Override
public ObjectInspector initialize(ObjectInspector[] arguments) throws UDFArgumentException {
- if (arguments.length < 0) {
+ if (arguments.length != 1) {
throw new UDFArgumentLengthException(
"LOWER requires 1 argument, got " + arguments.length);
}
Modified: hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFUpper.java
URL: http://svn.apache.org/viewvc/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFUpper.java?rev=1653099&r1=1653098&r2=1653099&view=diff
==============================================================================
--- hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFUpper.java (original)
+++ hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFUpper.java Mon Jan 19 20:04:49 2015
@@ -33,7 +33,6 @@ import org.apache.hadoop.hive.serde2.obj
import org.apache.hadoop.hive.serde2.objectinspector.primitive.PrimitiveObjectInspectorFactory;
import org.apache.hadoop.hive.serde2.typeinfo.BaseCharTypeInfo;
import org.apache.hadoop.hive.serde2.typeinfo.TypeInfoFactory;
-import org.apache.hadoop.hive.serde2.typeinfo.VarcharTypeInfo;
/**
* UDFUpper.
@@ -52,7 +51,7 @@ public class GenericUDFUpper extends Gen
@Override
public ObjectInspector initialize(ObjectInspector[] arguments) throws UDFArgumentException {
- if (arguments.length < 0) {
+ if (arguments.length != 1) {
throw new UDFArgumentLengthException(
"UPPER requires 1 argument, got " + arguments.length);
}