[jira] [Commented] (TIKA-1720) Collect multiple exceptions in TemporaryResources.close() using Throwable.addSuppressed()

["Hudson (JIRA)" <ji...@apache.org>]

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

Hudson commented on TIKA-1720:
------------------------------

SUCCESS: Integrated in tika-trunk-jdk1.7 #845 (See [https://builds.apache.org/job/tika-trunk-jdk1.7/845/])
TIKA-1720: Collect multiple exceptions in TemporaryResources.close() using Throwable.addSuppressed()

Patch by Yaniv Kunda. (jukka: http://svn.apache.org/viewvc/tika/trunk/?view=rev&rev=1698150)
* /tika/trunk/tika-core/src/main/java/org/apache/tika/io/TemporaryResources.java


> Collect multiple exceptions in TemporaryResources.close() using Throwable.addSuppressed()
> -----------------------------------------------------------------------------------------
>
>                 Key: TIKA-1720
>                 URL: https://issues.apache.org/jira/browse/TIKA-1720
>             Project: Tika
>          Issue Type: Improvement
>          Components: core
>            Reporter: Yaniv Kunda
>            Assignee: Jukka Zitting
>            Priority: Minor
>             Fix For: 1.11
>
>         Attachments: TIKA-1720.patch
>
>
> TemporaryResource.close() currently collects exceptions throw by trying to close its resources in a list.
> When the time to propagate an exception comes, information is lost - the thrown exception contains a message with the string descriptions of all exceptions, and the first exception as the cause - there is no stack trace describing what went wrong closing a resource.
> In addition, the thrown exception is IOExceptionWithCause, copied from commons-io, which is redundant since Java 6.



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