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 2012/10/31 23:01:14 UTC

svn commit: r1404376 - in /pig/branches/branch-0.10: CHANGES.txt src/org/apache/pig/PigServer.java

Author: daijy
Date: Wed Oct 31 22:01:14 2012
New Revision: 1404376

URL: http://svn.apache.org/viewvc?rev=1404376&view=rev
Log:
PIG-2800: pig.additional.jars path separator should align with File.pathSeparator instead of being hard-coded to ":"
(jgordon via azaroth)

Modified:
    pig/branches/branch-0.10/CHANGES.txt
    pig/branches/branch-0.10/src/org/apache/pig/PigServer.java

Modified: pig/branches/branch-0.10/CHANGES.txt
URL: http://svn.apache.org/viewvc/pig/branches/branch-0.10/CHANGES.txt?rev=1404376&r1=1404375&r2=1404376&view=diff
==============================================================================
--- pig/branches/branch-0.10/CHANGES.txt (original)
+++ pig/branches/branch-0.10/CHANGES.txt Wed Oct 31 22:01:14 2012
@@ -36,6 +36,9 @@ PIG-2727: PigStorage Source tagging does
 
 BUG FIXES
 
+PIG-2800: pig.additional.jars path separator should align with File.pathSeparator instead of being hard-coded to ":"
+(jgordon via azaroth)
+
 PIG-2798: pig streaming tests assume interpreters are auto-resolved (jgordon via daijy)
 
 PIG-2797: Tests should not create their own file URIs through string concatenation, should use Util.generateURI

Modified: pig/branches/branch-0.10/src/org/apache/pig/PigServer.java
URL: http://svn.apache.org/viewvc/pig/branches/branch-0.10/src/org/apache/pig/PigServer.java?rev=1404376&r1=1404375&r2=1404376&view=diff
==============================================================================
--- pig/branches/branch-0.10/src/org/apache/pig/PigServer.java (original)
+++ pig/branches/branch-0.10/src/org/apache/pig/PigServer.java Wed Oct 31 22:01:14 2012
@@ -252,8 +252,12 @@ public class PigServer {
     private void addJarsFromProperties() throws ExecException {
         //add jars from properties to extraJars
         String jar_str = pigContext.getProperties().getProperty("pig.additional.jars");
+
         if(jar_str != null){
-            for(String jar : jar_str.split(":")){
+            // Use File.pathSeparator (":" on Linux, ";" on Windows)
+            // to correctly handle path aggregates as they are represented
+            // on the Operating System.
+            for(String jar : jar_str.split(File.pathSeparator)){
                 try {
                     registerJar(jar);
                 } catch (IOException e) {