[GitHub] nifi pull request: NIFI-1047 Added additional method to get proces...

2016-03-15 Thread trkurc
Github user trkurc commented on the pull request: https://github.com/apache/nifi/pull/278#issuecomment-197021816 This makes shutdown work in cywin, but not for the right reason. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as w

[GitHub] nifi pull request: NIFI-1047 Added additional method to get proces...

2016-03-15 Thread trkurc
Github user trkurc closed the pull request at: https://github.com/apache/nifi/pull/278 --- 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-1047 Added additional method to get proces...

2016-03-15 Thread trkurc
Github user trkurc commented on a diff in the pull request: https://github.com/apache/nifi/pull/278#discussion_r56214991 --- Diff: nifi-bootstrap/src/main/java/org/apache/nifi/bootstrap/RunNiFi.java --- @@ -1131,6 +1135,22 @@ public void run() { this.loggingFutures = f

[GitHub] nifi pull request: NIFI-1047 Added additional method to get proces...

2016-03-15 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/278#discussion_r56213996 --- Diff: nifi-bootstrap/src/main/java/org/apache/nifi/bootstrap/RunNiFi.java --- @@ -1131,6 +1135,22 @@ public void run() { this.loggingFutures = fu

[GitHub] nifi pull request: NIFI-1047 Added additional method to get proces...

2016-03-14 Thread trkurc
GitHub user trkurc opened a pull request: https://github.com/apache/nifi/pull/278 NIFI-1047 Added additional method to get processId, fixed tools.jar path for cygwin You can merge this pull request into a Git repository by running: $ git pull https://github.com/trkurc/nifi NI

[GitHub] nifi pull request: NIFI-1047 Added additional method to get proces...

2016-03-14 Thread trkurc
Github user trkurc commented on the pull request: https://github.com/apache/nifi/pull/276#issuecomment-196619738 whoa, something went screwy when I merged this in. --- 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 p

[GitHub] nifi pull request: NIFI-1047 Added additional method to get proces...

2016-03-14 Thread trkurc
Github user trkurc closed the pull request at: https://github.com/apache/nifi/pull/276 --- 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-1047 Added additional method to get proces...

2016-03-14 Thread trkurc
GitHub user trkurc opened a pull request: https://github.com/apache/nifi/pull/276 NIFI-1047 Added additional method to get processId, fixed tools.jar path for cygwin Uses ugly internal classes, but works on windows You can merge this pull request into a Git repository by running: