You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by ha...@apache.org on 2016/05/31 18:42:05 UTC

[1/2] hive git commit: HIVE-13196 : UDFLike: reduce Regex NFA sizes (Gopal V via Gunther Hagleitner)

Repository: hive
Updated Branches:
  refs/heads/branch-2.1 3fb299750 -> 74c81a892


HIVE-13196 : UDFLike: reduce Regex NFA sizes (Gopal V via Gunther Hagleitner)

Signed-off-by: Ashutosh Chauhan <ha...@apache.org>


Project: http://git-wip-us.apache.org/repos/asf/hive/repo
Commit: http://git-wip-us.apache.org/repos/asf/hive/commit/549fc26c
Tree: http://git-wip-us.apache.org/repos/asf/hive/tree/549fc26c
Diff: http://git-wip-us.apache.org/repos/asf/hive/diff/549fc26c

Branch: refs/heads/branch-2.1
Commit: 549fc26c933a36e4844f312d1c2cbfadfdd55537
Parents: 3fb2997
Author: Gopal V <go...@apache.org>
Authored: Tue May 31 11:06:44 2016 -0700
Committer: Ashutosh Chauhan <ha...@apache.org>
Committed: Tue May 31 11:41:46 2016 -0700

----------------------------------------------------------------------
 ql/src/java/org/apache/hadoop/hive/ql/udf/UDFLike.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hive/blob/549fc26c/ql/src/java/org/apache/hadoop/hive/ql/udf/UDFLike.java
----------------------------------------------------------------------
diff --git a/ql/src/java/org/apache/hadoop/hive/ql/udf/UDFLike.java b/ql/src/java/org/apache/hadoop/hive/ql/udf/UDFLike.java
index 7bcd36e..9ac9548 100755
--- a/ql/src/java/org/apache/hadoop/hive/ql/udf/UDFLike.java
+++ b/ql/src/java/org/apache/hadoop/hive/ql/udf/UDFLike.java
@@ -77,7 +77,7 @@ public class UDFLike extends UDF {
       if (n == '_') {
         sb.append(".");
       } else if (n == '%') {
-        sb.append(".*");
+        sb.append(".*?");
       } else {
         sb.append(Pattern.quote(Character.toString(n)));
       }


[2/2] hive git commit: HIVE-13859: mask() UDF not retaining day and month field values (Madhan Neethiraj via Ashutosh Chauhan)

Posted by ha...@apache.org.
HIVE-13859: mask() UDF not retaining day and month field values (Madhan Neethiraj via Ashutosh Chauhan)

Signed-off-by: Ashutosh Chauhan <ha...@apache.org>


Project: http://git-wip-us.apache.org/repos/asf/hive/repo
Commit: http://git-wip-us.apache.org/repos/asf/hive/commit/74c81a89
Tree: http://git-wip-us.apache.org/repos/asf/hive/tree/74c81a89
Diff: http://git-wip-us.apache.org/repos/asf/hive/diff/74c81a89

Branch: refs/heads/branch-2.1
Commit: 74c81a892f9e16ff0543c5284f46d99c22c55ddb
Parents: 549fc26
Author: Madhan Neethiraj <ma...@apache.org>
Authored: Sat May 28 03:16:38 2016 -0700
Committer: Ashutosh Chauhan <ha...@apache.org>
Committed: Tue May 31 11:41:53 2016 -0700

----------------------------------------------------------------------
 .../hive/ql/udf/generic/GenericUDFMask.java     | 12 ++++++++----
 ql/src/test/queries/clientpositive/udf_mask.q   |  9 ++++++++-
 .../test/results/clientpositive/udf_mask.q.out  | 20 +++++++++++++++++---
 3 files changed, 33 insertions(+), 8 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hive/blob/74c81a89/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFMask.java
----------------------------------------------------------------------
diff --git a/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFMask.java b/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFMask.java
index cd35142..b815aae 100644
--- a/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFMask.java
+++ b/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFMask.java
@@ -96,12 +96,16 @@ class MaskTransformer extends AbstractTransformer {
       maskedNumber = MASKED_NUMBER;
     }
 
-    if(maskedDayValue < 1 || maskedDayValue > 31) {
-      maskedDayValue = MASKED_DAY_COMPONENT_VAL;
+    if(maskedDayValue != UNMASKED_VAL) {
+      if(maskedDayValue < 1 || maskedDayValue > 31) {
+        maskedDayValue = MASKED_DAY_COMPONENT_VAL;
+      }
     }
 
-    if(maskedMonthValue < 0 || maskedMonthValue > 11) {
-      maskedMonthValue = MASKED_MONTH_COMPONENT_VAL;
+    if(maskedMonthValue != UNMASKED_VAL) {
+      if(maskedMonthValue < 0 || maskedMonthValue > 11) {
+        maskedMonthValue = MASKED_MONTH_COMPONENT_VAL;
+      }
     }
   }
 

http://git-wip-us.apache.org/repos/asf/hive/blob/74c81a89/ql/src/test/queries/clientpositive/udf_mask.q
----------------------------------------------------------------------
diff --git a/ql/src/test/queries/clientpositive/udf_mask.q b/ql/src/test/queries/clientpositive/udf_mask.q
index 82b8ee7..15f7d27 100644
--- a/ql/src/test/queries/clientpositive/udf_mask.q
+++ b/ql/src/test/queries/clientpositive/udf_mask.q
@@ -10,4 +10,11 @@ select mask('TestString-123', 'X', 'x', '0', ':'),
        mask(cast(12345 as smallint), -1, -1, -1, -1, '5'),
        mask(cast(12345 as int), -1, -1, -1, -1, '5'),
        mask(cast(12345 as bigint), -1, -1, -1, -1, '5'),
-       mask(cast('2016-04-20' as date), -1, -1, -1, -1, -1, 0, 0, 0);
+       mask(cast('2016-04-20' as date), -1, -1, -1, -1, -1, 0, 0, 0),
+       mask(cast('2016-04-20' as date), -1, -1, -1, -1, -1, -1, 0, 0),
+       mask(cast('2016-04-20' as date), -1, -1, -1, -1, -1, 1, -1, 0),
+       mask(cast('2016-04-20' as date), -1, -1, -1, -1, -1, 1, 0, -1),
+       mask(cast('2016-04-20' as date), -1, -1, -1, -1, -1, 1, -1, -1),
+       mask(cast('2016-04-20' as date), -1, -1, -1, -1, -1, -1, 0, -1),
+       mask(cast('2016-04-20' as date), -1, -1, -1, -1, -1, -1, -1, 0),
+       mask(cast('2016-04-20' as date), -1, -1, -1, -1, -1, -1, -1, -1);

http://git-wip-us.apache.org/repos/asf/hive/blob/74c81a89/ql/src/test/results/clientpositive/udf_mask.q.out
----------------------------------------------------------------------
diff --git a/ql/src/test/results/clientpositive/udf_mask.q.out b/ql/src/test/results/clientpositive/udf_mask.q.out
index cbafaee..ac0e273 100644
--- a/ql/src/test/results/clientpositive/udf_mask.q.out
+++ b/ql/src/test/results/clientpositive/udf_mask.q.out
@@ -53,7 +53,14 @@ PREHOOK: query: select mask('TestString-123', 'X', 'x', '0', ':'),
        mask(cast(12345 as smallint), -1, -1, -1, -1, '5'),
        mask(cast(12345 as int), -1, -1, -1, -1, '5'),
        mask(cast(12345 as bigint), -1, -1, -1, -1, '5'),
-       mask(cast('2016-04-20' as date), -1, -1, -1, -1, -1, 0, 0, 0)
+       mask(cast('2016-04-20' as date), -1, -1, -1, -1, -1, 0, 0, 0),
+       mask(cast('2016-04-20' as date), -1, -1, -1, -1, -1, -1, 0, 0),
+       mask(cast('2016-04-20' as date), -1, -1, -1, -1, -1, 1, -1, 0),
+       mask(cast('2016-04-20' as date), -1, -1, -1, -1, -1, 1, 0, -1),
+       mask(cast('2016-04-20' as date), -1, -1, -1, -1, -1, 1, -1, -1),
+       mask(cast('2016-04-20' as date), -1, -1, -1, -1, -1, -1, 0, -1),
+       mask(cast('2016-04-20' as date), -1, -1, -1, -1, -1, -1, -1, 0),
+       mask(cast('2016-04-20' as date), -1, -1, -1, -1, -1, -1, -1, -1)
 PREHOOK: type: QUERY
 PREHOOK: Input: _dummy_database@_dummy_table
 #### A masked pattern was here ####
@@ -64,8 +71,15 @@ POSTHOOK: query: select mask('TestString-123', 'X', 'x', '0', ':'),
        mask(cast(12345 as smallint), -1, -1, -1, -1, '5'),
        mask(cast(12345 as int), -1, -1, -1, -1, '5'),
        mask(cast(12345 as bigint), -1, -1, -1, -1, '5'),
-       mask(cast('2016-04-20' as date), -1, -1, -1, -1, -1, 0, 0, 0)
+       mask(cast('2016-04-20' as date), -1, -1, -1, -1, -1, 0, 0, 0),
+       mask(cast('2016-04-20' as date), -1, -1, -1, -1, -1, -1, 0, 0),
+       mask(cast('2016-04-20' as date), -1, -1, -1, -1, -1, 1, -1, 0),
+       mask(cast('2016-04-20' as date), -1, -1, -1, -1, -1, 1, 0, -1),
+       mask(cast('2016-04-20' as date), -1, -1, -1, -1, -1, 1, -1, -1),
+       mask(cast('2016-04-20' as date), -1, -1, -1, -1, -1, -1, 0, -1),
+       mask(cast('2016-04-20' as date), -1, -1, -1, -1, -1, -1, -1, 0),
+       mask(cast('2016-04-20' as date), -1, -1, -1, -1, -1, -1, -1, -1)
 POSTHOOK: type: QUERY
 POSTHOOK: Input: _dummy_database@_dummy_table
 #### A masked pattern was here ####
-XxxxXxxxxx:000	XxxxXxxxxx:000	XxxxXxxxxx:000::::::::::                                                                                                                                                                                                                                       	43	-9981	55555	55555	1900-01-01
+XxxxXxxxxx:000	XxxxXxxxxx:000	XxxxXxxxxx:000::::::::::                                                                                                                                                                                                                                       	43	-9981	55555	55555	1900-01-01	1900-01-20	1900-04-01	2016-01-01	2016-04-01	2016-01-20	1900-04-20	2016-04-20