commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jochen Wiedmann (JIRA)" <j...@apache.org>
Subject [jira] Resolved: (FILEUPLOAD-155) Fix documentation on resource clean up in conjunction with DiskFileItemFactory
Date Sun, 18 Jan 2009 02:18:59 GMT

     [ https://issues.apache.org/jira/browse/FILEUPLOAD-155?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Jochen Wiedmann resolved FILEUPLOAD-155.
----------------------------------------

       Resolution: Fixed
    Fix Version/s: 1.3
         Assignee: Jochen Wiedmann

Fixed, thank you!


> Fix documentation on resource clean up in conjunction with DiskFileItemFactory
> ------------------------------------------------------------------------------
>
>                 Key: FILEUPLOAD-155
>                 URL: https://issues.apache.org/jira/browse/FILEUPLOAD-155
>             Project: Commons FileUpload
>          Issue Type: Improvement
>    Affects Versions: 1.2.1
>            Reporter: Jörg Heinicke
>            Assignee: Jochen Wiedmann
>            Priority: Minor
>             Fix For: 1.3
>
>
> 1. http://commons.apache.org/fileupload/using.html:
> "return new DiskFileItemFactory(fileCleaningTracker, DiskFileItemFactory.DEFAULT_SIZE_THRESHOLD,
repository);"
> DiskFileItemFactory does not have such a constructor.
> 2. http://commons.apache.org/fileupload/apidocs/org/apache/commons/fileupload/disk/DiskFileItemFactory.html
> "When using the DiskFileItemFactory, then you should consider the following: Temporary
files are automatically deleted as soon as they are no longer needed. (More precisely, when
the corresponding instance of File is garbage collected.) Cleaning up those files is done
by an instance of FileCleaningTracker, and an associated thread. In a complex environment,
for example in a web application, you should consider terminating this thread, for example,
when your web application ends. See the section on "Resource cleanup" in the users guide of
commons-fileupload."
> That's at least not precise. By default there is no FileCleaningTracker.
> 3. http://commons.apache.org/fileupload/apidocs/org/apache/commons/fileupload/disk/DiskFileItemFactory.html#getFileCleaningTracker()
> "Returns: An instance of FileCleaningTracker, defaults to FileCleaner.getInstance().
Null, if temporary files aren't tracked."
> There is no default tracker.
> 4. http://commons.apache.org/fileupload/apidocs/org/apache/commons/fileupload/disk/DiskFileItemFactory.html#setFileCleaningTracker(org.apache.commons.io.FileCleaningTracker)
> "Returns the tracker, which is responsible for deleting temporary files."
> Obviously wrong for setter. Copy&Paste error from getter.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message