[GitHub] nifi pull request #256: NIFI-1578: Create PutSlack processor

2016-06-14 Thread alopresto
Github user alopresto commented on a diff in the pull request: https://github.com/apache/nifi/pull/256#discussion_r67098716 --- Diff: nifi-nar-bundles/nifi-slack-bundle/nifi-slack-processors/src/main/java/org/apache/nifi/processors/slack/PutSlack.java --- @@ -0,0 +1,238 @@

[GitHub] nifi pull request #256: NIFI-1578: Create PutSlack processor

2016-06-14 Thread adamonduty
Github user adamonduty commented on a diff in the pull request: https://github.com/apache/nifi/pull/256#discussion_r67095857 --- Diff: nifi-nar-bundles/nifi-slack-bundle/nifi-slack-processors/src/main/java/org/apache/nifi/processors/slack/PutSlack.java --- @@ -0,0 +1,238 @@

[GitHub] nifi pull request #256: NIFI-1578: Create PutSlack processor

2016-06-14 Thread adamonduty
Github user adamonduty commented on a diff in the pull request: https://github.com/apache/nifi/pull/256#discussion_r67095644 --- Diff: nifi-nar-bundles/nifi-slack-bundle/nifi-slack-processors/src/main/java/org/apache/nifi/processors/slack/PutSlack.java --- @@ -0,0 +1,238 @@

[GitHub] nifi pull request #256: NIFI-1578: Create PutSlack processor

2016-06-14 Thread adamonduty
Github user adamonduty commented on a diff in the pull request: https://github.com/apache/nifi/pull/256#discussion_r67095657 --- Diff: nifi-nar-bundles/nifi-slack-bundle/nifi-slack-processors/src/main/java/org/apache/nifi/processors/slack/PutSlack.java --- @@ -0,0 +1,238 @@

[GitHub] nifi pull request #328: NIFI-1690 Changed MonitorMemory to use allowable val...

2016-06-14 Thread alopresto
Github user alopresto commented on a diff in the pull request: https://github.com/apache/nifi/pull/328#discussion_r67093804 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-reporting-tasks/src/main/java/org/apache/nifi/controller/MonitorMemory.java --- @@ -154,30

[GitHub] nifi pull request #328: NIFI-1690 Changed MonitorMemory to use allowable val...

2016-06-14 Thread alopresto
Github user alopresto commented on a diff in the pull request: https://github.com/apache/nifi/pull/328#discussion_r67079817 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-reporting-tasks/src/main/java/org/apache/nifi/controller/MonitorMemory.java --- @@ -91,12

[GitHub] nifi pull request #328: NIFI-1690 Changed MonitorMemory to use allowable val...

2016-06-14 Thread alopresto
Github user alopresto commented on a diff in the pull request: https://github.com/apache/nifi/pull/328#discussion_r67076846 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-reporting-tasks/pom.xml --- @@ -1,60 +1,69 @@ -http://maven.apache.org/POM/4.0.0;

Re: [DISCUSS] Closing in on the Apache NiFi 0.7.0 Release

2016-06-14 Thread Andre
Tony, I second Joe's comments as well. Since the early discussions about the branching model I have been under the total impression that once 1.0 is released, 0.x would become support only and updates restricted to critical issues (security & data-loss break-fixes). This is not to say that a

Re: [DISCUSS] Closing in on the Apache NiFi 0.7.0 Release

2016-06-14 Thread Joe Witt
According to the discussion we had about the management of the release lines there would only be incremental releases when something was critical enough (security or data loss). And, if someone really wanted needed a minor release they could initiate and do that as well. But as far as continued

Re: ListenHTTP Questions

2016-06-14 Thread Mark Payne
Dale, That should work. I tried exactly that and go the result below. Can you verify that your ListenHTTP processor is running and that it doesn't show any bulletins? Have tried to telnet in? I.e., "telnet 8011" to see if you can connect that way? $ curl -i -X POST -H 'Content-Type:

ListenHTTP Questions

2016-06-14 Thread dale.chang13
So I am having some woes with sending HTTP Post requests to my NiFi server by using curl to send a JSON object. I have a ListenHTTP Processor running that is configured as so: - BasePath = contentListener - ListeningPort = 8011 - Authorized DN Pattern = .* - Max Unconfirmed FlowFile Time = 60 secs

Re: [DISCUSS] Closing in on the Apache NiFi 0.7.0 Release

2016-06-14 Thread Tony Kurc
Joe, for some reason, my mental image was that I expected we'd keep releasing new 0.x minor releases for a while along with 1.x. Is that everyone else's expectations? On Jun 14, 2016 4:41 AM, "Joe Witt" wrote: > Big thanks joe for stepping up to take on the RM tasks for the

[GitHub] nifi issue #502: Nifi-1972 Apache Ignite Put Cache Processor

2016-06-14 Thread pvillard31
Github user pvillard31 commented on the issue: https://github.com/apache/nifi/pull/502 @mans2singh Just had a new look and ran a workflow to test the processor. Few remarks even though I am not familiar with Ignite... First, I am not sure to see how I can check the content of

Re: NiFi Embedded/Headless/Portable

2016-06-14 Thread Andy LoPresto
Meant to add an additional link to the wiki page [1]. [1] https://cwiki.apache.org/confluence/display/NIFI/MiNiFi Andy LoPresto alopre...@apache.org alopresto.apa...@gmail.com PGP Fingerprint: 70EC B3E5 98A6 5A3F D3C4 BACE 3C6E F65B

Re: NiFi Embedded/Headless/Portable

2016-06-14 Thread Andy LoPresto
Ryan, I would suggest you take a look at Apache MiNiFi [1] — a sub-project of NiFi. Currently in development, it is designed specifically for this role (amongst others). I’m not sure if it is feature-complete enough for your needs, but it is definitely something you will be interested in.

[GitHub] nifi pull request #528: NIFI-2006 fixed NPE in PersistentProvenanceRepositor...

2016-06-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/nifi/pull/528 --- 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 #256: NIFI-1578: Create PutSlack processor

2016-06-14 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/256#discussion_r67040326 --- Diff: nifi-nar-bundles/nifi-slack-bundle/nifi-slack-processors/src/main/java/org/apache/nifi/processors/slack/PutSlack.java --- @@ -0,0 +1,238 @@ +/*

[GitHub] nifi pull request #256: NIFI-1578: Create PutSlack processor

2016-06-14 Thread alopresto
Github user alopresto commented on a diff in the pull request: https://github.com/apache/nifi/pull/256#discussion_r67039592 --- Diff: nifi-nar-bundles/nifi-slack-bundle/nifi-slack-processors/src/main/java/org/apache/nifi/processors/slack/PutSlack.java --- @@ -0,0 +1,238 @@

[GitHub] nifi-minifi pull request #17: MINIFI-30 added changes to the config transfor...

2016-06-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/nifi-minifi/pull/17 --- 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 #256: NIFI-1578: Create PutSlack processor

2016-06-14 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/256#discussion_r67039275 --- Diff: nifi-nar-bundles/nifi-slack-bundle/nifi-slack-processors/src/main/java/org/apache/nifi/processors/slack/PutSlack.java --- @@ -0,0 +1,238 @@

[GitHub] nifi pull request #256: NIFI-1578: Create PutSlack processor

2016-06-14 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/256#discussion_r67039181 --- Diff: nifi-nar-bundles/nifi-slack-bundle/nifi-slack-processors/src/main/java/org/apache/nifi/processors/slack/PutSlack.java --- @@ -0,0 +1,238 @@

[GitHub] nifi issue #528: NIFI-2006 fixed NPE in PersistentProvenanceRepository

2016-06-14 Thread JPercivall
Github user JPercivall commented on the issue: https://github.com/apache/nifi/pull/528 +1 Fixes problem and adds unit test check (which properly fails without the fix). I will merge it into 0.x and master. Thanks @olegz --- If your project is set up for it, you can reply

[GitHub] nifi issue #501: NIFI-1974 - Support Custom Properties in Expression Languag...

2016-06-14 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/501 Per my discussion with @markap14 having ControllerServiceLookup extend VariableRegistry interface did not fit in with the overall purpose of the ControllerServiceLookup interface. Also in some

[GitHub] nifi pull request #256: NIFI-1578: Create PutSlack processor

2016-06-14 Thread alopresto
Github user alopresto commented on a diff in the pull request: https://github.com/apache/nifi/pull/256#discussion_r67025553 --- Diff: nifi-nar-bundles/nifi-slack-bundle/nifi-slack-processors/src/main/java/org/apache/nifi/processors/slack/PutSlack.java --- @@ -0,0 +1,238 @@

[GitHub] nifi issue #256: NIFI-1578: Create PutSlack processor

2016-06-14 Thread alopresto
Github user alopresto commented on the issue: https://github.com/apache/nifi/pull/256 Built, passed checkstyle, and ran a sample flow posting to the Apache NiFi #test channel and directed at a user. Once the above comments are addressed, I'll revisit, but preliminary +1. --- If

[GitHub] nifi pull request #256: NIFI-1578: Create PutSlack processor

2016-06-14 Thread alopresto
Github user alopresto commented on a diff in the pull request: https://github.com/apache/nifi/pull/256#discussion_r67024788 --- Diff: nifi-nar-bundles/nifi-slack-bundle/nifi-slack-processors/src/main/java/org/apache/nifi/processors/slack/PutSlack.java --- @@ -0,0 +1,238 @@

[GitHub] nifi pull request #256: NIFI-1578: Create PutSlack processor

2016-06-14 Thread alopresto
Github user alopresto commented on a diff in the pull request: https://github.com/apache/nifi/pull/256#discussion_r67024579 --- Diff: nifi-nar-bundles/nifi-slack-bundle/nifi-slack-processors/src/main/java/org/apache/nifi/processors/slack/PutSlack.java --- @@ -0,0 +1,238 @@

[GitHub] nifi pull request #248: NIFI-1568: Add Filter Capability to UnpackContent

2016-06-14 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/248#discussion_r67017365 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/UnpackContent.java --- @@ -154,75

[GitHub] nifi pull request #256: NIFI-1578: Create PutSlack processor

2016-06-14 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/256#discussion_r67014112 --- Diff: nifi-nar-bundles/nifi-slack-bundle/nifi-slack-processors/src/main/java/org/apache/nifi/processors/slack/PutSlack.java --- @@ -0,0 +1,238 @@

[GitHub] nifi pull request #256: NIFI-1578: Create PutSlack processor

2016-06-14 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/256#discussion_r67014211 --- Diff: nifi-nar-bundles/nifi-slack-bundle/nifi-slack-processors/src/main/java/org/apache/nifi/processors/slack/PutSlack.java --- @@ -0,0 +1,238 @@

[GitHub] nifi-minifi pull request #17: MINIFI-30 added changes to the config transfor...

2016-06-14 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi-minifi/pull/17#discussion_r67004047 --- Diff: minifi-bootstrap/src/main/java/org/apache/nifi/minifi/bootstrap/util/schema/common/BaseSchema.java --- @@ -51,18 +55,80 @@ public void

[GitHub] nifi issue #256: NIFI-1578: Create PutSlack processor

2016-06-14 Thread adamonduty
Github user adamonduty commented on the issue: https://github.com/apache/nifi/pull/256 @mattyb149 Done! There were a couple of merge conflicts in pom files due to another processor being merged. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] nifi-minifi pull request #17: MINIFI-30 added changes to the config transfor...

2016-06-14 Thread apiri
Github user apiri commented on a diff in the pull request: https://github.com/apache/nifi-minifi/pull/17#discussion_r66997854 --- Diff: minifi-bootstrap/src/main/java/org/apache/nifi/minifi/bootstrap/util/schema/common/BaseSchema.java --- @@ -51,18 +55,80 @@ public void

[GitHub] nifi pull request #528: NIFI-2006 fixed NPE in PersistentProvenanceRepositor...

2016-06-14 Thread olegz
GitHub user olegz opened a pull request: https://github.com/apache/nifi/pull/528 NIFI-2006 fixed NPE in PersistentProvenanceRepository Fixed NPE in PersistentProvenanceRepository caused by attribute value being null added test You can merge this pull request into a Git

[GitHub] nifi issue #508: NIFI-1981 Cluster communication without client certificate

2016-06-14 Thread alopresto
Github user alopresto commented on the issue: https://github.com/apache/nifi/pull/508 @mattyb149 merged to `0.x`. --- 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

[GitHub] nifi pull request #508: NIFI-1981 Cluster communication without client certi...

2016-06-14 Thread alopresto
Github user alopresto commented on a diff in the pull request: https://github.com/apache/nifi/pull/508#discussion_r66991211 --- Diff: nifi-commons/nifi-security-utils/pom.xml --- @@ -30,6 +30,21 @@ org.apache.commons commons-lang3

[GitHub] nifi pull request #508: NIFI-1981 Cluster communication without client certi...

2016-06-14 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/508#discussion_r66990173 --- Diff: nifi-commons/nifi-security-utils/pom.xml --- @@ -30,6 +30,21 @@ org.apache.commons commons-lang3

[GitHub] nifi issue #508: NIFI-1981 Cluster communication without client certificate

2016-06-14 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/508 +1 LGTM, built and ran with Java 7 and 8 on 0.x branch. Mind squashing the commits? Then I'll merge to 0.x and master, thanks! --- If your project is set up for it, you can reply to this email and

[GitHub] nifi issue #508: NIFI-1981 Cluster communication without client certificate

2016-06-14 Thread alopresto
Github user alopresto commented on the issue: https://github.com/apache/nifi/pull/508 Squashing and will push. Thanks @mattyb149 . --- 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

[GitHub] nifi pull request #527: NIFI-2009 removed 'annotationData' and 'validationCo...

2016-06-14 Thread olegz
GitHub user olegz opened a pull request: https://github.com/apache/nifi/pull/527 NIFI-2009 removed 'annotationData' and 'validationContextFactory' ins… …tance variables from StandardProcessNode You can merge this pull request into a Git repository by running: $ git pull

Re: unwedgeable flow caused by "Cannot create Provenance Event Record because FlowFile UUID is not set"

2016-06-14 Thread Oleg Zhurakousky
Thank you Chris Will be addressed shortly Oleg > On Jun 14, 2016, at 9:56 AM, McDermott, Chris Kevin (MSDU - > STaTS/StorefrontRemote) wrote: > > Thanks, Oleg. I’ve created NIFI-2015. > > > Regards, > > Chris McDermott > > Remote Business Analytics >

Re: unwedgeable flow caused by "Cannot create Provenance Event Record because FlowFile UUID is not set"

2016-06-14 Thread McDermott, Chris Kevin (MSDU - STaTS/StorefrontRemote)
Thanks, Oleg. I’ve created NIFI-2015. Regards, Chris McDermott Remote Business Analytics STaTS/StoreFront Remote HPE Storage Hewlett Packard Enterprise Mobile: +1 978-697-5315 https://www.storefrontremote.com On 6/13/16, 3:10 PM, "Oleg Zhurakousky" wrote:

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

2016-06-14 Thread mcgilman
Github user mcgilman commented on a diff in the pull request: https://github.com/apache/nifi/pull/526#discussion_r66968920 --- 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_r66968481 --- Diff: nifi-api/src/main/java/org/apache/nifi/authorization/Group.java --- @@ -24,7 +24,7 @@ /** * A group that users can belong to. */

[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 issue #526: NIFI-1952 Create REST endpoints for user/group/policy manag...

2016-06-14 Thread mcgilman
Github user mcgilman commented on the issue: https://github.com/apache/nifi/pull/526 @jtstorck Lets knock out these couple trivial items. Looks good so far. --- 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 #526: NIFI-1952 Create REST endpoints for user/group/polic...

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

[GitHub] nifi issue #255: NIFI-1594: Add option to bulk using Index or Update.

2016-06-14 Thread joaohf
Github user joaohf commented on the issue: https://github.com/apache/nifi/pull/255 @mattyb149, done. Could you take a look? --- 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

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

2016-06-14 Thread mcgilman
Github user mcgilman commented on a diff in the pull request: https://github.com/apache/nifi/pull/526#discussion_r66960292 --- 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 mcgilman
Github user mcgilman commented on a diff in the pull request: https://github.com/apache/nifi/pull/526#discussion_r66959842 --- 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 mcgilman
Github user mcgilman commented on a diff in the pull request: https://github.com/apache/nifi/pull/526#discussion_r66959717 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/api/UserGroupsResource.java --- @@

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

2016-06-14 Thread mcgilman
Github user mcgilman commented on a diff in the pull request: https://github.com/apache/nifi/pull/526#discussion_r66959598 --- 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 issue #515: NIFI-826 [REVIEW ONLY] Initial commit for deterministic tem...

2016-06-14 Thread mcgilman
Github user mcgilman commented on the issue: https://github.com/apache/nifi/pull/515 @olegz I believe that we're going to need to introduce the templateId or inceptionId into the flow.xml not just the template DTO. The reasoning is that we need to support the following case:

[GitHub] nifi issue #518: NIFI-1901: Component based access control tests

2016-06-14 Thread mcgilman
Github user mcgilman commented on the issue: https://github.com/apache/nifi/pull/518 It appears that the unit tests I have added take a long time to complete and causes Travis to timeout and fail. Specifically, the slow running tests stand up an embedded Jetty server and deploys NiFi