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

Jeffrey Zhong commented on PHOENIX-1395:
----------------------------------------

[~giacomotaylor] I think the patch is mainly adding a test case to guard future 
possible spill file leaks and we don't have it now. We can add one more case in 
the patch for situation  "work rejected by thread executor" to guard us with 
future errors. How do u think?

> ResultSpooler spill files are left behind in /tmp folder
> --------------------------------------------------------
>
>                 Key: PHOENIX-1395
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-1395
>             Project: Phoenix
>          Issue Type: Bug
>            Reporter: Jeffrey Zhong
>            Assignee: Alicia Ying Shu
>             Fix For: 4.2
>
>         Attachments: PHOENIX-1395.patch, PHOENIX-1395.v1.patch
>
>
> Recently we found that some ResultSpooler*.bin files left in the tmp folder. 
> I think those are due to some client code doesn't call close on the returned 
> Resultset(which internally will invoke underlying 
> OnDiskResultIterator.close()) or client code get killed during result 
> iterating.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to