Author: knoguchi
Date: Wed Jul  3 17:46:57 2019
New Revision: 1862503

URL: http://svn.apache.org/viewvc?rev=1862503&view=rev
Log:
PIG-5389: Passing null to REPLACE udf preventing JIT optimization (knoguchi)

Modified:
    pig/trunk/CHANGES.txt
    pig/trunk/src/org/apache/pig/builtin/REPLACE.java

Modified: pig/trunk/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/pig/trunk/CHANGES.txt?rev=1862503&r1=1862502&r2=1862503&view=diff
==============================================================================
--- pig/trunk/CHANGES.txt (original)
+++ pig/trunk/CHANGES.txt Wed Jul  3 17:46:57 2019
@@ -26,6 +26,8 @@ PIG-5282: Upgade to Java 8 (satishsaley
  
 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)

Modified: pig/trunk/src/org/apache/pig/builtin/REPLACE.java
URL: 
http://svn.apache.org/viewvc/pig/trunk/src/org/apache/pig/builtin/REPLACE.java?rev=1862503&r1=1862502&r2=1862503&view=diff
==============================================================================
--- pig/trunk/src/org/apache/pig/builtin/REPLACE.java (original)
+++ pig/trunk/src/org/apache/pig/builtin/REPLACE.java Wed Jul  3 17:46:57 2019
@@ -58,6 +58,10 @@ public class REPLACE extends EvalFunc<St
         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;


Reply via email to