[GitHub] nifi issue #321: Nifi 1214

Github user apiri commented on the issue: https://github.com/apache/nifi/pull/321 Hey @ToivoAdams, This is replaced by #388, correct? If so, could we please close this one out? --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] nifi issue #379: NIFI-1022 Added Tachyon/Alluxio processors

Github user apiri commented on the issue: https://github.com/apache/nifi/pull/379 Hey @pvillard31. Gave an initial scan over this code and played around a bit with Alluxio. I didn't have much success interacting with the processors but have left some notes on some of the

[GitHub] nifi pull request #379: NIFI-1022 Added Tachyon/Alluxio processors

Github user apiri commented on a diff in the pull request: https://github.com/apache/nifi/pull/379#discussion_r67610606 --- Diff: nifi-nar-bundles/nifi-alluxio-bundle/nifi-alluxio-processors/src/main/java/org/apache/nifi/processors/alluxio/GetAlluxio.java --- @@ -0,0 +1,184 @@

[GitHub] nifi pull request #379: NIFI-1022 Added Tachyon/Alluxio processors

Github user apiri commented on a diff in the pull request: https://github.com/apache/nifi/pull/379#discussion_r67610514 --- Diff: nifi-nar-bundles/nifi-alluxio-bundle/nifi-alluxio-processors/src/main/java/org/apache/nifi/processors/alluxio/AbstractAlluxioProcessor.java --- @@

[GitHub] nifi pull request #379: NIFI-1022 Added Tachyon/Alluxio processors

Github user apiri commented on a diff in the pull request: https://github.com/apache/nifi/pull/379#discussion_r67610439 --- Diff: nifi-nar-bundles/nifi-alluxio-bundle/nifi-alluxio-processors/src/main/java/org/apache/nifi/processors/alluxio/GetAlluxio.java --- @@ -0,0 +1,184 @@

[GitHub] nifi pull request #379: NIFI-1022 Added Tachyon/Alluxio processors

Github user apiri commented on a diff in the pull request: https://github.com/apache/nifi/pull/379#discussion_r67610380 --- Diff: nifi-nar-bundles/nifi-alluxio-bundle/nifi-alluxio-processors/src/main/java/org/apache/nifi/processors/alluxio/AbstractAlluxioProcessor.java --- @@

[GitHub] nifi pull request #379: NIFI-1022 Added Tachyon/Alluxio processors

Github user apiri commented on a diff in the pull request: https://github.com/apache/nifi/pull/379#discussion_r67610311 --- Diff: nifi-nar-bundles/pom.xml --- @@ -59,6 +59,7 @@ nifi-jms-bundle nifi-cassandra-bundle nifi-spring-bundle +

[GitHub] nifi pull request #379: NIFI-1022 Added Tachyon/Alluxio processors

Github user apiri commented on a diff in the pull request: https://github.com/apache/nifi/pull/379#discussion_r67609796 --- Diff: nifi-nar-bundles/nifi-alluxio-bundle/nifi-alluxio-processors/src/main/java/org/apache/nifi/processors/alluxio/GetAlluxio.java --- @@ -0,0 +1,184 @@

[GitHub] nifi pull request #379: NIFI-1022 Added Tachyon/Alluxio processors

Github user apiri commented on a diff in the pull request: https://github.com/apache/nifi/pull/379#discussion_r67609786 --- Diff: nifi-nar-bundles/nifi-alluxio-bundle/nifi-alluxio-processors/src/main/java/org/apache/nifi/processors/alluxio/AbstractAlluxioProcessor.java --- @@

[GitHub] nifi pull request #379: NIFI-1022 Added Tachyon/Alluxio processors

Github user apiri commented on a diff in the pull request: https://github.com/apache/nifi/pull/379#discussion_r67609747 --- Diff: nifi-nar-bundles/nifi-alluxio-bundle/nifi-alluxio-processors/src/main/java/org/apache/nifi/processors/alluxio/AbstractAlluxioProcessor.java --- @@

[GitHub] nifi issue #379: NIFI-1022 Added Tachyon/Alluxio processors

Github user apiri commented on the issue: https://github.com/apache/nifi/pull/379 Starting review. For anyone who was interested in checking it out, I was able to locate a Docker image that seems quite nice. You can run it locally via: `docker run -it -p

Re: Suggestion of processors

Johny, I haven't used graylog heavily so would you mind clarifying what do you mean by "Would those work like graylog also" Cheers On Sun, Jun 19, 2016 at 12:48 PM, johny casanova wrote: > Great idea! Would those work like graylog also? > On Jun 18, 2016 9:30

[GitHub] nifi pull request #377: NIFI-1805

Github user asfgit closed the pull request at: https://github.com/apache/nifi/pull/377 --- 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 #377: NIFI-1805

Github user apiri commented on the issue: https://github.com/apache/nifi/pull/377 @jdye64 Apologies for delays. These changes look fine and everything builds fine. As far as the other commentary on this thread, I will leave that to folks to generate other issues to track, but these

Re: Suggestion of processors

Great idea! Would those work like graylog also? On Jun 18, 2016 9:30 PM, "Andre" wrote: > Devs, > > I am continuing to drive the migration of our logging pipeline to NiFi and > in the process identified some areas of log processing that could be > improved by the

Suggestion of processors

Devs, I am continuing to drive the migration of our logging pipeline to NiFi and in the process identified some areas of log processing that could be improved by the introduction of new processors. I wonder Would anyone oppose the idea of introducing the following processors: 1. ParseCEF

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

Github user markap14 commented on the issue: https://github.com/apache/nifi/pull/501 Yolanda - I have merged to 0.x. This was very well done. I like the class hierarchy you put together - the breakout of interfaces, abstract classes, and concrete classes. I think it will yield itself

Re: patch for NIFI-1828

Vincent, Thank you for your contribution. Welcome to NiFi. 1828 is marked as a duplicate of NIFI-1760 [1]. Can you please update that ticket with your patch? There are helpful resources for getting started with contributing available here [2]. If you have any other questions, please ask on the

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

Github user markap14 commented on the issue: https://github.com/apache/nifi/pull/501 @YolandaMDavis All looks good with the exception of needed to trim() the filenames and one other thing that i noticed, which is that if multiple properties files are specified and one cannot be

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

Github user markap14 commented on a diff in the pull request: https://github.com/apache/nifi/pull/501#discussion_r67608495 --- Diff: nifi-api/src/test/resources/TestVariableRegistry/foobar.properties --- @@ -0,0 +1 @@ +fake.property.3=test me out 3, test me out 4 --- End

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

Github user markap14 commented on a diff in the pull request: https://github.com/apache/nifi/pull/501#discussion_r67608449 --- Diff: nifi-api/src/test/resources/TestVariableRegistry/foobar.properties --- @@ -0,0 +1 @@ +fake.property.3=test me out 3, test me out 4 --- End

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

Github user markap14 commented on a diff in the pull request: https://github.com/apache/nifi/pull/501#discussion_r67608435 --- Diff: nifi-commons/nifi-properties/src/main/java/org/apache/nifi/util/NiFiProperties.java --- @@ -1073,4 +1078,28 @@ public File

[GitHub] nifi issue #93: NIFI-991: Add "upsert" verb support for ConvertJSONToSQL pro...

Github user joshelser commented on the issue: https://github.com/apache/nifi/pull/93 > closing after suggestion to use existing processor in input mode with a downstream ReplaceText Processor(replace "INSERT" with "UPSERT". Where was this suggestion, @randerzander? Any

patch for NIFI-1828

I wanted to crawl into contributing to nifi so I created a patch for NIFI-1828. https://issues.apache.org/jira/browse/NIFI-1828 -Vincent 

[GitHub] nifi issue #335: Adjust batch.size property description and validation.

Github user apiri commented on the issue: https://github.com/apache/nifi/pull/335 Hey @owieboy, Just been doing some maintenance and wanted to check in to see if the PR changes of #366 satisfy the efforts for this PR. If so, could you please close this item out and if not,

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

Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/248#discussion_r67605391 --- 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 #248: NIFI-1568: Add Filter Capability to UnpackContent

Github user joewitt commented on a diff in the pull request: https://github.com/apache/nifi/pull/248#discussion_r67604647 --- 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 #248: NIFI-1568: Add Filter Capability to UnpackContent

Github user rickysaltzer commented on a diff in the pull request: https://github.com/apache/nifi/pull/248#discussion_r67604557 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/UnpackContent.java --- @@

[GitHub] nifi pull request #272: NIFI-1620 Allow empty Content-Type in InvokeHTTP pro...

Github user asfgit closed the pull request at: https://github.com/apache/nifi/pull/272 --- 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 #272: NIFI-1620 Allow empty Content-Type in InvokeHTTP processor

Github user apiri commented on the issue: https://github.com/apache/nifi/pull/272 @taftster @JPercivall I'll aid in the assist and wrap this up as I have been scanning through PRs and can merge in. --- If your project is set up for it, you can reply to this email and have your reply

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

And if I my poor JAVA hasn't failed me again this seems to be the commit that introduced the upstream change https://github.com/apache/hadoop/commit/6e13fc62e1f284f22fd0089f06ce281198bc7c2a https://issues.apache.org/jira/browse/HDFS-7513 On Sun, Jun 19, 2016 at 1:36 AM, Andre

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

Pierre, My build is currently hitting a wall when trying to compile nifi-hdfs-processors. Seems the reason for this is the commit 2ce7857. Both $ mvn clean install -Dhadoop.version=2.7.0 and $ mvn clean install -DskipTests -Pmapr -Dhadoop.version=2.7.0-mapr-1506 result in: [ERROR]

[GitHub] nifi issue #483: NIFI-1899 - Introduce ExtractEmailAttachments and ExtractEm...

Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/483 @joewitt any comments regarding this PR? Cheers --- 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

[GitHub] nifi issue #534: Fix for NIFI-1838 & NIFI-1152

Github user pvillard31 commented on the issue: https://github.com/apache/nifi/pull/534 thanks @PuspenduBanerjee, I'm quite busy this WE, will have another look asap --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If