[
https://issues.apache.org/jira/browse/PIG-2591?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13598554#comment-13598554
]
Cheolsoo Park commented on PIG-2591:
------------------------------------
[~prkommireddi], you're right about the original intention of this jira and
pig.temp.dir.
I think that there are 2 problems here:
# Some unit tests hard-coded /tmp.
# pig.temp.dir is not set, so FileLocalizer.getTemporaryPath() creates files in
/tmp.
This is what I suggest:
# We make every unit test use FileLocalizer.getTemporaryPath().
# We set pig.temp.dir to java.io.tmpdir in build.xml (or ./build so ant clean
can delete them).
Then, we meet all the requirements of PIG-2995 and PIG-2591. Do you agree?
> Unit tests should not write to /tmp but respect java.io.tmpdir
> --------------------------------------------------------------
>
> Key: PIG-2591
> URL: https://issues.apache.org/jira/browse/PIG-2591
> Project: Pig
> Issue Type: Bug
> Components: tools
> Reporter: Thomas Weise
> Assignee: Jarek Jarcec Cecho
> Fix For: 0.12
>
> Attachments: bugPIG-2591.patch, PIG-2495.patch
>
>
> Several tests use /tmp but should derive temporary file location from
> java.io.tmpdir to avoid side effects (java.io.tmpdir is already set to a test
> run specific location in build.xml)
--
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
