[GitHub] nifi pull request: NIFI-1905 enabled ExecuteProcess to terminate p...

2016-05-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/nifi/pull/456 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enable

[GitHub] nifi pull request: NIFI-1905 enabled ExecuteProcess to terminate p...

2016-05-21 Thread aperepel
Github user aperepel commented on the pull request: https://github.com/apache/nifi/pull/456#issuecomment-220795892 LGTM, +1 on this one :) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] nifi pull request: NIFI-1905 enabled ExecuteProcess to terminate p...

2016-05-21 Thread olegz
Github user olegz commented on the pull request: https://github.com/apache/nifi/pull/456#issuecomment-220795133 @aperepel all squashed and pushed, including the change from WARN to INFO for log message. Let me know if there is anything else. --- If your project is set up for it,

[GitHub] nifi pull request: NIFI-1905 enabled ExecuteProcess to terminate p...

2016-05-21 Thread aperepel
Github user aperepel commented on a diff in the pull request: https://github.com/apache/nifi/pull/456#discussion_r64138418 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/ExecuteProcess.java --- @@ -187,7 +

[GitHub] nifi pull request: NIFI-1905 enabled ExecuteProcess to terminate p...

2016-05-20 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/456#discussion_r64044369 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/ExecuteProcess.java --- @@ -181,7 +183

[GitHub] nifi pull request: NIFI-1905 enabled ExecuteProcess to terminate p...

2016-05-20 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/456#discussion_r64043923 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/ExecuteProcess.java --- @@ -181,7 +183

[GitHub] nifi pull request: NIFI-1905 enabled ExecuteProcess to terminate p...

2016-05-20 Thread aperepel
Github user aperepel commented on a diff in the pull request: https://github.com/apache/nifi/pull/456#discussion_r64040583 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/ExecuteProcess.java --- @@ -181,7 +

[GitHub] nifi pull request: NIFI-1905 enabled ExecuteProcess to terminate p...

2016-05-20 Thread olegz
GitHub user olegz opened a pull request: https://github.com/apache/nifi/pull/456 NIFI-1905 enabled ExecuteProcess to terminate process You can merge this pull request into a Git repository by running: $ git pull https://github.com/olegz/nifi NIFI-1905 Alternatively you can re