[GitHub] nifi pull request #543: NIFI-1834 Create PutTCP Processor

2016-06-20 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/543#discussion_r67707758 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/PutTCP.java --- @@ -0,0 +1,270

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

2016-06-20 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/nifi/pull/248 --- 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 issue #525: NIFI-1976 - Windows Event Log native processor

2016-06-20 Thread brosander
Github user brosander commented on the issue: https://github.com/apache/nifi/pull/525 @JPercivall I believe I have incorporated all of the suggested changes (except for the XPath validator because it didn't like some of the provided examples in the api doc). Mind taking another

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

2016-06-20 Thread Joe Percivall
Team, We are doing a great job finishing up tickets. Six more were closed over the weekend and most of the others made progress as well. We only have three more that are tagged as 0.7.0 in Jira. There are a couple others which are unversioned that are close but haven't been identified as

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

2016-06-20 Thread Bryan Bende
I believe NIFI-1895 should be ready to go for 0.7.0 as well... https://github.com/apache/nifi/pull/542 On Mon, Jun 20, 2016 at 11:39 AM, Joe Percivall < joeperciv...@yahoo.com.invalid> wrote: > Team, > > We are doing a great job finishing up tickets. Six more were closed over > the weekend and

[GitHub] nifi pull request #542: NIFI-1895 for 0.x

2016-06-20 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/542#discussion_r67713339 --- Diff: nifi-nar-bundles/nifi-hbase-bundle/nifi-hbase-processors/src/main/java/org/apache/nifi/hbase/PutHBaseJSON.java --- @@ -89,6 +89,25 @@

[GitHub] nifi issue #542: NIFI-1895 for 0.x

2016-06-20 Thread rtempleton
Github user rtempleton commented on the issue: https://github.com/apache/nifi/pull/542 Good point, thanks @bbende, I hadn't thought to leave a backward compatibility feature in there. --- If your project is set up for it, you can reply to this email and have your reply appear on

ControllerService Enabling and Processor onPropertyModified

2016-06-20 Thread Michael D. Coon
All,    Before I get too deep in submitting Jira tickets, etc. I'm wondering if this is expected behavior. I'm using NiFi 0.6.1.    I have a ControllerService that I reference as a service property on my Processor. The Processor, in turn, uses the ControllerService's internal configuration

[GitHub] nifi pull request #543: NIFI-1834 Create PutTCP Processor

2016-06-20 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/543#discussion_r67707310 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/PutTCP.java --- @@ -0,0 +1,270

[GitHub] nifi issue #398: NIFI-1751 Added proxy authentication in InvokeHttp processo...

2016-06-20 Thread apiri
Github user apiri commented on the issue: https://github.com/apache/nifi/pull/398 Actually, I think the Response#priorResponse probably doesn't apply to our case after thinking about it a bit more. The key way to avoid the issues seen is a flag/counter within that authenticator to

[GitHub] nifi pull request #542: NIFI-1895 for 0.x

2016-06-20 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/542#discussion_r67714535 --- Diff: nifi-nar-bundles/nifi-hbase-bundle/nifi-hbase-processors/src/main/java/org/apache/nifi/hbase/PutHBaseJSON.java --- @@ -89,6 +89,25 @@

[GitHub] nifi pull request #542: NIFI-1895 for 0.x

2016-06-20 Thread bbende
Github user bbende commented on a diff in the pull request: https://github.com/apache/nifi/pull/542#discussion_r67718748 --- Diff: nifi-nar-bundles/nifi-hbase-bundle/nifi-hbase-processors/src/main/java/org/apache/nifi/hbase/PutHBaseJSON.java --- @@ -89,6 +89,25 @@

[GitHub] nifi pull request #543: NIFI-1834 Create PutTCP Processor

2016-06-20 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/543#discussion_r67722269 --- Diff: nifi-commons/nifi-processor-utilities/src/main/java/org/apache/nifi/processor/util/put/AbstractPutEventProcessor.java --- @@ -119,7 +119,24 @@

[GitHub] nifi issue #362: NIFI-1769: added support for SSE-KMS and signature s3v4 aut...

2016-06-20 Thread jvwing
Github user jvwing commented on the issue: https://github.com/apache/nifi/pull/362 @miquillo, thanks for putting together this PR, Server-Side Encryption with customer keys is a great feature for NiFi to have. I'm doing some review, and considering the following topics: *

Re: HDFS inotify commit to nifi-hdfs-processors seems to break MapR "profile" / Hadoop 2.7(?)

2016-06-20 Thread Pierre Villard
Thanks Andre for reporting this! I'll try to have a look today but my MapR knowledge is limited... However, since it seems to be a problem in test classes only, it should be possible to find a workaround quite easily. Pierre 2016-06-18 17:43 GMT+02:00 Andre : > And if I my

[GitHub] nifi issue #398: NIFI-1751 Added proxy authentication in InvokeHttp processo...

2016-06-20 Thread pvillard31
Github user pvillard31 commented on the issue: https://github.com/apache/nifi/pull/398 Hey @apiri, Many thanks for the review! Regarding HTTPS, at the time I submitted the PR I was testing it with CCProxy (on Windows computer). I can't test it again with this setup

[GitHub] nifi pull request #545: NIFI-2009 port for 0.x

2016-06-20 Thread olegz
Github user olegz closed the pull request at: https://github.com/apache/nifi/pull/545 --- 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 #543: NIFI-1834 Create PutTCP Processor

2016-06-20 Thread bbende
Github user bbende commented on a diff in the pull request: https://github.com/apache/nifi/pull/543#discussion_r67698814 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/PutTCP.java --- @@ -0,0 +1,270 @@

[GitHub] nifi issue #397: NIFI-1815

2016-06-20 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/397 back to reviewing this. It has some merge conflicts but i'll fox them as I go along --- 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 #543: NIFI-1834 Create PutTCP Processor

2016-06-20 Thread bbende
Github user bbende commented on a diff in the pull request: https://github.com/apache/nifi/pull/543#discussion_r67698292 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/PutTCP.java --- @@ -0,0 +1,270 @@

[GitHub] nifi pull request #543: NIFI-1834 Create PutTCP Processor

2016-06-20 Thread bbende
Github user bbende commented on a diff in the pull request: https://github.com/apache/nifi/pull/543#discussion_r67696982 --- Diff: nifi-commons/nifi-processor-utilities/src/main/java/org/apache/nifi/processor/util/put/AbstractPutEventProcessor.java --- @@ -119,7 +119,24 @@

[GitHub] nifi pull request #543: NIFI-1834 Create PutTCP Processor

2016-06-20 Thread bbende
Github user bbende commented on a diff in the pull request: https://github.com/apache/nifi/pull/543#discussion_r67700294 --- Diff: nifi-commons/nifi-processor-utilities/src/main/java/org/apache/nifi/processor/util/put/AbstractPutEventProcessor.java --- @@ -126,8 +127,9 @@

[GitHub] nifi pull request #543: NIFI-1834 Create PutTCP Processor

2016-06-20 Thread bbende
Github user bbende commented on a diff in the pull request: https://github.com/apache/nifi/pull/543#discussion_r67697377 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/PutTCP.java --- @@ -0,0 +1,270 @@

[GitHub] nifi issue #545: NIFI-2009 port for 0.x

2016-06-20 Thread pvillard31
Github user pvillard31 commented on the issue: https://github.com/apache/nifi/pull/545 Thanks @olegz! I just merged it into 0.x, could you close the PR? --- 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 #546: NIFI-1877, NIFI-1306: Add fields to FlowFile for FIF...

2016-06-20 Thread markap14
GitHub user markap14 opened a pull request: https://github.com/apache/nifi/pull/546 NIFI-1877, NIFI-1306: Add fields to FlowFile for FIFO Prioritizer, Ol… …dest/Newest FlowFile first prioritizers to work property You can merge this pull request into a Git repository by running:

[GitHub] nifi pull request #542: NIFI-1895 for 0.x

2016-06-20 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/nifi/pull/542 --- 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 issue #543: NIFI-1834 Create PutTCP Processor

2016-06-20 Thread bbende
Github user bbende commented on the issue: https://github.com/apache/nifi/pull/543 Looks good to me +1 --- 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 pull request #541: NIFI-2032 fixed 'enableControllerServices' logic

2016-06-20 Thread alopresto
Github user alopresto commented on a diff in the pull request: https://github.com/apache/nifi/pull/541#discussion_r67738328 --- Diff:

[GitHub] nifi pull request #542: NIFI-1895 for 0.x

2016-06-20 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/542#discussion_r67723205 --- Diff: nifi-nar-bundles/nifi-hbase-bundle/nifi-hbase-processors/src/main/java/org/apache/nifi/hbase/PutHBaseJSON.java --- @@ -89,6 +89,25 @@

Re: HDFS inotify commit to nifi-hdfs-processors seems to break MapR "profile" / Hadoop 2.7(?)

2016-06-20 Thread Pierre Villard
The way events are instantiated is not the same between 2.6.x and 2.7.x. The easiest solution is to simplify unit tests to have tests both compatible with 2.6 and 2.7 lines. I can submit a PR for that. The other option would be to upgrade the version of Hadoop libraries used in NiFi. However,

[GitHub] nifi pull request #543: NIFI-1834 Create PutTCP Processor

2016-06-20 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/543#discussion_r67732780 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/PutTCP.java --- @@ -0,0 +1,290 @@

[GitHub] nifi pull request #547: NIFI-2060 - fixed unit tests to be compatible with 2...

2016-06-20 Thread pvillard31
GitHub user pvillard31 opened a pull request: https://github.com/apache/nifi/pull/547 NIFI-2060 - fixed unit tests to be compatible with 2.7.x It’s a shame to limit what is tested here (in terms of event types) but it seems to be the only way to have both 2.6 and 2.7 lines

[GitHub] nifi issue #513: PutHBaseJSON processor treats all values as Strings

2016-06-20 Thread bbende
Github user bbende commented on the issue: https://github.com/apache/nifi/pull/513 @rtempleton just merged this work as part of PR 542, thanks for the contribution! can we close this PR? --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] nifi pull request #459: NIFI-1909 Adding ability to process schemaless Avro ...

2016-06-20 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/nifi/pull/459 --- 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 #541: NIFI-2032 fixed 'enableControllerServices' logic

2016-06-20 Thread mcgilman
Github user mcgilman commented on a diff in the pull request: https://github.com/apache/nifi/pull/541#discussion_r67739510 --- Diff:

[GitHub] nifi issue #542: NIFI-1895 for 0.x

2016-06-20 Thread JPercivall
Github user JPercivall commented on the issue: https://github.com/apache/nifi/pull/542 Specifically looking at Bryan's commit: I looked through the code and built locally (since travis failed for un-related issues) and it looks good. +1 --- If your project is set up for

[GitHub] nifi pull request #543: NIFI-1834 Create PutTCP Processor

2016-06-20 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/543#discussion_r67732149 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/PutTCP.java --- @@ -0,0 +1,290 @@

[GitHub] nifi pull request #543: NIFI-1834 Create PutTCP Processor

2016-06-20 Thread joewitt
Github user joewitt commented on a diff in the pull request: https://github.com/apache/nifi/pull/543#discussion_r67732066 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/PutTCP.java --- @@ -0,0 +1,290 @@

[GitHub] nifi issue #459: NIFI-1909 Adding ability to process schemaless Avro records...

2016-06-20 Thread markap14
Github user markap14 commented on the issue: https://github.com/apache/nifi/pull/459 @ryanpersaud thanks for getting this in! I have verified the checkstyle and pushed to both the 0.x and 1.0.0 baselines. Thanks again for knocking this out! --- If your project is set up for it, you

[GitHub] nifi pull request #541: NIFI-2032 fixed 'enableControllerServices' logic

2016-06-20 Thread mcgilman
Github user mcgilman commented on a diff in the pull request: https://github.com/apache/nifi/pull/541#discussion_r67739760 --- Diff:

[GitHub] nifi issue #398: NIFI-1751 Added proxy authentication in InvokeHttp processo...

2016-06-20 Thread pvillard31
Github user pvillard31 commented on the issue: https://github.com/apache/nifi/pull/398 Yep, not sure it's worth it. I think it is fine to let the PR as is. --- 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 #543: NIFI-1834 Create PutTCP Processor

2016-06-20 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/543#discussion_r67731299 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/PutTCP.java --- @@ -0,0 +1,290 @@

[GitHub] nifi pull request #543: NIFI-1834 Create PutTCP Processor

2016-06-20 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/543#discussion_r67732631 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/test/java/org/apache/nifi/processors/standard/TestPutTCP.java --- @@ -0,0 +1,48 @@

Using NiFi S3 Processors with S3-Compatible Services?

2016-06-20 Thread James Wing
Is anyone using NiFi's S3 processors to work with non-Amazon, S3-compatible services? Would you be willing to share which service you are using and if you have experienced compatibility issues? Are you aware of the difference between S3's API signature versions 2 and 4? NiFi's current default

[GitHub] nifi pull request #548: NIFI-2045 - Removing mqtt message from queue after a...

2016-06-20 Thread brosander
GitHub user brosander opened a pull request: https://github.com/apache/nifi/pull/548 NIFI-2045 - Removing mqtt message from queue after after session.comit() You can merge this pull request into a Git repository by running: $ git pull https://github.com/brosander/nifi

Re: ControllerService Enabling and Processor onPropertyModified

2016-06-20 Thread Andy LoPresto
I am reviewing PR 541 [1] for that bug right now. [1] https://github.com/apache/nifi/pull/541 Andy LoPresto alopre...@apache.org alopresto.apa...@gmail.com PGP Fingerprint: 70EC B3E5 98A6 5A3F D3C4 BACE 3C6E F65B 2F7D EF69 > On Jun 20, 2016, at 1:25

[GitHub] nifi pull request #543: NIFI-1834 Create PutTCP Processor

2016-06-20 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/543#discussion_r67733058 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/PutTCP.java --- @@ -0,0 +1,290 @@

Re: ControllerService Enabling and Processor onPropertyModified

2016-06-20 Thread Michael Moser
Michael, You may be encountering the bug NIFI-2032 [1] which exists in NiFi 0.6.1. [1] - https://issues.apache.org/jira/browse/NIFI-2032 -- Mike On Mon, Jun 20, 2016 at 12:20 PM, Michael D. Coon wrote: > All, >Before I get too deep in submitting Jira

[GitHub] nifi pull request #541: NIFI-2032 fixed 'enableControllerServices' logic

2016-06-20 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/nifi/pull/541 --- 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 #550: NIFI-1900: Verify that connection's destination is n...

2016-06-20 Thread markap14
GitHub user markap14 opened a pull request: https://github.com/apache/nifi/pull/550 NIFI-1900: Verify that connection's destination is not running when t… …rying to change destination You can merge this pull request into a Git repository by running: $ git pull

[GitHub] nifi pull request #549: NIFI-2046 - Adding reference to @TriggerSerially in ...

2016-06-20 Thread brosander
GitHub user brosander opened a pull request: https://github.com/apache/nifi/pull/549 NIFI-2046 - Adding reference to @TriggerSerially in Processor javadoc You can merge this pull request into a Git repository by running: $ git pull https://github.com/brosander/nifi NIFI-2046

[GitHub] nifi pull request #541: NIFI-2032 fixed 'enableControllerServices' logic

2016-06-20 Thread alopresto
Github user alopresto commented on a diff in the pull request: https://github.com/apache/nifi/pull/541#discussion_r67740493 --- Diff:

[GitHub] nifi pull request #517: NIFI-1994: Fixed issues with controller services and...

2016-06-20 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/nifi/pull/517 --- 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 #543: NIFI-1834 Create PutTCP Processor

2016-06-20 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/543#discussion_r67800576 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/PutTCP.java --- @@ -0,0 +1,290

[GitHub] nifi pull request #543: NIFI-1834 Create PutTCP Processor

2016-06-20 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/543#discussion_r67801359 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/PutTCP.java --- @@ -0,0 +1,290

[GitHub] nifi issue #541: NIFI-2032 fixed 'enableControllerServices' logic

2016-06-20 Thread alopresto
Github user alopresto commented on the issue: https://github.com/apache/nifi/pull/541 I merged into master but I've had to rebase and resolve conflicts 3 times on `0.x` branch and still have tests failing. Will continue working on that part in the morning. --- If your project is

[GitHub] nifi issue #362: NIFI-1769: added support for SSE-KMS and signature s3v4 aut...

2016-06-20 Thread jvwing
Github user jvwing commented on the issue: https://github.com/apache/nifi/pull/362 I now better understand that we do not need to do anything to the FetchS3Object processor to read KMS encrypted files. Server-Side Encryption is, after all, on the server. Strike that point. --- If

[GitHub] nifi issue #362: NIFI-1769: added support for SSE-KMS and signature s3v4 aut...

2016-06-20 Thread jvwing
Github user jvwing commented on the issue: https://github.com/apache/nifi/pull/362 @miquillo , what do you think about using

[GitHub] nifi pull request #511: NIFI-1850 - JSON-to-JSON Schema Converter Editor

2016-06-20 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/nifi/pull/511 --- 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 issue #511: NIFI-1850 - JSON-to-JSON Schema Converter Editor

2016-06-20 Thread mcgilman
Github user mcgilman commented on the issue: https://github.com/apache/nifi/pull/511 Looks great @YolandaMDavis I'm merged this into master. 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 your

[GitHub] nifi pull request #543: NIFI-1834 Create PutTCP Processor

2016-06-20 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/543#discussion_r67799164 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/PutTCP.java --- @@ -0,0 +1,290

[GitHub] nifi issue #252: NIFI-615 - Create a processor to extract WAV file character...

2016-06-20 Thread JPercivall
Github user JPercivall commented on the issue: https://github.com/apache/nifi/pull/252 @jskora were you able to rebase and open a new PR? --- 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 issue #517: NIFI-1994: Fixed issues with controller services and templa...

2016-06-20 Thread mcgilman
Github user mcgilman commented on the issue: https://github.com/apache/nifi/pull/517 Looks good @markap14 Had one minor issue ensuring controller services are copied as part of a process group dto. I tweak your commit to include the additionally functionality. --- If your

[GitHub] nifi pull request #543: NIFI-1834 Create PutTCP Processor

2016-06-20 Thread joewitt
Github user joewitt commented on a diff in the pull request: https://github.com/apache/nifi/pull/543#discussion_r67799555 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/PutTCP.java --- @@ -0,0 +1,290 @@

[GitHub] nifi pull request #543: NIFI-1834 Create PutTCP Processor

2016-06-20 Thread joewitt
Github user joewitt commented on a diff in the pull request: https://github.com/apache/nifi/pull/543#discussion_r67800869 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/PutTCP.java --- @@ -0,0 +1,290 @@

[GitHub] nifi issue #541: NIFI-2032 fixed 'enableControllerServices' logic

2016-06-20 Thread alopresto
Github user alopresto commented on the issue: https://github.com/apache/nifi/pull/541 Thanks @olegz . I will build locally 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 not have this

0.7.0 install bug centos 7

2016-06-20 Thread Ryan H
Looks like 0.7.0 doesn't install as a service quite right on linux/centos 7. I untar'd it in: #/opt/nifi/current -> nifi-0.7.0-SNAPSHOT I followed these steps: #/opt/nifi/current/bin/nifi.sh install #chkconfig nifi on #service nifi start That copies the nifi.sh script into /etc/init.d

Re: ControllerService Enabling and Processor onPropertyModified

2016-06-20 Thread Oleg Zhurakousky
Indeed this sounds like https://issues.apache.org/jira/browse/NIFI-2032. The PR for it has been out https://github.com/apache/nifi/pull/541 so if you get a chance to try it please let us know. Hopefully it will be merged soon. Cheers Oleg On Jun 20, 2016, at 4:25 PM, Michael Moser

[GitHub] nifi issue #541: NIFI-2032 fixed 'enableControllerServices' logic

2016-06-20 Thread alopresto
Github user alopresto commented on the issue: https://github.com/apache/nifi/pull/541 Getting a compilation failure missing `import org.apache.nifi.reporting.BulletinFactory`. Did this get missed in the second commit? --- If your project is set up for it, you can reply to this

[GitHub] nifi pull request #541: NIFI-2032 fixed 'enableControllerServices' logic

2016-06-20 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/541#discussion_r67766144 --- Diff:

[GitHub] nifi pull request #543: NIFI-1834 Create PutTCP Processor

2016-06-20 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/543#discussion_r6337 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/test/java/org/apache/nifi/processors/standard/TestPutTCP.java --- @@ -0,0

[GitHub] nifi pull request #543: NIFI-1834 Create PutTCP Processor

2016-06-20 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/543#discussion_r67778279 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/PutTCP.java --- @@ -0,0 +1,290 @@

[GitHub] nifi pull request #543: NIFI-1834 Create PutTCP Processor

2016-06-20 Thread bbende
Github user bbende commented on a diff in the pull request: https://github.com/apache/nifi/pull/543#discussion_r67779821 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/PutTCP.java --- @@ -0,0 +1,290 @@

[GitHub] nifi issue #541: NIFI-2032 fixed 'enableControllerServices' logic

2016-06-20 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/541 @alopresto so I just fixed it and squashed it with the last commit. No compilation error on my end. LMK --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] nifi pull request #543: NIFI-1834 Create PutTCP Processor

2016-06-20 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/543#discussion_r67776824 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/PutTCP.java --- @@ -0,0 +1,290

[GitHub] nifi issue #541: NIFI-2032 fixed 'enableControllerServices' logic

2016-06-20 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/541 I see, it appears that I have to reference different ```org.apache.nifi.events.BulletinFactory```, building now --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] nifi pull request #543: NIFI-1834 Create PutTCP Processor

2016-06-20 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/543#discussion_r67779956 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/test/java/org/apache/nifi/processors/standard/TestPutTCP.java --- @@ -0,0 +1,48 @@

[GitHub] nifi pull request #543: NIFI-1834 Create PutTCP Processor

2016-06-20 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/543#discussion_r67780170 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/PutTCP.java --- @@ -0,0 +1,290 @@

Re: 0.7.0 install bug centos 7

2016-06-20 Thread James Wing
Thanks for reporting this, Ryan, I see the same behavior. I created a JIRA for this issue (https://issues.apache.org/jira/browse/NIFI-2063). If you need a workaround, you can edit /etc/init.d/nifi to hard-code the path to the script directory: SCRIPT_DIR=/opt/nifi/current/bin

[GitHub] nifi pull request #543: NIFI-1834 Create PutTCP Processor

2016-06-20 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/543#discussion_r67782856 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/PutTCP.java --- @@ -0,0 +1,290

[GitHub] nifi pull request #543: NIFI-1834 Create PutTCP Processor

2016-06-20 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/543#discussion_r67782966 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/PutTCP.java --- @@ -0,0 +1,290

[GitHub] nifi pull request #543: NIFI-1834 Create PutTCP Processor

2016-06-20 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/543#discussion_r67783621 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/PutTCP.java --- @@ -0,0 +1,290 @@

Re: 0.7.0 install bug centos 7

2016-06-20 Thread Nicholas Hughes
FWIW, I "fixed" this on my system with a different workaround: 1. Don't use the "install" option. 2. Symlink /etc/init.d/nifi to /opt/nifi/current/bin/nifi.sh 3. Edit the script to use readlink and find the actual install directory: SCRIPT_DIR=$(dirname $(readlink $0 || echo $0))