[GitHub] nifi pull request: NIFI-786 AWS credential refactoring and enhance...

2016-05-23 Thread jskora
Github user jskora commented on the pull request: https://github.com/apache/nifi/pull/244#issuecomment-221163835 This is really nice, it was great being able to just provide the AWS CLI profile and have setting automatically picked up. Great improvement, thanks James! ---

[GitHub] nifi pull request: NIFI-786 AWS credential refactoring and enhance...

2016-05-23 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/nifi/pull/244 --- 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

[GitHub] nifi pull request: NIFI-1884 Defining API for Users, Groups, and P...

2016-05-23 Thread mcgilman
Github user mcgilman commented on the pull request: https://github.com/apache/nifi/pull/452#issuecomment-221086451 @bbende The updated PR looks good to me. A number of great iterations here. Definitely think this is a solid foundation to continue the policy based authorizers managed

[GitHub] nifi pull request: NIFI-1799 Implements auto-scaling flow layout

2016-05-23 Thread alopresto
Github user alopresto commented on a diff in the pull request: https://github.com/apache/nifi/pull/442#discussion_r64277394 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/main/java/org/apache/nifi/groups/StandardProcessGroup.java --- @@

[GitHub] nifi pull request: NIFI-1660 - Enhance the expression language wit...

2016-05-23 Thread ckmcd
Github user ckmcd commented on the pull request: https://github.com/apache/nifi/pull/303#issuecomment-221073806 I've rebase on master. Other suggested changes are in the works. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] nifi pull request: NIFI-1913 Properly removing flowfile in HandleH...

2016-05-23 Thread pvillard31
Github user pvillard31 commented on the pull request: https://github.com/apache/nifi/pull/462#issuecomment-221058590 There is a double space on line 160 in test class. Travis seems not to be run on 0.x branch so I don't know if contrib check would fail because of that (cannot

[GitHub] nifi pull request: NIFI-1781: UI authorization updates

2016-05-23 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/nifi/pull/461 --- 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

[GitHub] nifi pull request: NIFI-1781: UI authorization updates

2016-05-23 Thread mcgilman
Github user mcgilman commented on the pull request: https://github.com/apache/nifi/pull/461#issuecomment-221051044 Thanks @markap14 All changes have been applied. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] nifi pull request: NIFI-1781: UI authorization updates

2016-05-23 Thread mcgilman
Github user mcgilman commented on a diff in the pull request: https://github.com/apache/nifi/pull/461#discussion_r64262070 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/api/ProcessGroupResource.java --- @@

[GitHub] nifi-minifi-cpp pull request: MINIFI-6: Basic C++ implementation f...

2016-05-23 Thread simonellistonball
Github user simonellistonball commented on the pull request: https://github.com/apache/nifi-minifi-cpp/pull/4#issuecomment-221049133 Tried building this with the default settings (-O0), which yields 1.3MB binary. Is the intention to provide a different build script for more

[GitHub] nifi pull request: NIFI-1781: UI authorization updates

2016-05-23 Thread markap14
Github user markap14 commented on a diff in the pull request: https://github.com/apache/nifi/pull/461#discussion_r64259242 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/api/SnippetResource.java --- @@ -0,0

[GitHub] nifi pull request: NIFI-1781: UI authorization updates

2016-05-23 Thread mcgilman
Github user mcgilman commented on a diff in the pull request: https://github.com/apache/nifi/pull/461#discussion_r64259949 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/api/FlowResource.java --- @@ -321,6

[GitHub] nifi pull request: NIFI-1884 Defining API for Users, Groups, and P...

2016-05-23 Thread bbende
Github user bbende commented on the pull request: https://github.com/apache/nifi/pull/452#issuecomment-221046683 @mcgilman @jtstorck @alopresto pushed two new commits, the first contains minor changes to address some of Matt's comments, the second is an attempt at what Matt suggested

[GitHub] nifi pull request: NIFI-1781: UI authorization updates

2016-05-23 Thread markap14
Github user markap14 commented on a diff in the pull request: https://github.com/apache/nifi/pull/461#discussion_r64259092 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/api/SnippetResource.java --- @@ -0,0

[GitHub] nifi pull request: NIFI-1781: UI authorization updates

2016-05-23 Thread markap14
Github user markap14 commented on a diff in the pull request: https://github.com/apache/nifi/pull/461#discussion_r64256472 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/api/FlowResource.java --- @@ -321,6

[GitHub] nifi pull request: NIFI-1781: UI authorization updates

2016-05-23 Thread markap14
Github user markap14 commented on a diff in the pull request: https://github.com/apache/nifi/pull/461#discussion_r64256311 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/api/FlowResource.java --- @@ -321,6

[GitHub] nifi pull request: NIFI-1913 Properly removing flowfile in HandleH...

2016-05-23 Thread JPercivall
GitHub user JPercivall opened a pull request: https://github.com/apache/nifi/pull/462 NIFI-1913 Properly removing flowfile in HandleHttpRequest when the Co… …ntext Map fails to register the request You can merge this pull request into a Git repository by running: $ git

[GitHub] nifi pull request: NIFI-1781: UI authorization updates

2016-05-23 Thread mcgilman
Github user mcgilman commented on a diff in the pull request: https://github.com/apache/nifi/pull/461#discussion_r64251051 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core-api/src/main/java/org/apache/nifi/groups/ProcessGroup.java --- @@ -46,6

[GitHub] nifi pull request: NIFI-1781: UI authorization updates

2016-05-23 Thread mcgilman
Github user mcgilman commented on a diff in the pull request: https://github.com/apache/nifi/pull/461#discussion_r64250889 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/StandardNiFiServiceFacade.java --- @@

[GitHub] nifi pull request: NIFI-1781: UI authorization updates

2016-05-23 Thread mcgilman
Github user mcgilman commented on a diff in the pull request: https://github.com/apache/nifi/pull/461#discussion_r64250267 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/main/java/org/apache/nifi/groups/StandardProcessGroup.java --- @@

[GitHub] nifi pull request: NIFI-1781: UI authorization updates

2016-05-23 Thread mcgilman
Github user mcgilman commented on a diff in the pull request: https://github.com/apache/nifi/pull/461#discussion_r64250769 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core-api/src/main/java/org/apache/nifi/groups/ProcessGroup.java --- @@ -46,6

[GitHub] nifi pull request: NIFI-1781: UI authorization updates

2016-05-23 Thread markap14
Github user markap14 commented on a diff in the pull request: https://github.com/apache/nifi/pull/461#discussion_r64249465 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/StandardNiFiServiceFacade.java --- @@

[GitHub] nifi pull request: NIFI-1781: UI authorization updates

2016-05-23 Thread markap14
Github user markap14 commented on a diff in the pull request: https://github.com/apache/nifi/pull/461#discussion_r64248276 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/main/java/org/apache/nifi/groups/StandardProcessGroup.java --- @@

[GitHub] nifi pull request: NIFI-1781: UI authorization updates

2016-05-23 Thread markap14
Github user markap14 commented on a diff in the pull request: https://github.com/apache/nifi/pull/461#discussion_r64248023 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core-api/src/main/java/org/apache/nifi/groups/ProcessGroup.java --- @@ -46,6

[GitHub] nifi pull request: NIFI-1781: UI authorization updates

2016-05-23 Thread markap14
Github user markap14 commented on a diff in the pull request: https://github.com/apache/nifi/pull/461#discussion_r64247888 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core-api/src/main/java/org/apache/nifi/groups/ProcessGroup.java --- @@ -46,6

[GitHub] nifi pull request: NIFI-1660 - Enhance the expression language wit...

2016-05-23 Thread ckmcd
Github user ckmcd commented on the pull request: https://github.com/apache/nifi/pull/303#issuecomment-221019823 @mattyb149, @markap14 I am working on your suggestions. Thanks. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] nifi pull request: NIFI-1781: UI authorization updates

2016-05-23 Thread mcgilman
GitHub user mcgilman opened a pull request: https://github.com/apache/nifi/pull/461 NIFI-1781: UI authorization updates NIFI-1781: - Including access policies in the breadcrumb's trail. - Updating toolbox according to group access policies. - Updating actions in palette

[GitHub] nifi pull request: NIFI-1660 - Enhance the expression language wit...

2016-05-23 Thread markap14
Github user markap14 commented on a diff in the pull request: https://github.com/apache/nifi/pull/303#discussion_r64235176 --- Diff: nifi-commons/nifi-expression-language/src/test/java/org/apache/nifi/attribute/expression/language/TestQuery.java --- @@ -233,6 +233,24 @@ public

[GitHub] nifi pull request: NIFI-1660 - Enhance the expression language wit...

2016-05-23 Thread markap14
Github user markap14 commented on a diff in the pull request: https://github.com/apache/nifi/pull/303#discussion_r64234770 --- Diff: nifi-commons/nifi-expression-language/src/main/java/org/apache/nifi/attribute/expression/language/evaluation/functions/JsonPathEvaluator.java ---

[GitHub] nifi pull request: NIFI-1907 Moving lazy init of SSLContext...

2016-05-23 Thread joewitt
Github user joewitt commented on the pull request: https://github.com/apache/nifi/pull/457#issuecomment-220988156 @bbende i think the move makes sense to improve the lifecycle of when such issues are detected. However, it does seem like throwing RuntimeExceptions for those could be

RE: EXTERNAL: Re: Provenance disabled

2016-05-23 Thread Gresock, Joseph
Ah, so obvious. That was it, thanks =) Joe Gresock AMPERE TTO Software Engineer Stf Lockheed Martin MST joseph.gres...@lmco.com -Original Message- From: Simon Elliston Ball [mailto:si...@simonellistonball.com] Sent: Monday, May 23, 2016 9:06 AM To: dev@nifi.apache.org Subject:

Re: Provenance disabled

2016-05-23 Thread Simon Elliston Ball
This will happen if you are authenticated as a user who does have the provenance role. That’s certainly the most common cause. There are other configurations which can do this, but they are unlikely on a clean install. Can you share anything about the authentication / provenance config? Simon

Provenance disabled

2016-05-23 Thread Gresock, Joseph
Hi folks, What would cause Nifi (0.5.1) to start up with the Data Provenance icon disabled (grayed out) and no Provenance context menu on processors? I don't see any errors or warnings in the logs related to provenance, and it appears to be writing files to the provenance_repository on disk.