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/10/30 07:01:49 UTC
svn commit: r1635397 - in /pig/trunk: CHANGES.txt
src/org/apache/pig/impl/PigContext.java
src/org/apache/pig/scripting/jython/JythonScriptEngine.java
Author: daijy
Date: Thu Oct 30 06:01:49 2014
New Revision: 1635397
URL: http://svn.apache.org/r1635397
Log:
PIG-4182: e2e tests Scripting_[1-12] fail on Windows
Modified:
pig/trunk/CHANGES.txt
pig/trunk/src/org/apache/pig/impl/PigContext.java
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=1635397&r1=1635396&r2=1635397&view=diff
==============================================================================
--- pig/trunk/CHANGES.txt (original)
+++ pig/trunk/CHANGES.txt Thu Oct 30 06:01:49 2014
@@ -105,6 +105,8 @@ OPTIMIZATIONS
BUG FIXES
+PIG-4182: e2e tests Scripting_[1-12] fail on Windows (daijy)
+
PIG-4235: Fix unit test failures on Windows (daijy)
PIG-4245: 1-1 edge vertices should use same jvm opts (rohini)
Modified: pig/trunk/src/org/apache/pig/impl/PigContext.java
URL: http://svn.apache.org/viewvc/pig/trunk/src/org/apache/pig/impl/PigContext.java?rev=1635397&r1=1635396&r2=1635397&view=diff
==============================================================================
--- pig/trunk/src/org/apache/pig/impl/PigContext.java (original)
+++ pig/trunk/src/org/apache/pig/impl/PigContext.java Thu Oct 30 06:01:49 2014
@@ -334,7 +334,7 @@ public class PigContext implements Seria
*/
public void addScriptFile(String name, String path) {
if (path != null) {
- aliasedScriptFiles.put(name.replaceFirst("^/", "").replaceAll(":", ""), new File(path));
+ aliasedScriptFiles.put(name.replaceAll(":", ""), new File(path));
}
}
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=1635397&r1=1635396&r2=1635397&view=diff
==============================================================================
--- pig/trunk/src/org/apache/pig/scripting/jython/JythonScriptEngine.java (original)
+++ pig/trunk/src/org/apache/pig/scripting/jython/JythonScriptEngine.java Thu Oct 30 06:01:49 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);
}