[GitHub] nifi pull request #727: NIFI-1663: Add ConvertAvroToORC processor

2016-07-26 Thread mattyb149
GitHub user mattyb149 opened a pull request: https://github.com/apache/nifi/pull/727 NIFI-1663: Add ConvertAvroToORC processor This PR is based on #706 which removed the ConvertAvroToORC processor using Hive 2.x and Apache ORC 1.x. This PR replaces that processor with one that uses

[GitHub] nifi pull request #642: NIFI-2156: Add ListDatabaseTables processor

2016-07-13 Thread mattyb149
GitHub user mattyb149 opened a pull request: https://github.com/apache/nifi/pull/642 NIFI-2156: Add ListDatabaseTables processor You can merge this pull request into a Git repository by running: $ git pull https://github.com/mattyb149/nifi NIFI-2156 Alternatively you can

[GitHub] nifi issue #649: Nifi 1413

2016-07-14 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/649 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 and wishes so

[GitHub] nifi issue #629: NIFI-2173 Fixing typo in LDAP referral strategy

2016-07-14 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/629 I'll test this with an LDAP server when I get a 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

[GitHub] nifi issue #649: Nifi 1413

2016-07-14 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/649 +1 LGTM built (against latest master) and tested. Want to rebase the commits or push as is? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] nifi pull request #254: NIFI-1583 Added ConvertJSONtoCSV processor

2016-07-23 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/254#discussion_r71980920 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/ConvertJSONtoCSV.java --- @@ -0,0

[GitHub] nifi pull request #254: NIFI-1583 Added ConvertJSONtoCSV processor

2016-07-23 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/254#discussion_r71980929 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/ConvertJSONtoCSV.java --- @@ -0,0

[GitHub] nifi pull request #254: NIFI-1583 Added ConvertJSONtoCSV processor

2016-07-23 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/254#discussion_r71981046 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/ConvertJSONtoCSV.java --- @@ -0,0

[GitHub] nifi pull request #254: NIFI-1583 Added ConvertJSONtoCSV processor

2016-07-23 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/254#discussion_r71981018 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/ConvertJSONtoCSV.java --- @@ -0,0

[GitHub] nifi pull request #254: NIFI-1583 Added ConvertJSONtoCSV processor

2016-07-23 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/254#discussion_r71981054 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/ConvertJSONtoCSV.java --- @@ -0,0

[GitHub] nifi pull request #254: NIFI-1583 Added ConvertJSONtoCSV processor

2016-07-23 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/254#discussion_r71980960 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/ConvertJSONtoCSV.java --- @@ -0,0

[GitHub] nifi pull request #744: NIFI-2422: Fix SelectHiveQL handling of Number types

2016-07-29 Thread mattyb149
GitHub user mattyb149 opened a pull request: https://github.com/apache/nifi/pull/744 NIFI-2422: Fix SelectHiveQL handling of Number types You can merge this pull request into a Git repository by running: $ git pull https://github.com/mattyb149/nifi NIFI-2422 Alternatively

[GitHub] nifi pull request #706: NIFI-1868: Add PutHiveStreaming processor

2016-07-29 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/706#discussion_r72797764 --- Diff: nifi-nar-bundles/nifi-hive-bundle/nifi-hive-processors/src/main/java/org/apache/nifi/processors/hive/PutHiveStreaming.java --- @@ -0,0 +1,657

[GitHub] nifi pull request #706: NIFI-1868: Add PutHiveStreaming processor

2016-07-29 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/706#discussion_r72871651 --- Diff: nifi-nar-bundles/nifi-hive-bundle/nifi-hive-processors/src/main/java/org/apache/nifi/processors/hive/PutHiveStreaming.java --- @@ -0,0 +1,657

[GitHub] nifi issue #547: NIFI-2060 - fixed unit tests to be compatible with 2.7.x

2016-08-01 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/547 +1 LGTM, ran tests with Hadoop 2.6.2 and 2.7.0, all passed. Merging to 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

[GitHub] nifi issue #706: NIFI-1868: Add PutHiveStreaming processor

2016-08-02 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/706 Per an offline discussion with @bbende, we decided a good way to handle the outgoing flow files is to have a "success", "failure", and "retry" relationship. If a

[GitHub] nifi issue #777: Add String Escape Utils to Nifi Expression Language

2016-08-11 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/777 yes please :) --- 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 #823: NIFI-2531: Fixed JdbcCommon handling of BigInteger o...

2016-08-09 Thread mattyb149
GitHub user mattyb149 opened a pull request: https://github.com/apache/nifi/pull/823 NIFI-2531: Fixed JdbcCommon handling of BigInteger objects for Avro You can merge this pull request into a Git repository by running: $ git pull https://github.com/mattyb149/nifi NIFI-2531

[GitHub] nifi issue #823: NIFI-2531: Fixed JdbcCommon handling of BigInteger objects ...

2016-08-10 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/823 Good point, let me see if I can use the column width (for BIGINT types) to determine whether to expect a long or string in the value. --- If your project is set up for it, you can reply

[GitHub] nifi pull request #830: NIFI-1663: Adding attribution for derived work for O...

2016-08-10 Thread mattyb149
Github user mattyb149 closed the pull request at: https://github.com/apache/nifi/pull/830 --- 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 pull request #777: Add String Escape Utils to Nifi Expression Language

2016-08-10 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/777#discussion_r74322615 --- Diff: nifi-docs/src/main/asciidoc/expression-language-guide.adoc --- @@ -993,6 +954,229 @@ Expressions will provide the following results

[GitHub] nifi issue #777: Add String Escape Utils to Nifi Expression Language

2016-08-10 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/777 Either is good, a new PR on a new branch would just get you in the habit of using a "feature branch" for development versus the local master branch. --- If your project is set up for i

[GitHub] nifi issue #706: NIFI-1868: Add PutHiveStreaming processor

2016-08-03 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/706 Yeah I think that's a bug in this version of Hive: https://issues.apache.org/jira/browse/HIVE-13725 For now I'd like to make the processor TriggerSerially (unless there are any objections

[GitHub] nifi pull request #710: NIFI-1148 added IMAP/POP3 support

2016-08-04 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/710#discussion_r73515360 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/AbstractEmailProcessor.java --- @@ -0,0

[GitHub] nifi pull request #710: NIFI-1148 added IMAP/POP3 support

2016-08-04 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/710#discussion_r73515235 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/pom.xml --- @@ -1,73 +1,84 @@ - -http://maven.apache.org/POM/4.0.0

[GitHub] nifi pull request #727: NIFI-1663: Add ConvertAvroToORC processor

2016-08-04 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/727#discussion_r73520631 --- Diff: nifi-nar-bundles/nifi-hive-bundle/nifi-hive-processors/src/main/java/org/apache/hadoop/hive/ql/io/orc/NiFiOrcUtils.java --- @@ -0,0 +1,466

[GitHub] nifi issue #710: NIFI-1148 added IMAP/POP3 support

2016-08-04 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/710 I ran ConsumePop3 against GMail but hadn't seen your comment on all the dynamic properties that need to be set. I started and stopped the processor, but there is a thread continuing to run. Stopped

[GitHub] nifi pull request #710: NIFI-1148 added IMAP/POP3 support

2016-08-04 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/710#discussion_r73518325 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ConsumeIMAP.java --- @@ -0,0 +1,91

[GitHub] nifi issue #710: NIFI-1148 added IMAP/POP3 support

2016-08-04 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/710 +1 to the "Additional Details" file for all the dynamic properties, etc. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as wel

[GitHub] nifi pull request #727: NIFI-1663: Add ConvertAvroToORC processor

2016-08-03 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/727#discussion_r73331976 --- Diff: nifi-nar-bundles/nifi-hive-bundle/nifi-hive-processors/src/main/java/org/apache/hadoop/hive/ql/io/orc/NiFiOrcUtils.java --- @@ -0,0 +1,466

[GitHub] nifi pull request #727: NIFI-1663: Add ConvertAvroToORC processor

2016-08-03 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/727#discussion_r73331884 --- Diff: nifi-nar-bundles/nifi-hive-bundle/nifi-hive-processors/src/main/java/org/apache/hadoop/hive/ql/io/orc/NiFiOrcUtils.java --- @@ -0,0 +1,466

[GitHub] nifi issue #727: NIFI-1663: Add ConvertAvroToORC processor

2016-08-03 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/727 I see a few comments about PutHiveStreaming, that is part of a previous commit that is being handled under #706, it's just in here in order to inherit the downgrade of Hive. Only the last commit

[GitHub] nifi issue #777: Add String Escape Utils to Nifi Expression Language

2016-08-12 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/777 #833 has been merged (thanks again!) Would you mind closing 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

[GitHub] nifi issue #833: Add String Escape Utils to Nifi Expression Language (redo)

2016-08-12 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/833 +1 Great job! I built, ran, and tested the functions with escaped/unescaped inputs and empty strings, everything works well. Thanks again for the contribution! Will merge to master. --- If your

[GitHub] nifi issue #777: Add String Escape Utils to Nifi Expression Language

2016-08-10 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/777 No worries! I think it is easiest to do development from a branch other than your master branch. Here is a sequence of commands that might work: git checkout -b add_escape_utils git

[GitHub] nifi issue #564: NIFI-2020 - Enhance JoltTransformJSON processor to support ...

2016-07-12 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/564 I had forgotten about the checked exception thing. If the lamdbas/streams are just as awkward as Java 7 code (due to the exceptions, etc.) then there's probably no need to force it. Everything else

[GitHub] nifi pull request #645: NIFI-2157: Add GenerateTableFetch processor

2016-07-13 Thread mattyb149
GitHub user mattyb149 opened a pull request: https://github.com/apache/nifi/pull/645 NIFI-2157: Add GenerateTableFetch processor Refactored common code out of QueryDatabaseTable into an Abstract base class, which involved some refactor of QueryDatabaseTable as well as the added

[GitHub] nifi issue #475: NIFI-2026 - Add Maven profile to compile nifi-hadoop-librar...

2016-07-13 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/475 Since other Apache projects are taking the stance of including the repositories in the top-level POM, I think NiFi should too. These would include (at a minimum) the MapR, Cloudera, and Hortonworks

[GitHub] nifi issue #596: NIFI-1413: Ensure that if a node's templates don't match th...

2016-07-13 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/596 +1 LGTM, built and ran with a 3-node cluster. Took one node offline, changed it's template information, then restarted and it reconnected successfully. Would be nice to have an integration test

[GitHub] nifi issue #642: NIFI-2156: Add ListDatabaseTables processor

2016-07-18 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/642 Yes, at one point I had a "Refresh Interval" property but I think that was in another branch, will restore it. Also, currently any change to properties will reset the state (since the tabl

[GitHub] nifi pull request #645: NIFI-2157: Add GenerateTableFetch processor

2016-07-17 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/645#discussion_r71087671 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/GenerateTableFetch.java

[GitHub] nifi pull request #642: NIFI-2156: Add ListDatabaseTables processor

2016-07-17 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/642#discussion_r71091826 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/ListDatabaseTables.java

[GitHub] nifi pull request #642: NIFI-2156: Add ListDatabaseTables processor

2016-07-17 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/642#discussion_r71093459 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/ListDatabaseTables.java

[GitHub] nifi pull request #645: NIFI-2157: Add GenerateTableFetch processor

2016-07-16 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/645#discussion_r71074368 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/QueryDatabaseTable.java

[GitHub] nifi pull request #645: NIFI-2157: Add GenerateTableFetch processor

2016-07-16 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/645#discussion_r71073757 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/GenerateTableFetch.java

[GitHub] nifi pull request #642: NIFI-2156: Add ListDatabaseTables processor

2016-07-18 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/642#discussion_r71249117 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/ListDatabaseTables.java

[GitHub] nifi issue #629: NIFI-2173 Fixing typo in LDAP referral strategy

2016-07-15 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/629 +1 LGTM, reproduced the issue then tested with the fix, verified the error is no longer present --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] nifi pull request #613: NIFI-2126: Add processors to enable distributed fetc...

2016-07-11 Thread mattyb149
Github user mattyb149 closed the pull request at: https://github.com/apache/nifi/pull/613 --- 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 #613: NIFI-2126: Add processors to enable distributed fetching of...

2016-07-11 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/613 Closing this as incremental fetch capability is needed. Will produce a new PR against NIFI-2126 with both processors inside --- If your project is set up for it, you can reply to this email

[GitHub] nifi pull request #616: NIFI-1998: Update Cassandra processor docs to includ...

2016-07-07 Thread mattyb149
GitHub user mattyb149 opened a pull request: https://github.com/apache/nifi/pull/616 NIFI-1998: Update Cassandra processor docs to include 3.0.x clusters Compatibility matrix here: http://datastax.github.io/java-driver/manual/native_protocol/ You can merge this pull request

[GitHub] nifi pull request #564: NIFI-2020 - Enhance JoltTransformJSON processor to s...

2016-07-08 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/564#discussion_r70156828 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-jolt-transform-json-ui/src/main/java/org/apache/nifi/web/standard/api/transformjson

[GitHub] nifi pull request #564: NIFI-2020 - Enhance JoltTransformJSON processor to s...

2016-07-08 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/564#discussion_r70156926 --- Diff: nifi-commons/nifi-utils/src/main/java/org/apache/nifi/util/file/classloader/ClassLoaderUtils.java --- @@ -0,0 +1,66 @@ +/* + * Licensed

[GitHub] nifi pull request #564: NIFI-2020 - Enhance JoltTransformJSON processor to s...

2016-07-08 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/564#discussion_r70157498 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/JoltTransformJSON.java

[GitHub] nifi pull request #564: NIFI-2020 - Enhance JoltTransformJSON processor to s...

2016-07-08 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/564#discussion_r70157576 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/JoltTransformJSON.java

[GitHub] nifi pull request #564: NIFI-2020 - Enhance JoltTransformJSON processor to s...

2016-07-08 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/564#discussion_r70157635 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/JoltTransformJSON.java

[GitHub] nifi pull request #710: NIFI-1148 added IMAP/POP3 support

2016-08-05 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/710#discussion_r73770089 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/resources/docs/org.apache.nifi.processors.email.ConsumePOP3/additionalDetails.html

[GitHub] nifi pull request #710: NIFI-1148 added IMAP/POP3 support

2016-08-05 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/710#discussion_r73769802 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/resources/docs/org.apache.nifi.processors.email.ConsumeIMAP/additionalDetails.html

[GitHub] nifi pull request #710: NIFI-1148 added IMAP/POP3 support

2016-08-05 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/710#discussion_r73769714 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/resources/docs/org.apache.nifi.processors.email.ConsumeIMAP/additionalDetails.html

[GitHub] nifi pull request #710: NIFI-1148 added IMAP/POP3 support

2016-08-05 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/710#discussion_r73769916 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/resources/docs/org.apache.nifi.processors.email.ConsumeIMAP/additionalDetails.html

[GitHub] nifi issue #710: NIFI-1148 added IMAP/POP3 support

2016-08-08 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/710 +1 LGTM, ran tests and verified comments were incorporated, merging to master --- 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 #699: * NIFI-2026 - Add Cloudera, Hortonworks and MapR repositori...

2016-08-08 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/699 +1 LGTM, was able to build and run with various vendor-specific Hadoop versions (after some small tweaks I'll add to the PR before merging). Merging to master --- If your project is set up

[GitHub] nifi issue #727: NIFI-1663: Add ConvertAvroToORC processor

2016-08-08 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/727 Closing the PR and cancelling the current patch as there are issues with complex types, will reopen once fixed --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] nifi pull request #812: NIFI-2026: Moved vendor repositories into Maven prof...

2016-08-08 Thread mattyb149
GitHub user mattyb149 opened a pull request: https://github.com/apache/nifi/pull/812 NIFI-2026: Moved vendor repositories into Maven profiles @trixpan Basically moved the repos into profiles and added default properties so users can see the naming pattern (might help them select

[GitHub] nifi pull request #727: NIFI-1663: Add ConvertAvroToORC processor

2016-08-08 Thread mattyb149
GitHub user mattyb149 reopened a pull request: https://github.com/apache/nifi/pull/727 NIFI-1663: Add ConvertAvroToORC processor This PR is based on #706 which removed the ConvertAvroToORC processor using Hive 2.x and Apache ORC 1.x. This PR replaces that processor with one

[GitHub] nifi pull request #727: NIFI-1663: Add ConvertAvroToORC processor

2016-08-08 Thread mattyb149
Github user mattyb149 closed the pull request at: https://github.com/apache/nifi/pull/727 --- 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 pull request #818: NIFI-2520: Added attribution for storm-hive and othe...

2016-08-09 Thread mattyb149
GitHub user mattyb149 opened a pull request: https://github.com/apache/nifi/pull/818 NIFI-2520: Added attribution for storm-hive and other Hive dependencies to NOTICEs Also removed Apache ORC as it is not used (hive-orc within the Apache Hive project is used instead) You can

[GitHub] nifi pull request #821: NIFI-2518: Added support for fractional seconds to A...

2016-08-09 Thread mattyb149
GitHub user mattyb149 opened a pull request: https://github.com/apache/nifi/pull/821 NIFI-2518: Added support for fractional seconds to AbstractDatabaseFetchProcessor You can merge this pull request into a Git repository by running: $ git pull https://github.com/mattyb149

[GitHub] nifi issue #706: NIFI-1868: Add PutHiveStreaming processor

2016-08-02 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/706 Yes that's a good idea, will update, 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 project does not have

[GitHub] nifi issue #813: Updating UI to reflect administrator policies do not overri...

2016-08-09 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/813 +1 LGTM, merging --- 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 #727: NIFI-1663: Add ConvertAvroToORC processor

2016-08-03 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/727#discussion_r73331820 --- Diff: nifi-nar-bundles/nifi-hive-bundle/nifi-hive-processors/src/main/java/org/apache/hadoop/hive/ql/io/orc/NiFiOrcUtils.java --- @@ -0,0 +1,466

[GitHub] nifi pull request #872: NIFI-2574 checkpoint commit to allow 1831 progress

2016-08-16 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/872#discussion_r74981298 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-security/src/test/groovy/org/apache/nifi/web/security/x509/ocsp

[GitHub] nifi pull request #865: NIFI-2552: Updated Curator for HiveQL processors usi...

2016-08-15 Thread mattyb149
GitHub user mattyb149 opened a pull request: https://github.com/apache/nifi/pull/865 NIFI-2552: Updated Curator for HiveQL processors using Zookeeper quorum in URL You can merge this pull request into a Git repository by running: $ git pull https://github.com/mattyb149/nifi

[GitHub] nifi pull request #877: QueryDatabaseTable updates

2016-08-16 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/877#discussion_r75051154 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/QueryDatabaseTable.java

[GitHub] nifi pull request #877: QueryDatabaseTable updates

2016-08-16 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/877#discussion_r75051588 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/QueryDatabaseTable.java

[GitHub] nifi pull request #877: QueryDatabaseTable updates

2016-08-16 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/877#discussion_r75051737 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/QueryDatabaseTable.java

[GitHub] nifi pull request #877: QueryDatabaseTable updates

2016-08-16 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/877#discussion_r75051357 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/QueryDatabaseTable.java

[GitHub] nifi pull request #877: QueryDatabaseTable updates

2016-08-16 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/877#discussion_r75051853 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/QueryDatabaseTable.java

[GitHub] nifi pull request #877: QueryDatabaseTable updates

2016-08-16 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/877#discussion_r75051917 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/test/java/org/apache/nifi/processors/standard/QueryDatabaseTableTest.java

[GitHub] nifi pull request #850: NIFI-2547: Add DeleteHDFS Processor

2016-08-16 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/850#discussion_r75044228 --- Diff: nifi-nar-bundles/nifi-hadoop-bundle/nifi-hdfs-processors/src/main/java/org/apache/nifi/processors/hadoop/DeleteHDFS.java --- @@ -0,0 +1,161

[GitHub] nifi pull request #850: NIFI-2547: Add DeleteHDFS Processor

2016-08-16 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/850#discussion_r75044175 --- Diff: nifi-nar-bundles/nifi-hadoop-bundle/nifi-hdfs-processors/src/main/java/org/apache/nifi/processors/hadoop/DeleteHDFS.java --- @@ -0,0 +1,161

[GitHub] nifi pull request #877: QueryDatabaseTable updates

2016-08-17 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/877#discussion_r75114893 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/QueryDatabaseTable.java

[GitHub] nifi issue #877: QueryDatabaseTable updates

2016-08-17 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/877 Awesome, thanks! Looking forward to these improvements, will make QDT much more flexible and powerful --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] nifi issue #877: QueryDatabaseTable updates

2016-08-17 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/877 Everything looks good and works well, thanks! @patricker what do you think about adding fragment.count? Could be a later improvement but I think it'd be a good thing to add if not too troublesome

[GitHub] nifi pull request #877: QueryDatabaseTable updates

2016-08-17 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/877#discussion_r75120131 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/QueryDatabaseTable.java

[GitHub] nifi pull request #877: QueryDatabaseTable updates

2016-08-17 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/877#discussion_r75114294 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/QueryDatabaseTable.java

[GitHub] nifi pull request #877: QueryDatabaseTable updates

2016-08-17 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/877#discussion_r75114562 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/QueryDatabaseTable.java

[GitHub] nifi issue #877: QueryDatabaseTable updates

2016-08-17 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/877 Your second commit is labelled NIFI-2584, but I believe it's for NIFI-2583. Would you mind re-submitting with the new commit label? Just want to make sure when the merge happens that the Jira trail

[GitHub] nifi issue #877: QueryDatabaseTable updates

2016-08-17 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/877 +1 LGTM, thanks again for the great contribution! Merging to master --- 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 #850: NIFI-2547: Add DeleteHDFS Processor

2016-08-16 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/850 Just a couple of minor comments; otherwise looks great, 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 project does

[GitHub] nifi pull request #850: NIFI-2547: Add DeleteHDFS Processor

2016-08-16 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/850#discussion_r75044365 --- Diff: nifi-nar-bundles/nifi-hadoop-bundle/nifi-hdfs-processors/src/test/java/org/apache/nifi/processors/hadoop/TestDeleteHDFS.java --- @@ -0,0 +1,187

[GitHub] nifi pull request #1407: NIFI-2881: Added EL support to DB Fetch processors,...

2017-01-31 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/1407#discussion_r98680344 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/GenerateTableFetch.java

[GitHub] nifi pull request #1407: NIFI-2881: Added EL support to DB Fetch processors,...

2017-01-31 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/1407#discussion_r98680239 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/AbstractDatabaseFetchProcessor.java

[GitHub] nifi pull request #1407: NIFI-2881: Added EL support to DB Fetch processors,...

2017-01-31 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/1407#discussion_r98684537 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/QueryDatabaseTable.java

[GitHub] nifi pull request #1407: NIFI-2881: Added EL support to DB Fetch processors,...

2017-01-31 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/1407#discussion_r98681382 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/QueryDatabaseTable.java

[GitHub] nifi pull request #1407: NIFI-2881: Added EL support to DB Fetch processors,...

2017-02-01 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/1407#discussion_r98898226 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/QueryDatabaseTable.java

[GitHub] nifi pull request #1461: NIFI-3426: Add dynamic property support to DBCPConn...

2017-02-01 Thread mattyb149
GitHub user mattyb149 opened a pull request: https://github.com/apache/nifi/pull/1461 NIFI-3426: Add dynamic property support to DBCPConnectionPool Unit test adds "create=true" as a Derby property (instead of via the URL), written in Groovy to leverage the recursive

[GitHub] nifi issue #1450: NIFI-3339b Add getDataSource() to DBCPService, second vers...

2017-02-06 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/1450 My use case is a third-party library that takes the same information you might put into a DBCPControllerService, either in the URL and/or the username/password, etc. I was hoping to avoid trying

[GitHub] nifi pull request #1407: NIFI-2881: Added EL support to DB Fetch processors,...

2017-02-02 Thread mattyb149
Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/1407#discussion_r99111299 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/AbstractDatabaseFetchProcessor.java

[GitHub] nifi issue #1450: NIFI-3339b Add getDataSource() to DBCPService, second vers...

2017-02-06 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/1450 Koji has a good point about managing the actual DataSource instance internally, if your use case can be accomplished without needing access to this object, then I would tend to agree that we

[GitHub] nifi issue #1455: NIFI-3418 Add records-per-transaction property to putHiveS...

2017-01-30 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/1455 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 and wishes so

  1   2   3   4   5   6   7   8   9   10   >