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

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

I see.  Penalization of the flow files should not occur.  The issue is not with 
the flowfiles but with the state of the connection/environment itself.  That is 
precisely what 'yielding' is for and why the call to 'context.yield()' exists 
there.  All that needs to occur is the user may elect a larger yielding period 
if they wish.  So i'd back that part out.

I don't quite follow the JMS send vs commit part but I need to apply the patch 
to understand that more.  Will check and reply back.

The tests you added are awesome!

> 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