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

Mark Payne commented on NIFI-421:
---------------------------------

Toivo,

So in the second case, where BATCH_DURATION is not set, the idea is that it 
should not be split. Instead, the processor should just run until the command 
finishes executing, and all of the output should go to that one file. So this 
appears to be correct.

In the first case, though, where BATCH_DURATION is set to 500 millis, the test 
should only be outputing the first FlowFile. To process them all you would call 
"runner.run(19);"

You've got a good point about the future continuing to run. We should probably 
create a new method with @OnStopped that calls process.destroy():

{code}
@OnStopped
public void kilProcess() {
    if ( process != null ) {
        process.destroy();
    }
}
{code}

> ExecuteProcess does not honor back pressure setting
> ---------------------------------------------------
>
>                 Key: NIFI-421
>                 URL: https://issues.apache.org/jira/browse/NIFI-421
>             Project: Apache NiFi
>          Issue Type: Bug
>          Components: Extensions
>            Reporter: Joseph Witt
>            Assignee: Toivo Adams
>              Labels: beginner
>             Fix For: 0.1.0
>
>         Attachments: TestExecuteProcess.java
>
>
> ExecuteProcess does not pay attention to back pressure settings when 
> operating on a continuous stream of output.  It needs to be able to do so.  
> It should also document or make optional its behavior in the event of a back 
> pressure condition.  One it can kill the process it is executing or two it 
> can simply stop reading from the output stream.  There may be valid cases for 
> both options.



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

Reply via email to