[GitHub] nifi pull request #589: NIFI-1952 Update to revision-locking for users and g...

2016-06-28 Thread jtstorck
GitHub user jtstorck opened a pull request: https://github.com/apache/nifi/pull/589 NIFI-1952 Update to revision-locking for users and groups Adding user and group summary objects (TenantEntity) Fixed ComponentEntity JSON mapping issues when the id field is null You can merge

[GitHub] nifi pull request #582: NIFI-1952 Updates to REST resources for user/group m...

2016-06-24 Thread jtstorck
GitHub user jtstorck opened a pull request: https://github.com/apache/nifi/pull/582 NIFI-1952 Updates to REST resources for user/group management Updated StandardPolicyBasedAuthorizerDAO to throw ResourceNotFoundExceptions when user/group/policy not found Added spec

[GitHub] nifi issue #526: NIFI-1952 Create REST endpoints for user/group/policy manag...

2016-06-16 Thread jtstorck
Github user jtstorck commented on the issue: https://github.com/apache/nifi/pull/526 @mcgilman I've addressed most of your comments on the PR aside from renaming Group to UserGroup, which I still plan on doing soon. Please take a look at my latest commit on the PR. --- If your

[GitHub] nifi pull request #526: NIFI-1952 Create REST endpoints for user/group/polic...

2016-06-14 Thread jtstorck
Github user jtstorck commented on a diff in the pull request: https://github.com/apache/nifi/pull/526#discussion_r66968481 --- Diff: nifi-api/src/main/java/org/apache/nifi/authorization/Group.java --- @@ -24,7 +24,7 @@ /** * A group that users can belong

[GitHub] nifi pull request #526: NIFI-1952 Create REST endpoints for user/group/polic...

2016-06-14 Thread jtstorck
Github user jtstorck commented on a diff in the pull request: https://github.com/apache/nifi/pull/526#discussion_r66968299 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/resources/nifi-web-api-context.xml --- @@ -114,7 +123,23

[GitHub] nifi pull request #526: NIFI-1952 Create REST endpoints for user/group/polic...

2016-06-14 Thread jtstorck
Github user jtstorck commented on a diff in the pull request: https://github.com/apache/nifi/pull/526#discussion_r66966751 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/api/UsersResource.java --- @@ -0,0

[GitHub] nifi pull request #526: NIFI-1952 Create REST endpoints for user/group/polic...

2016-06-14 Thread jtstorck
Github user jtstorck commented on a diff in the pull request: https://github.com/apache/nifi/pull/526#discussion_r66966643 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/api/AccessPolicyResource.java

[GitHub] nifi pull request #526: NIFI-1952 Create REST endpoints for user/group/polic...

2016-06-13 Thread jtstorck
GitHub user jtstorck opened a pull request: https://github.com/apache/nifi/pull/526 NIFI-1952 Create REST endpoints for user/group/policy management created REST Resources for users, groups, and access policies added Authorizables for users, groups, and access policies added

[GitHub] nifi pull request #485: NIFI-1948 Fixed encoding-verison attribute not getti...

2016-06-01 Thread jtstorck
GitHub user jtstorck opened a pull request: https://github.com/apache/nifi/pull/485 NIFI-1948 Fixed encoding-verison attribute not getting created in exp… …orted templates You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] nifi pull request: NIFI-1916 Updating FileAuthorizer to extend AbstractPolic...

2016-05-31 Thread jtstorck
Github user jtstorck commented on a diff in the pull request: https://github.com/apache/nifi/pull/473#discussion_r65270822 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-file-authorizer/src/main/java/org/apache/nifi/authorization/FileAuthorizer.java

[GitHub] nifi pull request: NIFI-1916 Updating FileAuthorizer to extend AbstractPolic...

2016-05-31 Thread jtstorck
Github user jtstorck commented on a diff in the pull request: https://github.com/apache/nifi/pull/473#discussion_r65268067 --- Diff: nifi-api/src/main/java/org/apache/nifi/authorization/AccessPolicy.java --- @@ -28,15 +28,15 @@ private final String identifier

[GitHub] nifi pull request: NIFI-1908 Added encoding-version attribute to T...

2016-05-30 Thread jtstorck
Github user jtstorck commented on a diff in the pull request: https://github.com/apache/nifi/pull/471#discussion_r65096222 --- 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-1908 Added encoding-version attribute to T...

2016-05-27 Thread jtstorck
GitHub user jtstorck opened a pull request: https://github.com/apache/nifi/pull/471 NIFI-1908 Added encoding-version attribute to TemplateDTO added scaling of templates upon instantiation for placement on the canvas added template-0.7.0.xml for live-testing the import

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

2016-05-20 Thread jtstorck
Github user jtstorck commented on a diff in the pull request: https://github.com/apache/nifi/pull/452#discussion_r64086436 --- Diff: nifi-api/src/main/java/org/apache/nifi/authorization/AccessPolicy.java --- @@ -0,0 +1,291 @@ +/* + * Licensed to the Apache Software

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

2016-05-20 Thread jtstorck
Github user jtstorck commented on the pull request: https://github.com/apache/nifi/pull/442#issuecomment-220668197 Hey @alopresto, I added a test case to address your previous comment regarding the flow file versioning. The two test specs, PositionScalerSpec

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

2016-05-20 Thread jtstorck
Github user jtstorck commented on the pull request: https://github.com/apache/nifi/pull/452#issuecomment-220607952 +1 looks good to me at this point, though @mcgilman's earlier comment on making MutableAuthorizer a base class does sound good, since the idea of the MutableAuthorizer

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

2016-05-20 Thread jtstorck
Github user jtstorck commented on a diff in the pull request: https://github.com/apache/nifi/pull/452#discussion_r64040788 --- Diff: nifi-api/src/main/java/org/apache/nifi/authorization/AccessPolicy.java --- @@ -0,0 +1,291 @@ +/* + * Licensed to the Apache Software

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

2016-05-18 Thread jtstorck
Github user jtstorck commented on a diff in the pull request: https://github.com/apache/nifi/pull/452#discussion_r63745403 --- Diff: nifi-api/src/main/java/org/apache/nifi/authorization/Group.java --- @@ -0,0 +1,115 @@ +/* + * Licensed to the Apache Software Foundation

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

2016-05-18 Thread jtstorck
Github user jtstorck commented on a diff in the pull request: https://github.com/apache/nifi/pull/452#discussion_r63744574 --- Diff: nifi-api/src/main/java/org/apache/nifi/authorization/AccessPolicy.java --- @@ -0,0 +1,125 @@ +/* + * Licensed to the Apache Software

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

2016-05-18 Thread jtstorck
Github user jtstorck commented on a diff in the pull request: https://github.com/apache/nifi/pull/452#discussion_r63743405 --- Diff: nifi-api/src/main/java/org/apache/nifi/authorization/AccessPolicy.java --- @@ -0,0 +1,125 @@ +/* + * Licensed to the Apache Software

[GitHub] nifi pull request: NIFI-1881 Added surefire-junit4 dependency to t...

2016-05-16 Thread jtstorck
Github user jtstorck commented on the pull request: https://github.com/apache/nifi/pull/449#issuecomment-219592267 Closed PR at the request of @alopresto. --- 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

[GitHub] nifi pull request: NIFI-1881 Added surefire-junit4 dependency to t...

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

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

2016-05-13 Thread jtstorck
GitHub user jtstorck opened a pull request: https://github.com/apache/nifi/pull/442 NIFI-1799 Implements auto-scaling flow layout added utility class to scale positions of components on the canvas, extracted get/setPosition methods from ProcesGroup, RemoteProcessGroup, Label