[GitHub] nifi issue #541: NIFI-2032 fixed 'enableControllerServices' logic

2016-06-20 Thread alopresto
Github user alopresto commented on the issue: https://github.com/apache/nifi/pull/541 I merged into master but I've had to rebase and resolve conflicts 3 times on `0.x` branch and still have tests failing. Will continue working on that part in the morning. --- If your project is

[GitHub] nifi issue #541: NIFI-2032 fixed 'enableControllerServices' logic

2016-06-20 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/541 @alopresto so I just fixed it and squashed it with the last commit. No compilation error on my end. LMK --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] nifi issue #541: NIFI-2032 fixed 'enableControllerServices' logic

2016-06-20 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/541 I see, it appears that I have to reference different ```org.apache.nifi.events.BulletinFactory```, building now --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] nifi issue #541: NIFI-2032 fixed 'enableControllerServices' logic

2016-06-20 Thread alopresto
Github user alopresto commented on the issue: https://github.com/apache/nifi/pull/541 Getting a compilation failure missing `import org.apache.nifi.reporting.BulletinFactory`. Did this get missed in the second commit? --- If your project is set up for it, you can reply to this

[GitHub] nifi issue #541: NIFI-2032 fixed 'enableControllerServices' logic

2016-06-20 Thread alopresto
Github user alopresto commented on the issue: https://github.com/apache/nifi/pull/541 Thanks @olegz . I will build locally and merge. --- 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