You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@pig.apache.org by "John Gordon (JIRA)" <ji...@apache.org> on 2012/07/07 02:43:33 UTC
[jira] [Created] (PIG-2800) pig.additional.jars path separator
should align with File.pathSeparator instead of being hard-coded to ":"
John Gordon created PIG-2800:
--------------------------------
Summary: pig.additional.jars path separator should align with File.pathSeparator instead of being hard-coded to ":"
Key: PIG-2800
URL: https://issues.apache.org/jira/browse/PIG-2800
Project: Pig
Issue Type: Sub-task
Reporter: John Gordon
Windows paths use : after the drive letter, so pig currently fails to properly parse pig.additional.jars because it reads C:\file.txt as the list of jars: ["C", "\file.txt"] on Windows. ":" is the right separator for Linux, as it aligns with env variable separators in PATH very well. However, on Windows, it should be something that aligns similarly well.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (PIG-2800) pig.additional.jars path separator
should align with File.pathSeparator instead of being hard-coded to ":"
Posted by "John Gordon (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/PIG-2800?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13474804#comment-13474804 ]
John Gordon commented on PIG-2800:
----------------------------------
Thank you!
> pig.additional.jars path separator should align with File.pathSeparator instead of being hard-coded to ":"
> ----------------------------------------------------------------------------------------------------------
>
> Key: PIG-2800
> URL: https://issues.apache.org/jira/browse/PIG-2800
> Project: Pig
> Issue Type: Sub-task
> Reporter: John Gordon
> Fix For: 0.11
>
> Attachments: trunkJarsPatch.patch
>
>
> Windows paths use : after the drive letter, so pig currently fails to properly parse pig.additional.jars because it reads C:\file.txt as the list of jars: ["C", "\file.txt"] on Windows. ":" is the right separator for Linux, as it aligns with env variable separators in PATH very well. However, on Windows, it should be something that aligns similarly well.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Updated] (PIG-2800) pig.additional.jars path separator
should align with File.pathSeparator instead of being hard-coded to ":"
Posted by "John Gordon (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/PIG-2800?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
John Gordon updated PIG-2800:
-----------------------------
Attachment: trunkJarsPatch.patch
Split jars on File.pathSeparator instead of ":" -- test case that covers this is TestPigContext. Verified it still passes on Linux, and passes on Windows without Cygwin as well with branch-1-win and some unrelated test fixes.
> pig.additional.jars path separator should align with File.pathSeparator instead of being hard-coded to ":"
> ----------------------------------------------------------------------------------------------------------
>
> Key: PIG-2800
> URL: https://issues.apache.org/jira/browse/PIG-2800
> Project: Pig
> Issue Type: Sub-task
> Reporter: John Gordon
> Attachments: trunkJarsPatch.patch
>
>
> Windows paths use : after the drive letter, so pig currently fails to properly parse pig.additional.jars because it reads C:\file.txt as the list of jars: ["C", "\file.txt"] on Windows. ":" is the right separator for Linux, as it aligns with env variable separators in PATH very well. However, on Windows, it should be something that aligns similarly well.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Resolved] (PIG-2800) pig.additional.jars path separator
should align with File.pathSeparator instead of being hard-coded to ":"
Posted by "Gianmarco De Francisci Morales (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/PIG-2800?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Gianmarco De Francisci Morales resolved PIG-2800.
-------------------------------------------------
Resolution: Fixed
Fix Version/s: 0.11
Tests passed.
+1
Committed.
Thanks, John!
> pig.additional.jars path separator should align with File.pathSeparator instead of being hard-coded to ":"
> ----------------------------------------------------------------------------------------------------------
>
> Key: PIG-2800
> URL: https://issues.apache.org/jira/browse/PIG-2800
> Project: Pig
> Issue Type: Sub-task
> Reporter: John Gordon
> Fix For: 0.11
>
> Attachments: trunkJarsPatch.patch
>
>
> Windows paths use : after the drive letter, so pig currently fails to properly parse pig.additional.jars because it reads C:\file.txt as the list of jars: ["C", "\file.txt"] on Windows. ":" is the right separator for Linux, as it aligns with env variable separators in PATH very well. However, on Windows, it should be something that aligns similarly well.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Updated] (PIG-2800) pig.additional.jars path separator
should align with File.pathSeparator instead of being hard-coded to ":"
Posted by "Daniel Dai (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/PIG-2800?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Daniel Dai updated PIG-2800:
----------------------------
Fix Version/s: 0.10.1
Assignee: John Gordon
Further commit to 0.10 branch.
> pig.additional.jars path separator should align with File.pathSeparator instead of being hard-coded to ":"
> ----------------------------------------------------------------------------------------------------------
>
> Key: PIG-2800
> URL: https://issues.apache.org/jira/browse/PIG-2800
> Project: Pig
> Issue Type: Sub-task
> Reporter: John Gordon
> Assignee: John Gordon
> Fix For: 0.11, 0.10.1
>
> Attachments: trunkJarsPatch.patch
>
>
> Windows paths use : after the drive letter, so pig currently fails to properly parse pig.additional.jars because it reads C:\file.txt as the list of jars: ["C", "\file.txt"] on Windows. ":" is the right separator for Linux, as it aligns with env variable separators in PATH very well. However, on Windows, it should be something that aligns similarly well.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira