[ 
https://issues.apache.org/jira/browse/PIG-3169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13639336#comment-13639336
 ] 

Cheolsoo Park commented on PIG-3169:
------------------------------------

Let's do option #2! I think it's safe to remove 
{{getTemporaryPath(ElementDescriptor, PigContext)}} in 0.12. Do you mind 
updating the patch by removing this method?

Btw, you have unused imports in MapReduceLauncher.java and FileLocalizer.java:
{code}
> +import java.util.Set;
> +import org.apache.pig.backend.datastorage.ElementDescriptor;

> +import java.util.HashSet;
> +import java.util.Set;
{code}
I am running unit tests with PIG-3169.5.patch now. I will let you know if I see 
any failures.

Thanks!
                
> Remove intermediate data after a job finishes
> ---------------------------------------------
>
>                 Key: PIG-3169
>                 URL: https://issues.apache.org/jira/browse/PIG-3169
>             Project: Pig
>          Issue Type: Improvement
>            Reporter: Mark Wagner
>            Assignee: Mark Wagner
>            Priority: Minor
>             Fix For: 0.12
>
>         Attachments: PIG-3169.1.patch, PIG-3169.2.patch, PIG-3169.3.patch, 
> PIG-3169.4.patch, PIG-3169.5.patch, PIG-3169-hotfix.patch
>
>
> When using Grunt, intermediate data and distributed caches files are left in 
> 'pig.temp.dir' until the session is closed. It would be nice to cleanup files 
> as they are no longer needed.

--
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

Reply via email to