-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15654/
-----------------------------------------------------------
Review request for hive, Ashutosh Chauhan, Carl Steinbach, and Jitendra Pandey.
Bugs: HIVE-5829
https://issues.apache.org/jira/browse/HIVE-5829
Repository: hive-git
Description
-------
Rewrite the UDFS *pads and *trim using GenericUDF.
Diffs
-----
ql/src/java/org/apache/hadoop/hive/ql/exec/FunctionRegistry.java 5eb321c
ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/Vectorizer.java
7c1ab0d
ql/src/java/org/apache/hadoop/hive/ql/udf/UDFLTrim.java dc00cf9
ql/src/java/org/apache/hadoop/hive/ql/udf/UDFLpad.java d1da19a
ql/src/java/org/apache/hadoop/hive/ql/udf/UDFRTrim.java 2bcc5fa
ql/src/java/org/apache/hadoop/hive/ql/udf/UDFRpad.java 9652ce2
ql/src/java/org/apache/hadoop/hive/ql/udf/UDFTrim.java 490886d
ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFBasePad.java
PRE-CREATION
ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFBaseTrim.java
PRE-CREATION
ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFLTrim.java
PRE-CREATION
ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFLpad.java
PRE-CREATION
ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFRTrim.java
PRE-CREATION
ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFRpad.java
PRE-CREATION
ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFTrim.java
PRE-CREATION
ql/src/test/org/apache/hadoop/hive/ql/exec/vector/TestVectorizationContext.java
3f3e67f
ql/src/test/org/apache/hadoop/hive/ql/udf/TestGenericUDFLTrim.java
PRE-CREATION
ql/src/test/org/apache/hadoop/hive/ql/udf/TestGenericUDFLpad.java
PRE-CREATION
ql/src/test/org/apache/hadoop/hive/ql/udf/TestGenericUDFRTrim.java
PRE-CREATION
ql/src/test/org/apache/hadoop/hive/ql/udf/TestGenericUDFRpad.java
PRE-CREATION
ql/src/test/org/apache/hadoop/hive/ql/udf/TestGenericUDFTrim.java
PRE-CREATION
Diff: https://reviews.apache.org/r/15654/diff/
Testing
-------
Thanks,
Mohammad Islam
