[GitHub] nifi issue #328: NIFI-1690 Changed MonitorMemory to use allowable values for...

2016-06-16 Thread alopresto
Github user alopresto commented on the issue: https://github.com/apache/nifi/pull/328 Merged and closed. Will do the same for [PR 533](https://github.com/apache/nifi/pull/533). --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] nifi issue #328: NIFI-1690 Changed MonitorMemory to use allowable values for...

2016-06-16 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/328 @alopresto just fixed it. Took me a while to figure out what's going on but you can read some details here: https://issues.apache.org/jira/browse/NIFI-1730 (last comment). Also, I am now going to have

[GitHub] nifi issue #328: NIFI-1690 Changed MonitorMemory to use allowable values for...

2016-06-15 Thread alopresto
Github user alopresto commented on the issue: https://github.com/apache/nifi/pull/328 I've tried to build this locally twice (clean checkouts and everything). It is failing with a compilation error in `MonitorMemoryTest` saying that `UserService` cannot be found. I had to clean up a

[GitHub] nifi issue #328: NIFI-1690 Changed MonitorMemory to use allowable values for...

2016-06-15 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/328 @alopresto all done --- 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

[GitHub] nifi issue #328: NIFI-1690 Changed MonitorMemory to use allowable values for...

2016-06-15 Thread alopresto
Github user alopresto commented on the issue: https://github.com/apache/nifi/pull/328 Thanks @olegz . If you rebase and squash, I'll verify 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

[GitHub] nifi issue #328: NIFI-1690 Changed MonitorMemory to use allowable values for...

2016-06-15 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/328 @alopresto all is addressed. Let me know. --- 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