[
https://issues.apache.org/jira/browse/NIFI-920?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15351110#comment-15351110
]
Joseph Gresock commented on NIFI-920:
-------------------------------------
The https://cwiki.apache.org/confluence/display/NIFI/Contributor+Guide says to
upload the patch and then select 'Submit Patch', but I don't see that option.
> UnpackContent can continuously rollback on java.nio.InvalidPathException
> ------------------------------------------------------------------------
>
> Key: NIFI-920
> URL: https://issues.apache.org/jira/browse/NIFI-920
> Project: Apache NiFi
> Issue Type: Bug
> Reporter: Brandon DeVries
> Priority: Minor
> Attachments:
> 0001-NIFI-920-Adding-InvalidPathException-handling-to-Unp.patch
>
>
> In UnpackContent, file.toPath() is called without catching
> java.nio.file.InvalidPathException:
> https://github.com/apache/nifi/blob/master/nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/UnpackContent.java#L265#L266
> https://github.com/apache/nifi/blob/master/nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/UnpackContent.java#L316#L318
> This bubbles up out of the processor and causes a rollback, instead of simply
> failing the file in the processor.
> This could be fixed in each "Unpacker" implementation (possibly by catching
> and wrapping in a ProcessException), or the catch in the OnTrigger method
> here:
> https://github.com/apache/nifi/blob/master/nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/UnpackContent.java#L238
> ... could simply be changed to "catch (final ProcessException |
> InvalidPathException e)". Whoever implements the fix can think that all the
> way through.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
