[GitHub] nifi issue #1418: NIFI-3238 - Introduce ListenBeats processor and deprecates...

2017-02-07 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1418 For future reference: https://discuss.elastic.co/t/testers-wanted-beat-integration-with-apache-nifi/73671/ --- If your project is set up for it, you can reply to this email and have your

[GitHub] nifi issue #1418: NIFI-3238 - Introduce ListenBeats processor and deprecates...

2017-02-07 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1418 @nickcarenza thanks for the feedback --- 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 issue #1480: NIFI-1657 - Configure travis-ci to run mvn verify in jp an...

2017-02-07 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1480 @pvillard31 seems good to go. failures were caused by TCP processors --- 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 issue #1480: NIFI-1657 - Configure travis-ci to run mvn verify in jp an...

2017-02-07 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1480 Only if you commit to port NiFi to minitel. :rofl: ... Just kidding, done --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] nifi issue #1480: NIFI-1657 - Configure travis-ci to run mvn verify in jp an...

2017-02-07 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1480 by the way, just in case someone ever wonder the rationale behind the countries: basic English (Java's default env), a Latin and a "Asian" language with no official status at the UN. -

[GitHub] nifi issue #1480: NIFI-1657 - Configure travis-ci to run mvn verify in jp an...

2017-02-07 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1480 @pvillard31 good catch! --- 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

[GitHub] nifi issue #1480: NIFI-1657 - Configure travis-ci to run mvn verify in jp an...

2017-02-07 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1480 @pvillard31 I changed the commit a bit so to use envs (I think things should move faster this way) I reckon we should merge and use it to correct residual issues. This second attempt should hopefully

[GitHub] nifi pull request #1480: NIFI-1657 - Configure travis-ci to run mvn verify i...

2017-02-07 Thread trixpan
GitHub user trixpan opened a pull request: https://github.com/apache/nifi/pull/1480 NIFI-1657 - Configure travis-ci to run mvn verify in jp and fr Thank you for submitting a contribution to Apache NiFi. In order to streamline the review of the contribution we ask you

[GitHub] nifi issue #1472: NIFI-3434 - Introduce the ability to skip the packaging of...

2017-02-06 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1472 @pvillard31 @apiri done. Please note I added a second commit as this helps keeping track of why the travis file was changed. --- If your project is set up for it, you can reply

[GitHub] nifi issue #1108: NIFI-2565: add Grok parser

2017-02-06 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1108 @joewitt when you have time can you have a final look on this one? LGTM and I am happy to address the two remaining cosmetic comments as part of a separate PR. --- If your project is set up

[GitHub] nifi issue #1474: NIFI-3442 - Explictly support EL on Username, password, po...

2017-02-06 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1474 @markap14 you may want to have a look note that in the case of transfer method I simply removed the EL evaluation. Happy to follow the opposite approach if needed --- If your project is set

[GitHub] nifi issue #1473: NIFI-940 - Exposes filename attribute to DFM on PutFile, P...

2017-02-05 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1473 still not ready to commit as sftp and ftp modification have not yet been added to commit. Hope to finish those in the next few days. --- If your project is set up for it, you can reply to this email

[GitHub] nifi pull request #1473: NIFI-940 - Exposes filename attribute to DFM on Put...

2017-02-05 Thread trixpan
GitHub user trixpan opened a pull request: https://github.com/apache/nifi/pull/1473 NIFI-940 - Exposes filename attribute to DFM on PutFile, PutHDFS, Put… …SFTP, PutFTP Thank you for submitting a contribution to Apache NiFi. In order to streamline the review

[GitHub] nifi issue #1467: NIFI-3408 Add exception class when InvokeHTTP fails

2017-02-03 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1467 @gglanzani - thank you for the contribution. Code looks great but would it be possible for you to extend the test harness to include a test that will trigger your code(i.e. failure

[GitHub] nifi pull request #1472: NIFI-3434 - Introduce the ability to skip the packa...

2017-02-03 Thread trixpan
GitHub user trixpan opened a pull request: https://github.com/apache/nifi/pull/1472 NIFI-3434 - Introduce the ability to skip the packaging of zip and ta… …r.gz archive via maven profile (i.e. -Pdir-only) Thank you for submitting a contribution to Apache NiFi

[GitHub] nifi issue #1468: NIFI-3430 adds support for date format property to parse i...

2017-02-02 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1468 Nick, thank you for the contribution. would you mind squashing the commits? --- 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 issue #1462: NIFI-819 - Extend GetHTTP to use dynamically add HTTP head...

2017-02-02 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1462 @apiri just pushed a refreshed version of the code. let me know how it goes. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] nifi pull request #1462: NIFI-819 - Extend GetHTTP to use dynamically add HT...

2017-02-01 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/1462#discussion_r99039594 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/GetHTTP.java --- @@ -102,6 +96,11

[GitHub] nifi pull request #1462: NIFI-819 - Extend GetHTTP to use dynamically add HT...

2017-02-01 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/1462#discussion_r99039524 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/GetHTTP.java --- @@ -62,18 +62,12

[GitHub] nifi pull request #1462: NIFI-819 - Extend GetHTTP to use dynamically add HT...

2017-02-01 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/1462#discussion_r99038757 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/GetHTTP.java --- @@ -467,6 +486,18

[GitHub] nifi pull request #1462: NIFI-819 - Extend GetHTTP to use dynamically add HT...

2017-02-01 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/1462#discussion_r99038321 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/GetHTTP.java --- @@ -467,6 +486,18

[GitHub] nifi pull request #1462: NIFI-819 - Extend GetHTTP to use dynamically add HT...

2017-02-01 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/1462#discussion_r99038165 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/GetHTTP.java --- @@ -102,6 +96,11

[GitHub] nifi pull request #1462: NIFI-819 - Extend GetHTTP to use dynamically add HT...

2017-02-01 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/1462#discussion_r99038126 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/GetHTTP.java --- @@ -102,6 +96,11

[GitHub] nifi pull request #1462: NIFI-819 - Extend GetHTTP to use dynamically add HT...

2017-02-01 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/1462#discussion_r99038013 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/GetHTTP.java --- @@ -62,18 +62,12

[GitHub] nifi pull request #1462: NIFI-819 - Extend GetHTTP to use dynamically add HT...

2017-02-01 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/1462#discussion_r99038019 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/GetHTTP.java --- @@ -62,18 +62,12

[GitHub] nifi issue #1462: NIFI-819 - Extend GetHTTP to use dynamically produce HTTP ...

2017-02-01 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1462 @apiri A long time ago in a galaxy far far away you raised this JIRA ticket. Does this PR addresses what you had in mind? Cheers --- If your project is set up for it, you can

[GitHub] nifi pull request #1462: NIFI-819 - Extend GetHTTP to use dynamically produc...

2017-02-01 Thread trixpan
GitHub user trixpan opened a pull request: https://github.com/apache/nifi/pull/1462 NIFI-819 - Extend GetHTTP to use dynamically produce HTTP headers upo… …n a request Thank you for submitting a contribution to Apache NiFi. In order to streamline the review

[GitHub] nifi issue #1016: NIFI-2724 New JMX Processor

2017-02-01 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1016 @brianburnett can't you simply rebase? 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-site pull request #11: NIFI-1448 - Update NiFi and MiNiFi site to use t...

2017-02-01 Thread trixpan
GitHub user trixpan opened a pull request: https://github.com/apache/nifi-site/pull/11 NIFI-1448 - Update NiFi and MiNiFi site to use the new ASF branding @apiri can you have a look at this PR? You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] nifi issue #1421: NIFI-3352 - Address issue where ListenLumberjack was sendi...

2017-01-24 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1421 @joewitt sure thing: https://github.com/elastic/logstash-forwarder/blob/master/PROTOCOL.md#ack-frame-type --- If your project is set up for it, you can reply to this email

[GitHub] nifi pull request #1422: NIFI-3015 - Corrects issue were RPM profile run.as ...

2017-01-23 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/1422#discussion_r97304871 --- Diff: nifi-assembly/pom.xml --- @@ -19,6 +19,10 @@ language governing permissions and limitations under the License. --> nifi-assem

[GitHub] nifi pull request #1422: NIFI-3015 - Corrects issue were RPM profile run.as ...

2017-01-19 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/1422#discussion_r96984003 --- Diff: nifi-assembly/pom.xml --- @@ -19,6 +19,10 @@ language governing permissions and limitations under the License. --> nifi-assem

[GitHub] nifi issue #1402: NIFI-3288: Add Clojure support for ExecuteScript

2017-01-18 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1402 LGTM --- 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

[GitHub] nifi pull request #1422: NIFI-3015 - Corrects issue were RPM profile run.as ...

2017-01-16 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/1422#discussion_r96322520 --- Diff: nifi-assembly/pom.xml --- @@ -19,6 +19,10 @@ language governing permissions and limitations under the License. --> nifi-assem

[GitHub] nifi pull request #1422: NIFI-3015 - Corrects issue were RPM profile run.as ...

2017-01-16 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/1422#discussion_r96312325 --- Diff: nifi-assembly/pom.xml --- @@ -19,6 +19,10 @@ language governing permissions and limitations under the License. --> nifi-assem

[GitHub] nifi issue #1422: NIFI-3015 - Corrects issue were RPM profile run.as propert...

2017-01-16 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1422 @fidget83 and @jfrazee mind having a look and let me know if it addresses the issue? Cheers --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] nifi issue #1422: NIFI-3015 - Corrects issue were RPM profile run.as propert...

2017-01-16 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1422 Reviewers: Key here is that bootstrap.conf with RPM profile should result on `run.as=nifi` while vanilla build should result in `run.as=` (naturally assuming nothing else breaks

[GitHub] nifi pull request #1422: NIFI-3015 - Corrects issue were RPM profile run.as ...

2017-01-16 Thread trixpan
GitHub user trixpan opened a pull request: https://github.com/apache/nifi/pull/1422 NIFI-3015 - Corrects issue were RPM profile run.as property was not b… …eing used This supersedes #1198 Thank you for submitting a contribution to Apache NiFi. In order

[GitHub] nifi issue #1326: NIFI-2779 - Add processor to GetEmail Supporting Exchange ...

2017-01-16 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1326 @patricker Joe will have a more complete view on this but looking at the MSFT code there seem to be some dependencies with NOTICE. From top of my mind, joda-time, commons-io and others that must

[GitHub] nifi pull request #1326: NIFI-2779 - Add processor to GetEmail Supporting Ex...

2017-01-16 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/1326#discussion_r96259066 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ConsumeEWS.java --- @@ -0,0 +1,518

[GitHub] nifi pull request #1326: NIFI-2779 - Add processor to GetEmail Supporting Ex...

2017-01-16 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/1326#discussion_r96258831 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ConsumeEWS.java --- @@ -0,0 +1,518

[GitHub] nifi issue #1302: NIFI-2656: replace -k [password] with -K [passwordfile].

2017-01-16 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1302 @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 does not have this feature enabled and wishes so

[GitHub] nifi issue #1198: [NIFI-3015] Run NiFi service from nifi user instead of roo...

2017-01-16 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1198 @jfrazee - odd. Should the bootstrap.conf `run.as` be [populated](https://github.com/apache/nifi/blob/d838f61291d2582592754a37314911b701c6891b/nifi-nar-bundles/nifi-framework-bundle/nifi-framework

[GitHub] nifi pull request #420: NIFI-1280 Create FilterCSVColumns Processor.

2017-01-16 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/420#discussion_r96251033 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/pom.xml --- @@ -235,6 +235,11 @@ language governing permissions and limitations under

[GitHub] nifi issue #397: NIFI-1815

2017-01-16 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/397 @jdye64 is the PR active or should we close it? 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

[GitHub] nifi pull request #991: NIFI-2742 - Explicitly declare the kite hadoop depen...

2017-01-16 Thread trixpan
Github user trixpan closed the pull request at: https://github.com/apache/nifi/pull/991 --- 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 issue #709: NIFI-2369 - Added EL support to CSV properties

2017-01-16 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/709 @simonellistonball @mattyb149 should we perhaps close this 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

[GitHub] nifi pull request #1421: NIFI-3352 - Address issue where ListenLumberjack wa...

2017-01-16 Thread trixpan
GitHub user trixpan opened a pull request: https://github.com/apache/nifi/pull/1421 NIFI-3352 - Address issue where ListenLumberjack was sending long num… …bers back to clients - instead of int Thank you for submitting a contribution to Apache NiFi. In order

[GitHub] nifi pull request #1108: NIFI-2565: add Grok parser

2017-01-16 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/1108#discussion_r96236367 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/ExtractGrok.java --- @@ -0,0

[GitHub] nifi issue #1418: NIFI-3238 - Introduce ListenBeats processor and deprecates...

2017-01-14 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1418 @scottcrespo thanks for the message. the work around here is fairly simple in nature: to allow any beat (filebeat, metricbeat, etc) to send their data directly into NiFi, allowing people

[GitHub] nifi issue #1418: NIFI-3238 - Introduce ListenBeats processor and deprecates...

2017-01-14 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1418 @mattyb149 - this isn't ready yet but I suspect it may be functional. So far I have been testing with filebeat but it should work with metricbeat as well. Of note: - the processor

[GitHub] nifi pull request #1418: NIFI-3238 - Introduce ListenBeat processor and depr...

2017-01-14 Thread trixpan
GitHub user trixpan opened a pull request: https://github.com/apache/nifi/pull/1418 NIFI-3238 - Introduce ListenBeat processor and deprecates ListenLumbe… …rjack processor Thank you for submitting a contribution to Apache NiFi. In order to streamline the review

[GitHub] nifi issue #1282: NIFI-3112 Revert "NIFI-2199 - allows nifi.sh restart throu...

2016-11-29 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1282 Joe, Andy, I am away from my dev station but assuming both "ssh host.name service nifi restart", "ssh host.name /path/to/nifi/bin/nifi.sh restart" continue working I would imag

[GitHub] nifi pull request #1108: NIFI-2565: add Grok parser

2016-11-15 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/1108#discussion_r87978657 --- Diff: nifi-commons/nifi-processor-utilities/pom.xml --- @@ -45,5 +45,16 @@ nifi-ssl-context-service-api provided

[GitHub] nifi pull request #1108: NIFI-2565: add Grok parser

2016-11-14 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/1108#discussion_r87909770 --- Diff: nifi-commons/nifi-processor-utilities/src/main/java/org/apache/nifi/processor/util/StandardValidators.java --- @@ -26,6 +26,8 @@ import

[GitHub] nifi issue #1198: [NIFI-3015] Run NiFi service from nifi user instead of roo...

2016-11-11 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1198 Have you configured bootstrap after install? The runas setting is where this logic happens, not within the init script --- If your project is set up for it, you can reply to this email and have

[GitHub] nifi issue #1047: [NIFI-2812] Fix creating NiFi RPM

2016-10-27 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1047 Notes: * created RPM. * did diff between RPM with RPM profile and TGZ without the RPM profile * installed from the RPM. started NiFi. Connected to it. LGTM, will merge later

[GitHub] nifi-minifi pull request #:

2016-10-27 Thread trixpan
Github user trixpan commented on the pull request: https://github.com/apache/nifi-minifi/commit/483a4f996197b33c3e4de594a33f51e4b573fc44#commitcomment-19592136 In minifi-docs/src/main/markdown/minifi-java-agent-quick-start.md: In minifi-docs/src/main/markdown/minifi-java-agent

[GitHub] nifi pull request #1016: NIFI-2724 New JMX Processor

2016-10-26 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/1016#discussion_r85259465 --- Diff: nifi-nar-bundles/nifi-jmx-bundle/nifi-jmx-processors/src/main/java/org/apache/nifi/processors/jmx/GetJMX.java --- @@ -0,0 +1,385

[GitHub] nifi pull request #1016: NIFI-2724 New JMX Processor

2016-10-26 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/1016#discussion_r85244415 --- Diff: nifi-nar-bundles/nifi-jmx-bundle/nifi-jmx-processors/src/main/java/org/apache/nifi/processors/jmx/GetJMX.java --- @@ -0,0 +1,385

[GitHub] nifi pull request #1016: NIFI-2724 New JMX Processor

2016-10-26 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/1016#discussion_r85244266 --- Diff: nifi-nar-bundles/nifi-jmx-bundle/nifi-jmx-processors/src/main/java/org/apache/nifi/processors/jmx/GetJMX.java --- @@ -0,0 +1,385

[GitHub] nifi pull request #1016: NIFI-2724 New JMX Processor

2016-10-26 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/1016#discussion_r85244032 --- Diff: nifi-nar-bundles/nifi-jmx-bundle/nifi-jmx-processors/src/main/java/org/apache/nifi/processors/jmx/GetJMX.java --- @@ -0,0 +1,385

[GitHub] nifi issue #1016: NIFI-2724 New JMX Processor

2016-10-26 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1016 Seems fixed. Will start 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 #1154: NIFI-1531 - Remove bogus .gitignore files from sub ...

2016-10-22 Thread trixpan
GitHub user trixpan opened a pull request: https://github.com/apache/nifi/pull/1154 NIFI-1531 - Remove bogus .gitignore files from sub directories ### For all changes: - [x] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message

[GitHub] nifi issue #1016: NIFI-2724 New JMX Processor

2016-10-22 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1016 @brianburnett seems to be better now. I also suggest you run $ git commit --amend and remove the ``` # The first commit's message

[GitHub] nifi issue #1108: NIFI-2565: add Grok parser

2016-10-21 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1108 @selim-namsi - good work. We are getting there. --- 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 pull request #1108: NIFI-2565: add Grok parser

2016-10-21 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/1108#discussion_r84481939 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/GrokParser.java --- @@ -0,0 +1,243

[GitHub] nifi pull request #1108: NIFI-2565: add Grok parser

2016-10-21 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/1108#discussion_r84477121 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/GrokParser.java --- @@ -0,0 +1,243

[GitHub] nifi pull request #1108: NIFI-2565: add Grok parser

2016-10-21 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/1108#discussion_r84476015 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/GrokParser.java --- @@ -0,0 +1,243

[GitHub] nifi issue #1016: NIFI-2724 New JMX Processor

2016-10-21 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1016 @brianburnett would you be able to rebase and squash this PR? It currently includes other people commits that already have been merged to the master codebase --- If your project is set up

[GitHub] nifi issue #1018: NIFI-1662 adding Expression Language decimal support

2016-10-21 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1018 > (who uses Hex literals for floating point?!), I do! Like... All the time! But I also joke sometimes... :smiley: --- If your project is set up for it, you can re

[GitHub] nifi issue #1111: NIFI-2751 NIFI-2848 Get batch of flow files in a round-rob...

2016-10-21 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/ @gresockj can you confirm thij has this fixed NIFI-2848 (as per your template)? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] nifi pull request #1141: NIFI-2685 - Configure surefire to use JAVA Headless...

2016-10-20 Thread trixpan
Github user trixpan closed the pull request at: https://github.com/apache/nifi/pull/1141 --- 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 issue #1141: NIFI-2685 - Configure surefire to use JAVA Headless mode (...

2016-10-20 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1141 @mattyb149 this was the 0.x version of that PR. Closing it. --- 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 issue #319: NIFI-1582 added state to UpdateAttribute as well as updated...

2016-10-20 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/319 @JPercivall no issue. let us know when ready to review --- 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 #583: NIFI-2115 Detailed Version Info for About Box

2016-10-20 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/583 @mcgilman not sure if you seen this PR but feels to me like something we should consider merging. What do you think? --- If your project is set up for it, you can reply to this email and have

[GitHub] nifi issue #675: LSB Adherence

2016-10-20 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/675 @PuspenduBanerjee The following commits seem to solve some this issues: * LSB compliant exit codes -> 99d3c397481a98a5b3d2d4e44b84516fc0e3675e * SSH restart ha

[GitHub] nifi pull request #675: LSB Adherence

2016-10-20 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/675#discussion_r84271876 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-resources/src/main/resources/bin/nifi.sh --- @@ -265,20 +265,54 @@ run

[GitHub] nifi issue #966: improve foreground run mode of nifi

2016-10-20 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/966 @automaticgiant thank you for the contribution. This look to me, but I would like someone like @alopresto or @apiri to have a second look. --- If your project is set up for it, you

[GitHub] nifi issue #1093: Fixes NIFI-1069 init script status exit code 3 when not ru...

2016-10-20 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1093 @michalklempa - thank you for the contribution. This has now been merged. We hope to see further contributions from you in the future. --- If your project is set up for it, you can reply

[GitHub] nifi-minifi-cpp issue #19: MINIFI-119 add source.hostname to all flowfiles

2016-10-19 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi-minifi-cpp/pull/19 Can you rebase this commit by any chance? --- 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 #1093: Fixes NIFI-1069 init script status exit code 3 when not ru...

2016-10-18 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1093 @michalklempa thanks for getting this sorted. Will merge later today. --- 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 issue #397: NIFI-1815

2016-10-16 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/397 @olegz @jdye64 is this PR still active / plan to review? 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

[GitHub] nifi pull request #1141: NIFI-2685 - Configure surefire to use JAVA Headless...

2016-10-16 Thread trixpan
GitHub user trixpan opened a pull request: https://github.com/apache/nifi/pull/1141 NIFI-2685 - Configure surefire to use JAVA Headless mode ### For all changes: - [x] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message

[GitHub] nifi pull request #1001: NIFI-2750

2016-10-15 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/1001#discussion_r83530556 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/ConvertJSONToSQL.java

[GitHub] nifi issue #972: NIFI-2711 - Making top-level nifi-assemblies directory with...

2016-10-15 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/972 @brosander conflicts again... :cry: --- 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 issue #817: NIFI-2380 - Introduce ExtractTNEFAttachments

2016-10-15 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/817 @olegz ? --- 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

[GitHub] nifi issue #785: NIFI-2341 - Introduce ParseCEF processor

2016-10-14 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/785 @mattyb149 hopefully all addressed --- 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 #785: NIFI-2341 - Introduce ParseCEF processor

2016-10-14 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/785#discussion_r83525194 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/ParseCEF.java --- @@ -0,0 +1,323

[GitHub] nifi issue #1059: NIFI-2818 - Minimise fs permission required by NiFi

2016-10-14 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1059 @joewitt - This should be ready for review. --- 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

[GitHub] nifi pull request #1093: Fixes NIFI-1069 init script status exit code 3 when...

2016-10-14 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/1093#discussion_r83521841 --- Diff: nifi-bootstrap/src/main/java/org/apache/nifi/bootstrap/RunNiFi.java --- @@ -581,30 +585,31 @@ private Status getStatus(final Logger logger

[GitHub] nifi issue #854: NIFI-2564 Add support for HL7 composite component names

2016-10-14 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/854 @jfrazee apologies for the delay, would you mind rebasing this against master so it can be reviewed? I thank you in advance --- If your project is set up for it, you can reply to this email

[GitHub] nifi issue #1137: NIFI-1710 - Resolve path name to nifi.sh script

2016-10-14 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1137 @jfrazee @arpitgupta @busbey Since PR#315 got stalled, I reworked from scratch. Would appreciate your feedback if this solves the problem reported on NIFI-1710 @apiri

[GitHub] nifi pull request #1137: NIFI-1710 - Resolve path name to nifi.sh script

2016-10-14 Thread trixpan
GitHub user trixpan opened a pull request: https://github.com/apache/nifi/pull/1137 NIFI-1710 - Resolve path name to nifi.sh script ## Description: This commit allows nifi.sh to recursively traverse its path, allowing the simple linkage of nifi.sh to a script within /etc/init.d

[GitHub] nifi pull request #1093: Fixes NIFI-1069 init script status exit code 3 when...

2016-10-13 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/1093#discussion_r83360074 --- Diff: nifi-bootstrap/src/main/java/org/apache/nifi/bootstrap/RunNiFi.java --- @@ -581,30 +585,31 @@ private Status getStatus(final Logger logger

[GitHub] nifi issue #399: NIFI-1833 - Adding Azure Storage processors

2016-10-13 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/399 @slachterman I would say so --- 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 issue #1002: NIFI-2684 - Change AbstractConfigurableComponent to use di...

2016-10-13 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1002 @joewitt hopefully last commit addresses your feedback. Mind having another look? Cheers --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] nifi-minifi-cpp issue #18: MINIFI-34 - attempt to progress the CMake environ...

2016-10-12 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi-minifi-cpp/pull/18 @apiri should we merge this? --- 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

[GitHub] nifi pull request #785: NIFI-2341 - Introduce ParseCEF processor

2016-10-10 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/785#discussion_r82681523 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/ParseCEF.java --- @@ -0,0 +1,323

[GitHub] nifi issue #1108: NIFI-2565: add Grok parser

2016-10-08 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1108 Please rebase this PR so it only include your changes. 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

<    1   2   3   4   5   6   7   >