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;