[GitHub] nifi pull request: Nifi 1495 - AWS Kinesis Firehose

2016-03-14 Thread mans2singh
Github user mans2singh commented on the pull request: https://github.com/apache/nifi/pull/213#issuecomment-196646278 @aldrin - I've added max buffer size limit check while getting flow files from the session as you had recommended. I've also kept the batch size limit in place. I've a

[GitHub] nifi pull request: NIFI-1481 fixed a couple bugs in bash script fo...

2016-03-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/nifi/pull/277 --- 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 enable

[GitHub] nifi pull request: NIFI-1481 fixed a couple bugs in bash script fo...

2016-03-14 Thread apiri
Github user apiri commented on the pull request: https://github.com/apache/nifi/pull/277#issuecomment-196624448 Verified on OS X with and without JAVA_HOME set to permit the usage of env. All worked as anticipated. Performed the same on Cygwin with anticipated results.

[GitHub] nifi pull request: NIFI-1047 Added additional method to get proces...

2016-03-14 Thread trkurc
GitHub user trkurc opened a pull request: https://github.com/apache/nifi/pull/278 NIFI-1047 Added additional method to get processId, fixed tools.jar path for cygwin You can merge this pull request into a Git repository by running: $ git pull https://github.com/trkurc/nifi NI

[GitHub] nifi pull request: NIFI-1047 Added additional method to get proces...

2016-03-14 Thread trkurc
Github user trkurc commented on the pull request: https://github.com/apache/nifi/pull/276#issuecomment-196619738 whoa, something went screwy when I merged this in. --- 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 p

[GitHub] nifi pull request: NIFI-1047 Added additional method to get proces...

2016-03-14 Thread trkurc
Github user trkurc closed the pull request at: https://github.com/apache/nifi/pull/276 --- 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 enable

[GitHub] nifi pull request: NIFI-1481 fixed a couple bugs in bash script fo...

2016-03-14 Thread trkurc
GitHub user trkurc opened a pull request: https://github.com/apache/nifi/pull/277 NIFI-1481 fixed a couple bugs in bash script for cygwin and when JAVA_HOME not set You can merge this pull request into a Git repository by running: $ git pull https://github.com/trkurc/nifi NIF

[GitHub] nifi pull request: NIFI-1481 fixed a couple bugs in bash script fo...

2016-03-14 Thread apiri
Github user apiri commented on the pull request: https://github.com/apache/nifi/pull/277#issuecomment-196618302 reviewing --- 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

[GitHub] nifi pull request: NIFI-1571 initial commit of SpringContext suppo...

2016-03-14 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/271#discussion_r56106623 --- Diff: nifi-nar-bundles/nifi-spring-bundle/nifi-spring-processors/src/main/java/org/apache/nifi/spring/bootstrap/SpringContextDelegate.java --- @@ -0,0 +1,141

[GitHub] nifi pull request: NIFI-1571 initial commit of SpringContext suppo...

2016-03-14 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/271#discussion_r56106557 --- Diff: nifi-nar-bundles/nifi-spring-bundle/nifi-spring-processors/src/main/java/org/apache/nifi/spring/SpringContextProcessor.java --- @@ -0,0 +1,382 @@ +

[GitHub] nifi pull request: NIFI-1571 initial commit of SpringContext suppo...

2016-03-14 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/271#discussion_r56106464 --- Diff: nifi-nar-bundles/nifi-spring-bundle/nifi-spring-processors/src/main/java/org/apache/nifi/spring/SpringContextFactory.java --- @@ -0,0 +1,139 @@ +/*

[GitHub] nifi pull request: NIFI-1571 initial commit of SpringContext suppo...

2016-03-14 Thread trkurc
Github user trkurc commented on a diff in the pull request: https://github.com/apache/nifi/pull/271#discussion_r56105950 --- Diff: nifi-nar-bundles/nifi-spring-bundle/nifi-spring-processors/src/main/java/org/apache/nifi/spring/bootstrap/SpringContextDelegate.java --- @@ -0,0 +1,14

[GitHub] nifi pull request: NIFI-1571 initial commit of SpringContext suppo...

2016-03-14 Thread trkurc
Github user trkurc commented on a diff in the pull request: https://github.com/apache/nifi/pull/271#discussion_r56105879 --- Diff: nifi-nar-bundles/nifi-spring-bundle/nifi-spring-processors/src/main/resources/docs/org.apache.nifi.spring.SpringContextProcessor/additionalDetails.html --

[GitHub] nifi pull request: NIFI-1571 initial commit of SpringContext suppo...

2016-03-14 Thread trkurc
Github user trkurc commented on a diff in the pull request: https://github.com/apache/nifi/pull/271#discussion_r56104556 --- Diff: nifi-nar-bundles/nifi-spring-bundle/nifi-spring-processors/src/main/java/org/apache/nifi/spring/SpringContextFactory.java --- @@ -0,0 +1,139 @@ +/

[GitHub] nifi pull request: NIFI-1571 initial commit of SpringContext suppo...

2016-03-14 Thread trkurc
Github user trkurc commented on a diff in the pull request: https://github.com/apache/nifi/pull/271#discussion_r56104123 --- Diff: nifi-nar-bundles/nifi-spring-bundle/nifi-spring-processors/src/main/java/org/apache/nifi/spring/SpringContextFactory.java --- @@ -0,0 +1,139 @@ +/

[GitHub] nifi pull request: NIFI-1571 initial commit of SpringContext suppo...

2016-03-14 Thread joewitt
Github user joewitt commented on a diff in the pull request: https://github.com/apache/nifi/pull/271#discussion_r56103524 --- Diff: nifi-nar-bundles/nifi-spring-bundle/nifi-spring-processors/src/main/java/org/apache/nifi/spring/SpringContextProcessor.java --- @@ -0,0 +1,382 @@

[GitHub] nifi pull request: NIFI-1571 initial commit of SpringContext suppo...

2016-03-14 Thread trkurc
Github user trkurc commented on a diff in the pull request: https://github.com/apache/nifi/pull/271#discussion_r56103351 --- Diff: nifi-nar-bundles/nifi-spring-bundle/nifi-spring-processors/src/main/resources/docs/org.apache.nifi.spring.SpringContextProcessor/additionalDetails.html --

[GitHub] nifi pull request: NIFI-1571 initial commit of SpringContext suppo...

2016-03-14 Thread trkurc
Github user trkurc commented on a diff in the pull request: https://github.com/apache/nifi/pull/271#discussion_r56102458 --- Diff: nifi-nar-bundles/pom.xml --- @@ -54,7 +54,8 @@ nifi-scripting-bundle nifi-elasticsearch-bundle nifi-amqp-bundle

Re: Closing in on the Apache NiFi 0.6.0 release

2016-03-14 Thread Joe Witt
Team, Things are closing in well but we need to get pretty specific on these to keep the release moving roughly along the lines of the schedule we've discussed previously. Those of you who have tickets on here that can be moved to 0.7.0 please do so. Otherwise, let's please keep discussion/statu

[GitHub] nifi pull request: NIFI-1047 Added additional method to get proces...

2016-03-14 Thread trkurc
GitHub user trkurc opened a pull request: https://github.com/apache/nifi/pull/276 NIFI-1047 Added additional method to get processId, fixed tools.jar path for cygwin Uses ugly internal classes, but works on windows You can merge this pull request into a Git repository by running:

[GitHub] nifi pull request: NiFi-1481 Enhancement[ nifi.sh env]

2016-03-14 Thread trkurc
Github user trkurc commented on the pull request: https://github.com/apache/nifi/pull/218#issuecomment-196564618 So, after discussion with @apiri, we removed the env batch script because it did not work due to limitations of bootstrap, and reworked nifi.sh to build the classpath for B

[GitHub] nifi pull request: NiFi-1481 Enhancement[ nifi.sh env]

2016-03-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/nifi/pull/218 --- 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 enable

[GitHub] nifi pull request: NIFI-614 Added initial support for new style JM...

2016-03-14 Thread olegz
Github user olegz commented on the pull request: https://github.com/apache/nifi/pull/222#issuecomment-196535593 I'll look at it. I think I know what's going on. --- 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 proj

[GitHub] nifi pull request: NIFI-614 Added initial support for new style JM...

2016-03-14 Thread mosermw
Github user mosermw commented on the pull request: https://github.com/apache/nifi/pull/222#issuecomment-196532379 Yep, that's what I did. The TibjmsConnectionFactory must not see the truststore I set in the SSL Context Service. I'm worried that different JMS providers require differ

Re: Establishment of MiNiFi repo and supporting tools

2016-03-14 Thread Matt Burgess
Excellent, thanks! > On Mar 14, 2016, at 4:57 PM, Aldrin Piri wrote: > > Given the positive response, I have submitted requests for both a new JIRA > and new Git repository which INFRA has kindly stood up for us. MiNiFi JIRA > and Git are available at [1] and [2], respectively. > > [1] https:/

[GitHub] nifi pull request: NIFI-614 Added initial support for new style JM...

2016-03-14 Thread JPercivall
Github user JPercivall commented on the pull request: https://github.com/apache/nifi/pull/222#issuecomment-196527092 @mosermw shouldn't you just be able to create a typical SSL controller service and set the "SSL Context Service" property to point to it? This should automatically use

Re: Establishment of MiNiFi repo and supporting tools

2016-03-14 Thread Aldrin Piri
Given the positive response, I have submitted requests for both a new JIRA and new Git repository which INFRA has kindly stood up for us. MiNiFi JIRA and Git are available at [1] and [2], respectively. [1] https://issues.apache.org/jira/browse/MINIFI/ [2] https://git-wip-us.apache.org/repos/asf/n

[GitHub] nifi pull request: NIFI-1464 life-cycle refactoring part-2

2016-03-14 Thread joewitt
Github user joewitt commented on a diff in the pull request: https://github.com/apache/nifi/pull/275#discussion_r56073702 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core-api/src/main/java/org/apache/nifi/controller/SchedulingAgentCallback.java ---

[GitHub] nifi pull request: NIFI-614 Added initial support for new style JM...

2016-03-14 Thread mosermw
Github user mosermw commented on the pull request: https://github.com/apache/nifi/pull/222#issuecomment-196508227 Tested using Tibco with SSL, didn't work. ERROR o.a.n.jms.processors.PublishJMS org.springframework.jms.JmsSecurityException: Can not initialize SSL client: no tr

[GitHub] nifi pull request: Nifi 1495 - AWS Kinesis Firehose

2016-03-14 Thread apiri
Github user apiri commented on the pull request: https://github.com/apache/nifi/pull/213#issuecomment-196476035 Hey @mans2singh, Just wanted to see how the progress was going. We are coming up on release time for 0.6.0 and need to start wrapping things up. If this is somethi

Re: NiFi error with embedded ZooKeeper for State Management

2016-03-14 Thread Mark Payne
Sumo, I do believe that you should be okay leaving that java.arg.15 in, in that case. The ZooKeeper Client will use the "Client" section of that file, if it is defined. In the example that you provided, that Client section does not exist, so it is okay. What would not work in this case would be

Re: NiFi error with embedded ZooKeeper for State Management

2016-03-14 Thread Sumanth Chinthagunta
Thanks Mark. I had java.arg.15 setting in bootstrap.conf from NiFi 0.4.x to make putHDFS processor work with my MapR cluster. If I remove it, putHDFS will fail. Now with NiFi 0.5.1, if I set nifi.state.management.embedded.zookeeper.start=false , can I point to dedicated external ZooKeeper in

[GitHub] nifi pull request: NIFI-1464 life-cycle refactoring part-2

2016-03-14 Thread markap14
Github user markap14 commented on a diff in the pull request: https://github.com/apache/nifi/pull/275#discussion_r56046482 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/main/java/org/apache/nifi/controller/scheduling/AbstractSchedulingAgent.jav

[GitHub] nifi pull request: NIFI-1464 life-cycle refactoring part-2

2016-03-14 Thread markap14
Github user markap14 commented on a diff in the pull request: https://github.com/apache/nifi/pull/275#discussion_r56045229 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/main/java/org/apache/nifi/controller/StandardProcessorNode.java --- @@

[GitHub] nifi pull request: NIFI-1464 life-cycle refactoring part-2

2016-03-14 Thread markap14
Github user markap14 commented on a diff in the pull request: https://github.com/apache/nifi/pull/275#discussion_r56044999 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/main/java/org/apache/nifi/controller/StandardProcessorNode.java --- @@

[GitHub] nifi pull request: NIFI-1464 life-cycle refactoring part-2

2016-03-14 Thread markap14
Github user markap14 commented on a diff in the pull request: https://github.com/apache/nifi/pull/275#discussion_r56044931 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/main/java/org/apache/nifi/controller/StandardProcessorNode.java --- @@

[GitHub] nifi pull request: NIFI-1464 life-cycle refactoring part-2

2016-03-14 Thread olegz
GitHub user olegz opened a pull request: https://github.com/apache/nifi/pull/275 NIFI-1464 life-cycle refactoring part-2 You can merge this pull request into a Git repository by running: $ git pull https://github.com/olegz/nifi NIFI-1464B Alternatively you can review and appl

Re: Split Content (One-to-Many) early commit

2016-03-14 Thread Devin Fisher
Thanks for the response. It gave my additional context such that I can proceed with more confidence. I've been playing with creating additional sessions and I believe that it will work for my use case. But I will keep the duplicate flowfile potential issue in mind, though. Thanks again. Devin On

Re: NiFi error with embedded ZooKeeper for State Management

2016-03-14 Thread Mark Payne
Sumo, If your intent is to use an external ZooKeeper, you should not be starting the embedded ZooKeeper. You will also not want to set the java.arg.15 parameter there to point to a login config file, as that is necessary only when enabling Kerberos - not for use when security disabled. So you w

NiFi error with embedded ZooKeeper for State Management

2016-03-14 Thread Sumanth Chinthagunta
I am getting following error when NiFi Cluster started with embedded ZooKeeper enabled. I would like to configure external ZooKeeper for NiFi State Management without interfering with java.security.auth.login.config setting for MapR HDFS. Can somebody provide me instructions how to configur

[GitHub] nifi pull request: NIFI-614 Added initial support for new style JM...

2016-03-14 Thread digitalplummer
Github user digitalplummer commented on the pull request: https://github.com/apache/nifi/pull/222#issuecomment-196402216 Tested on IBM MQ 7.5 and greater WebSphere Tibco --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as w

[GitHub] nifi pull request: NiFi-1481 Enhancement[ nifi.sh env]

2016-03-14 Thread trkurc
Github user trkurc commented on the pull request: https://github.com/apache/nifi/pull/218#issuecomment-196400346 @apiri - I tried last night to get the pid to get Windows to work. I could not. I say we shelve the env batch script (remove it for now from the patch set). On cy

[GitHub] nifi pull request: NiFi-1481 Enhancement[ nifi.sh env]

2016-03-14 Thread apiri
Github user apiri commented on the pull request: https://github.com/apache/nifi/pull/218#issuecomment-196384976 @trkurc What is the final verdict on handling of Windows and Cygwin? Are we punting those for now? If so, would like to provide handling for env with cygwin in nifi.sh and

[GitHub] nifi pull request: NiFi-1481 Enhancement[ nifi.sh env]

2016-03-14 Thread apiri
Github user apiri commented on the pull request: https://github.com/apache/nifi/pull/218#issuecomment-196365140 @trkurc Patch unfortunately is not applying even on a fresh checkout of the 218 PR. Will checkout your branch and work from there. Patch issue was: > $ git

[GitHub] nifi pull request: NiFi-1481 Enhancement[ nifi.sh env]

2016-03-14 Thread apiri
Github user apiri commented on the pull request: https://github.com/apache/nifi/pull/218#issuecomment-196348408 @trkurc Okay, will apply over the top and review. Thanks! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] nifi pull request: NIFI-1620 Allow empty Content-Type in InvokeHTT...

2016-03-14 Thread pvillard31
Github user pvillard31 commented on the pull request: https://github.com/apache/nifi/pull/272#issuecomment-196309847 @joewitt yes I can have a look this afternoon. I'll try to propose something before the end of the day. --- If your project is set up for it, you can reply to this ema

[GitHub] nifi pull request: NIFI-1575: Add QueryDatabaseTable processor

2016-03-14 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/261#discussion_r55995978 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/QueryDatabaseTable.java --- @@ -0,

[GitHub] nifi pull request: NIFI-1620 Allow empty Content-Type in InvokeHTT...

2016-03-14 Thread joewitt
Github user joewitt commented on the pull request: https://github.com/apache/nifi/pull/272#issuecomment-196284703 Thank you @pvillard31 . That does make sense and in the case that there is no request body being sent over I am supportive of the notion of not sending the content type h

[GitHub] nifi pull request: NIFI-1620 Allow empty Content-Type in InvokeHTT...

2016-03-14 Thread pvillard31
Github user pvillard31 commented on the pull request: https://github.com/apache/nifi/pull/272#issuecomment-196200486 Definitely agree with you that this API should not act this way. Anyway, just to clarify, this is only in the case there is no body/content sent with the reques

Re: InvokeHTTP body

2016-03-14 Thread Pierre Villard
Hi Adam, Yes it makes sense. It was just an idea for an additional possibility, not a fundamental change. Anyway, I used other processors as suggested. Thanks Adam, Pierre On Mar 13, 2016 23:52, "Adam Taft" wrote: > I think it makes total sense that POST/PUT requests read from the flowfile > co