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

Joseph Witt commented on NIFI-1300:
-----------------------------------

context.yield() was and is the right way to go in this case for avoiding 
hammering the remote system when clearly it is in a funky state.  So that part 
is fine.

I am not entirely sure about rollback vs transfer to failure in the case you 
changed.  That might be the right move.  I need to see it in context.

So the only thing i'm balking at right now is the addition of penalization.

Will reply in a sec once i can see the patch in context.

Thanks

> PutJMS should fail flowfile on commit exception and penalize flowfiles routed 
> to failure.
> -----------------------------------------------------------------------------------------
>
>                 Key: NIFI-1300
>                 URL: https://issues.apache.org/jira/browse/NIFI-1300
>             Project: Apache NiFi
>          Issue Type: Bug
>          Components: Core Framework
>            Reporter: Joe Skora
>            Assignee: Joe Skora
>            Priority: Minor
>
> PutJMS fails the flowfile if an exception occurs during message send, but 
> rollsback if an exception occurs during commit.  It should fail in both cases 
> and it should penalize the flowfile for either failure.



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

Reply via email to