[ https://issues.apache.org/jira/browse/PIG-1313?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12851228#action_12851228 ]
Hadoop QA commented on PIG-1313: -------------------------------- -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12440121/PIG-1313-2.patch against trunk revision 928384. +1 @author. The patch does not contain any @author tags. -1 tests included. The patch doesn't appear to include any new or modified tests. Please justify why no tests are needed for this patch. +1 javadoc. The javadoc tool did not generate any warning messages. +1 javac. The applied patch does not increase the total number of javac compiler warnings. +1 findbugs. The patch does not introduce any new Findbugs warnings. +1 release audit. The applied patch does not increase the total number of release audit warnings. -1 core tests. The patch failed core unit tests. +1 contrib tests. The patch passed contrib unit tests. Test results: http://hudson.zones.apache.org/hudson/job/Pig-Patch-h7.grid.sp2.yahoo.net/257/testReport/ Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Pig-Patch-h7.grid.sp2.yahoo.net/257/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html Console output: http://hudson.zones.apache.org/hudson/job/Pig-Patch-h7.grid.sp2.yahoo.net/257/console This message is automatically generated. > PigServer leaks memory over time > -------------------------------- > > Key: PIG-1313 > URL: https://issues.apache.org/jira/browse/PIG-1313 > Project: Pig > Issue Type: Bug > Reporter: Bill Graham > Assignee: Bill Graham > Attachments: PIG-1313-0.4.0-1.patch, PIG-1313-1.patch, > PIG-1313-1.patch, PIG-1313-2.patch, Pig1313Reproducer.java > > > When {{PigServer}} runs it creates temporary files using the > {{FileLocalizer.getTemporaryPath(..)}}. This static method creates and > returns a handle to a temporary file (as an instance of > {{ElementDescriptor}}). The {{ElementDescriptors}} returned by this method > are kept on a static {{Stack}} named {{toDelete}}. The items on {{toDelete}} > get removed by the {{FileLocalizer.deleteTempFile()}} method. > The only place in the code where I see {{FileLocalizer.deleteTempFile()}} > called is in the Main class. {{PigServer}} does not call that method though, > so a long-running VM that repeatedly uses instances of {{PigServer}} to run > jobs will leak memory via {{toDelete}}. > One suggested fix is to have {{PigServer.shutdown()}} call > {{FileLocalizer.deleteTempFile()}}, but this would cause problems in a > multi-threaded environment, since it seems {{ElementDescriptors}} are pushed > onto the {{toDelete}} stack before they're used, not once they're done with. > With this approach, running multiple instances of {{PigServer}} in separate > threads could cause one completed job to clobber the other's still-in-use > temp files. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.