commit 61cb452177c63ff49bc843a5a3555072be914079
author Koji Noguchi <knoguchi@apache.org> Wed Jul 03 17:46:57 2019 +0000
committer Koji Noguchi <knoguchi@apache.org> Wed Jul 03 17:46:57 2019 +0000
tree 72be3b2d2ab33eb874afe279c9f77ac78c00e656
parent 1e664f22932399a937ee8674d461796cd2645bc2

PIG-5389: Passing null to REPLACE udf preventing JIT optimization (knoguchi)


git-svn-id: https://svn.apache.org/repos/asf/pig/trunk@1862503 13f79535-47bb-0310-9956-ffa450edef68

CHANGES.txt

diff --git a/CHANGES.txt b/CHANGES.txt
index ee6ce66..dfe5c3e 100644
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -26,6 +26,8 @@
  
 IMPROVEMENTS
 
+PIG-5389: Passing null to REPLACE udf preventing JIT optimization (knoguchi)
+
 PIG-5382: Log the name when POStore fails on write (knoguchi)
 
 PIG-5255: Improvements to bloom join (satishsaley via rohini)

src/org/apache/pig/builtin/REPLACE.java

diff --git a/src/org/apache/pig/builtin/REPLACE.java b/src/org/apache/pig/builtin/REPLACE.java
index bdb155b..9ee0f03 100644
--- a/src/org/apache/pig/builtin/REPLACE.java
+++ b/src/org/apache/pig/builtin/REPLACE.java
@@ -58,6 +58,10 @@
         String source = (String)input.get(0);
         String target = (String)input.get(1);
 
+        if (source == null) {
+            return null;
+        }
+
         if (target == null) {
             warn("Replace : Regular expression is null", PigWarning.UDF_WARNING_1);
             return null;