You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pig.apache.org by da...@apache.org on 2014/11/01 18:45:17 UTC

svn commit: r1635998 - in /pig/trunk: CHANGES.txt src/org/apache/pig/scripting/jython/JythonScriptEngine.java

Author: daijy
Date: Sat Nov  1 17:45:17 2014
New Revision: 1635998

URL: http://svn.apache.org/r1635998
Log:
PIG-4182: e2e tests Scripting_[1-12] fail on Windows

Modified:
    pig/trunk/CHANGES.txt
    pig/trunk/src/org/apache/pig/scripting/jython/JythonScriptEngine.java

Modified: pig/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/pig/trunk/CHANGES.txt?rev=1635998&r1=1635997&r2=1635998&view=diff
==============================================================================
--- pig/trunk/CHANGES.txt (original)
+++ pig/trunk/CHANGES.txt Sat Nov  1 17:45:17 2014
@@ -110,6 +110,8 @@ OPTIMIZATIONS
  
 BUG FIXES
 
+PIG-4182: e2e tests Scripting_[1-12] fail on Windows (daijy)
+
 PIG-4259: Fix few issues related to Union, CROSS and auto parallelism in Tez (rohini)
 
 PIG-4250: Fix Security Risks found by Coverity (daijy)

Modified: pig/trunk/src/org/apache/pig/scripting/jython/JythonScriptEngine.java
URL: http://svn.apache.org/viewvc/pig/trunk/src/org/apache/pig/scripting/jython/JythonScriptEngine.java?rev=1635998&r1=1635997&r2=1635998&view=diff
==============================================================================
--- pig/trunk/src/org/apache/pig/scripting/jython/JythonScriptEngine.java (original)
+++ pig/trunk/src/org/apache/pig/scripting/jython/JythonScriptEngine.java Sat Nov  1 17:45:17 2014
@@ -300,7 +300,7 @@ public class JythonScriptEngine extends 
                                 // determine the relative path that the file should have in the jar
                                 int pos = apath.lastIndexOf(File.separatorChar + name.replace('.', File.separatorChar));
                                 if (pos > 0) {
-                                    files.put(apath.substring(pos), apath);
+                                    files.put(apath.substring(pos + 1), apath);
                                 } else {
                                     files.put(apath, apath);
                                 }