[jboss-jira] [JBoss JIRA] Commented: (JBVFS-161) TempFileProvider should have a injectable FileDelete strategy

[ https://jira.jboss.org/browse/JBVFS-161?page=com.atlassian.jira.plugin.sy... ] Aslak Knutsen commented on JBVFS-161: ------------------------------------- (10:39:27 AM) ALR: baileyje: Up yet? :) (10:39:48 AM) baileyje: Yeah :) (10:40:11 AM) ALR: baileyje: I'd like to get at a TempDir without a ScheduledExecutorService (10:40:21 AM) ALR: The service is just used for cleanup operations in close() (10:40:54 AM) ALR: http://pastebin.com/sBuqcbB2 (10:40:59 AM) baileyje: Yeah. Cleanup with some retry behavior to prevent locking. (10:41:05 AM) ALR: Yep. (10:41:12 AM) ALR: baileyje: Can that be optional? (10:41:32 AM) ALR: And instead go to some deleteOnExit logic if no SES was provided at construction? (10:42:41 AM) ALR: baileyje: I've a case where we shouldn't be flushing VirtualFiles to getPhysicalFile anyway (10:42:56 AM) ALR: So providing a SES is kind unnecessary in the first place. (10:42:57 AM) baileyje: ALR: hmmm. What is the issue with the executor? (10:43:03 AM) ALR: baileyje: I don't have one. (10:43:12 AM) baileyje: ok. You just want to avoid it.. (10:43:13 AM) ALR: baileyje: And in AS, most components are not allowed to make one. (10:43:15 AM) ALR: Yeah. (10:44:15 AM) ALR: baileyje: I'll be happy to make the patch/fix if we can agree on some alternate logic. (10:44:52 AM) baileyje: I will pull dmlloyd in on that. He was the originator of the TempFileProvider. (10:45:17 AM) ALR: baileyje: Cool, thanks. We'll see what he says in a bit. (10:45:44 AM) baileyje: I think if there is a legitimate reason to have a different cleanup strategy, something could be done. (10:46:02 AM) ALR: baileyje: Simplification / ease of ue. (10:46:04 AM) ALR: *use -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira