[GitHub] nifi pull request #1961: NIFI-4024 Added org.apache.nifi.hbase.PutHBaseRecor...

2017-08-02 Thread MikeThomsen
Github user MikeThomsen commented on a diff in the pull request: https://github.com/apache/nifi/pull/1961#discussion_r130873950 --- Diff: nifi-nar-bundles/nifi-hbase-bundle/nifi-hbase-processors/src/main/java/org/apache/nifi/hbase/PutHBaseRecord.java --- @@ -0,0 +1,323

[GitHub] nifi issue #1945: NIFI-4124 Added org.apache.nifi.mongo.PutMongoRecord.

2017-08-02 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/1945 @markap14 @joewitt Any chance of getting this reviewed? --- 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 #1961: NIFI-4024 Added org.apache.nifi.hbase.PutHBaseRecord

2017-08-03 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/1961 Looks good to me. --- 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 #1961: NIFI-4024 Added org.apache.nifi.hbase.PutHBaseRecord

2017-07-17 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/1961 @bbende Ok, should be good to go now. --- 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 issue #1961: NIFI-4024 Added org.apache.nifi.hbase.PutHBaseRecord

2017-07-10 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/1961 @bbende Updated. Let me know what you think when you get a chance. 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 issue #1945: NIFI-4124 Added org.apache.nifi.mongo.PutMongoRecord.

2017-07-07 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/1945 @markap14 @bbende @pvillard31 I made the changes. It should be ready for a merge. --- 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 #1961: NIFI-4024 Added org.apache.nifi.hbase.PutHBaseRecord

2017-07-07 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/1961 @bbende I ran this against a large body of our test data, and it seemed to work just fine. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] nifi issue #1945: NIFI-4124 Added org.apache.nifi.mongo.PutMongoRecord.

2017-07-17 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/1945 @bbende @markap14 @pvillard31 Any chance of getting this merged? --- 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 #1945: NIFI-4124 Added org.apache.nifi.mongo.PutMongoRecord.

2017-07-26 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/1945 Yay? Nay? --- 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 #1961: NIFI-4024 Added org.apache.nifi.hbase.PutHBaseRecord

2017-07-26 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/1961 @bbende Saw on the mailing list that you've been away for a little while. Any chance you're back and ready to take a look? --- If your project is set up for it, you can reply to this email

[GitHub] nifi pull request #1961: NIFI-4024 Added org.apache.nifi.hbase.PutHBaseRecor...

2017-06-29 Thread MikeThomsen
GitHub user MikeThomsen opened a pull request: https://github.com/apache/nifi/pull/1961 NIFI-4024 Added org.apache.nifi.hbase.PutHBaseRecord Thank you for submitting a contribution to Apache NiFi. In order to streamline the review of the contribution we ask you

[GitHub] nifi issue #1945: NIFI-4124 Added org.apache.nifi.mongo.PutMongoRecord.

2017-07-03 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/1945 @markap14 @bbende Everything looks good. Do you want to commit? --- 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 #1945: NIFI-4124 Added org.apache.nifi.mongo.PutMongoRecor...

2017-06-27 Thread MikeThomsen
Github user MikeThomsen commented on a diff in the pull request: https://github.com/apache/nifi/pull/1945#discussion_r124303897 --- Diff: nifi-nar-bundles/nifi-mongodb-bundle/nifi-mongodb-processors/pom.xml --- @@ -48,6 +48,16

[GitHub] nifi pull request #1945: NIFI-4124 Added org.apache.nifi.mongo.PutMongoRecor...

2017-06-27 Thread MikeThomsen
Github user MikeThomsen commented on a diff in the pull request: https://github.com/apache/nifi/pull/1945#discussion_r124295160 --- Diff: nifi-nar-bundles/nifi-mongodb-bundle/nifi-mongodb-processors/pom.xml --- @@ -48,6 +48,16

[GitHub] nifi pull request #1945: NIFI-4124 Added org.apache.nifi.mongo.PutMongoRecor...

2017-06-28 Thread MikeThomsen
Github user MikeThomsen commented on a diff in the pull request: https://github.com/apache/nifi/pull/1945#discussion_r124553265 --- Diff: nifi-nar-bundles/nifi-mongodb-bundle/nifi-mongodb-processors/pom.xml --- @@ -66,5 +74,10 @@ guava test

[GitHub] nifi pull request #1945: NIFI-4124 Added org.apache.nifi.mongo.PutMongoRecor...

2017-06-28 Thread MikeThomsen
Github user MikeThomsen commented on a diff in the pull request: https://github.com/apache/nifi/pull/1945#discussion_r124553180 --- Diff: nifi-nar-bundles/nifi-mongodb-bundle/nifi-mongodb-processors/src/main/java/org/apache/nifi/processors/mongodb/PutMongoRecord.java --- @@ -0,0

[GitHub] nifi pull request #1945: NIFI-4124 Added org.apache.nifi.mongo.PutMongoRecor...

2017-06-28 Thread MikeThomsen
Github user MikeThomsen commented on a diff in the pull request: https://github.com/apache/nifi/pull/1945#discussion_r124553212 --- Diff: nifi-nar-bundles/nifi-mongodb-bundle/nifi-mongodb-processors/src/main/java/org/apache/nifi/processors/mongodb/PutMongoRecord.java --- @@ -0,0

[GitHub] nifi issue #1945: NIFI-4124 Added org.apache.nifi.mongo.PutMongoRecord.

2017-06-28 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/1945 @markap14 I'm testing the change locally and will push the commit that addresses these once I'm satisfied it works. --- If your project is set up for it, you can reply to this email and have

[GitHub] nifi pull request #1945: NIFI-4124 Added org.apache.nifi.mongo.PutMongoRecor...

2017-06-28 Thread MikeThomsen
Github user MikeThomsen commented on a diff in the pull request: https://github.com/apache/nifi/pull/1945#discussion_r124553244 --- Diff: nifi-nar-bundles/nifi-mongodb-bundle/nifi-mongodb-processors/src/main/java/org/apache/nifi/processors/mongodb/PutMongoRecord.java --- @@ -0,0

[GitHub] nifi issue #1948: NIFI-4122 Added the ability to combine multiple Mongo resu...

2017-06-28 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/1948 Thanks @pvillard31. I'll get to these shortly. --- 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 #1961: NIFI-4024 Added org.apache.nifi.hbase.PutHBaseRecord

2017-08-02 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/1961 @bbende Done. --- 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 #2063: NIFI-4269 Added the ability to serialize Mongo docu...

2017-08-07 Thread MikeThomsen
GitHub user MikeThomsen opened a pull request: https://github.com/apache/nifi/pull/2063 NIFI-4269 Added the ability to serialize Mongo documents to a clean J… …SON view instead of just extended JSON. Thank you for submitting a contribution to Apache NiFi

[GitHub] nifi pull request #2063: NIFI-4269 Added the ability to serialize Mongo docu...

2017-08-22 Thread MikeThomsen
Github user MikeThomsen commented on a diff in the pull request: https://github.com/apache/nifi/pull/2063#discussion_r134629298 --- Diff: nifi-nar-bundles/nifi-mongodb-bundle/nifi-mongodb-processors/src/main/java/org/apache/nifi/processors/mongodb/GetMongo.java --- @@ -151,17

[GitHub] nifi issue #2063: NIFI-4269 Added the ability to serialize Mongo documents t...

2017-08-22 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2063 Thanks for the feedback @mattyb149 . I'll start working on these shortly (got some other stuff I have to do at the moment). --- If your project is set up for it, you can reply to this email

[GitHub] nifi issue #2063: NIFI-4269 Added the ability to serialize Mongo documents t...

2017-08-23 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2063 @mattyb149 Ok, changes committed and at least the French build ran to completion. --- 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 #1945: NIFI-4124 Added org.apache.nifi.mongo.PutMongoRecor...

2017-06-26 Thread MikeThomsen
GitHub user MikeThomsen opened a pull request: https://github.com/apache/nifi/pull/1945 NIFI-4124 Added org.apache.nifi.mongo.PutMongoRecord. https://issues.apache.org/jira/browse/NIFI-4124 You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] nifi pull request #1943: NIFI-4122 Added the ability to combine multiple Mon...

2017-06-26 Thread MikeThomsen
GitHub user MikeThomsen opened a pull request: https://github.com/apache/nifi/pull/1943 NIFI-4122 Added the ability to combine multiple Mongo result document… https://issues.apache.org/jira/browse/NIFI-4122 You can merge this pull request into a Git repository by running

[GitHub] nifi pull request #1945: NIFI-4124 Added org.apache.nifi.mongo.PutMongoRecor...

2017-06-26 Thread MikeThomsen
GitHub user MikeThomsen reopened a pull request: https://github.com/apache/nifi/pull/1945 NIFI-4124 Added org.apache.nifi.mongo.PutMongoRecord. https://issues.apache.org/jira/browse/NIFI-4124 You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] nifi pull request #1943: NIFI-4122 Added the ability to combine multiple Mon...

2017-06-26 Thread MikeThomsen
GitHub user MikeThomsen reopened a pull request: https://github.com/apache/nifi/pull/1943 NIFI-4122 Added the ability to combine multiple Mongo result document… https://issues.apache.org/jira/browse/NIFI-4122 You can merge this pull request into a Git repository by running

[GitHub] nifi pull request #1943: NIFI-4122 Added the ability to combine multiple Mon...

2017-06-26 Thread MikeThomsen
Github user MikeThomsen closed the pull request at: https://github.com/apache/nifi/pull/1943 --- 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 #1945: NIFI-4124 Added org.apache.nifi.mongo.PutMongoRecor...

2017-06-26 Thread MikeThomsen
Github user MikeThomsen closed the pull request at: https://github.com/apache/nifi/pull/1945 --- 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 #1943: NIFI-4122 Added the ability to combine multiple Mon...

2017-06-26 Thread MikeThomsen
Github user MikeThomsen closed the pull request at: https://github.com/apache/nifi/pull/1943 --- 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 #1945: NIFI-4124 Added org.apache.nifi.mongo.PutMongoRecor...

2017-06-27 Thread MikeThomsen
GitHub user MikeThomsen reopened a pull request: https://github.com/apache/nifi/pull/1945 NIFI-4124 Added org.apache.nifi.mongo.PutMongoRecord. https://issues.apache.org/jira/browse/NIFI-4124 You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] nifi pull request #1945: NIFI-4124 Added org.apache.nifi.mongo.PutMongoRecor...

2017-06-27 Thread MikeThomsen
Github user MikeThomsen closed the pull request at: https://github.com/apache/nifi/pull/1945 --- 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 #1945: NIFI-4124 Added org.apache.nifi.mongo.PutMongoRecor...

2017-06-27 Thread MikeThomsen
Github user MikeThomsen closed the pull request at: https://github.com/apache/nifi/pull/1945 --- 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 #1945: NIFI-4124 Added org.apache.nifi.mongo.PutMongoRecor...

2017-06-27 Thread MikeThomsen
GitHub user MikeThomsen reopened a pull request: https://github.com/apache/nifi/pull/1945 NIFI-4124 Added org.apache.nifi.mongo.PutMongoRecord. https://issues.apache.org/jira/browse/NIFI-4124 You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] nifi pull request #1945: NIFI-4124 Added org.apache.nifi.mongo.PutMongoRecor...

2017-06-27 Thread MikeThomsen
GitHub user MikeThomsen reopened a pull request: https://github.com/apache/nifi/pull/1945 NIFI-4124 Added org.apache.nifi.mongo.PutMongoRecord. https://issues.apache.org/jira/browse/NIFI-4124 You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] nifi pull request #1945: NIFI-4124 Added org.apache.nifi.mongo.PutMongoRecor...

2017-06-27 Thread MikeThomsen
Github user MikeThomsen closed the pull request at: https://github.com/apache/nifi/pull/1945 --- 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 #2125: NIFI-4346 Created a LookupService that uses HBase as its b...

2017-09-17 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2125 @markap14 @mattyb149 Can you take a look when you get a chance? ---

[GitHub] nifi issue #2123: NIFI-4345 Added a MongoDB controller service and a lookup ...

2017-09-17 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2123 @markap14 Do you think you could take a look at the updates? ---

[GitHub] nifi issue #2125: NIFI-4346 Created a LookupService that uses HBase as its b...

2017-10-06 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2125 +1 The string lookup one is going to be interesting. ---

[GitHub] nifi issue #2125: NIFI-4346 Created a LookupService that uses HBase as its b...

2017-10-06 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2125 (Go ahead and commit, seems like a good approach to me) ---

[GitHub] nifi issue #2104: NIFI-4307 Added Kotlin 1.1.X support to ExecuteScript.

2017-10-06 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2104 @mattyb149 I did a sample flow with each of ours and found the performance to be awful. I did some investigation and experimentation with the Kotlin ScriptEngine and found that after repeatedly

[GitHub] nifi issue #2104: NIFI-4307 Added Kotlin 1.1.X support to ExecuteScript.

2017-10-06 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2104 I think Kotlin makes no sense in light of this because it's really not a scripting language. Focusing on InvokeScriptProcessor would make more sense because that is where a language like Kotlin

[GitHub] nifi pull request #2180: Added GetMongoAggregation to support running Mongo ...

2017-10-03 Thread MikeThomsen
Github user MikeThomsen commented on a diff in the pull request: https://github.com/apache/nifi/pull/2180#discussion_r142563421 --- Diff: nifi-nar-bundles/nifi-mongodb-bundle/nifi-mongodb-processors/src/main/java/org/apache/nifi/processors/mongodb/GetMongoAggregation.java

[GitHub] nifi issue #2113: NIFI-4325 Added new processor that uses the JSON DSL.

2017-10-09 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2113 I set up a profile called integration-tests and renamed the file to JsonQueryElasticsearch_IT. It runs really well from the project folder, but for some reason running `mvn clean install

[GitHub] nifi pull request #2113: NIFI-4325 Added new processor that uses the JSON DS...

2017-10-09 Thread MikeThomsen
Github user MikeThomsen commented on a diff in the pull request: https://github.com/apache/nifi/pull/2113#discussion_r143463999 --- Diff: nifi-nar-bundles/nifi-elasticsearch-bundle/nifi-elasticsearch-5-processors/src/main/java/org/apache/nifi/processors/elasticsearch

[GitHub] nifi pull request #2113: NIFI-4325 Added new processor that uses the JSON DS...

2017-10-09 Thread MikeThomsen
Github user MikeThomsen commented on a diff in the pull request: https://github.com/apache/nifi/pull/2113#discussion_r143464047 --- Diff: nifi-nar-bundles/nifi-elasticsearch-bundle/nifi-elasticsearch-5-processors/src/main/java/org/apache/nifi/processors/elasticsearch

[GitHub] nifi pull request #2180: Added GetMongoAggregation to support running Mongo ...

2017-10-05 Thread MikeThomsen
Github user MikeThomsen commented on a diff in the pull request: https://github.com/apache/nifi/pull/2180#discussion_r142912872 --- Diff: nifi-nar-bundles/nifi-mongodb-bundle/nifi-mongodb-processors/src/main/java/org/apache/nifi/processors/mongodb/AbstractMongoProcessor.java

[GitHub] nifi issue #2125: NIFI-4346 Created a LookupService that uses HBase as its b...

2017-10-06 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2125 @bbende all of the changes you requested were added. ---

[GitHub] nifi issue #2180: Added GetMongoAggregation to support running Mongo aggrega...

2017-10-17 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2180 @mattyb149 @markap14 @milanchandna Do any of you have some time to do a quick look to see if this can get merged? ---

[GitHub] nifi issue #2113: NIFI-4325 Added new processor that uses the JSON DSL.

2017-10-17 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2113 @mattyb149 Ok. Changes are made. I refactored the commit to be based on a controller service. For now, that service only handles a single function: basic search. However, there is now

[GitHub] nifi issue #2113: NIFI-4325 Added new processor that uses the JSON DSL.

2017-10-13 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2113 @mattyb149 I'm going to leave this open, but I decided to refactor the heck out of it around a client service for ElasticSearch. The service only has one method for now, but I think it's the way

[GitHub] nifi pull request #2125: NIFI-4346 Created a LookupService that uses HBase a...

2017-09-11 Thread MikeThomsen
Github user MikeThomsen commented on a diff in the pull request: https://github.com/apache/nifi/pull/2125#discussion_r138158414 --- Diff: nifi-nar-bundles/nifi-standard-services/nifi-hbase_1_1_2-client-service-bundle/nifi-hbase_1_1_2-client-service/src/main/java/org/apache/nifi/hbase

[GitHub] nifi issue #2125: NIFI-4346 Created a LookupService that uses HBase as its b...

2017-09-11 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2125 @markap14 All of the changes should now be checked in. ---

[GitHub] nifi issue #2123: NIFI-4345 Added a MongoDB controller service and a lookup ...

2017-09-11 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2123 @markap14 Updated based on your feedback and added the ability to merge either a single field or the entire Mongo result minus the _id field. ---

[GitHub] nifi pull request #2123: NIFI-4345 Added a MongoDB controller service and a ...

2017-09-05 Thread MikeThomsen
Github user MikeThomsen commented on a diff in the pull request: https://github.com/apache/nifi/pull/2123#discussion_r137059065 --- Diff: nifi-nar-bundles/nifi-standard-services/nifi-mongodb-services-bundle/nifi-mongodb-services-nar/src/main/resources/META-INF/NOTICE --- @@ -0,0

[GitHub] nifi pull request #2123: NIFI-4345 Added a MongoDB controller service and a ...

2017-09-07 Thread MikeThomsen
Github user MikeThomsen commented on a diff in the pull request: https://github.com/apache/nifi/pull/2123#discussion_r137559094 --- Diff: nifi-nar-bundles/nifi-standard-services/nifi-mongodb-services-bundle/nifi-mongodb-services/src/main/java/org/apache/nifi/mongodb

[GitHub] nifi issue #2123: NIFI-4345 Added a MongoDB controller service and a lookup ...

2017-09-07 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2123 @mattyb149 Ok should be good to go now. ---

[GitHub] nifi pull request #2180: Added GetMongoAggregation to support running Mongo ...

2017-09-27 Thread MikeThomsen
GitHub user MikeThomsen opened a pull request: https://github.com/apache/nifi/pull/2180 Added GetMongoAggregation to support running Mongo aggregations. Thank you for submitting a contribution to Apache NiFi. In order to streamline the review of the contribution we ask you

[GitHub] nifi pull request #2180: Added GetMongoAggregation to support running Mongo ...

2017-10-03 Thread MikeThomsen
Github user MikeThomsen commented on a diff in the pull request: https://github.com/apache/nifi/pull/2180#discussion_r142458671 --- Diff: nifi-nar-bundles/nifi-mongodb-bundle/nifi-mongodb-processors/src/main/java/org/apache/nifi/processors/mongodb/AbstractMongoProcessor.java

[GitHub] nifi issue #2063: NIFI-4269 Added the ability to serialize Mongo documents t...

2017-08-21 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2063 @mattyb149 @pvillard31 @bbende @joewitt Could one of you take a look at this and give me some feedback? I can fix the merge conflicts that came up since I submitted this. --- If your project

[GitHub] nifi pull request #2113: NIFI-4325 Added new processor that uses the JSON DS...

2017-08-26 Thread MikeThomsen
GitHub user MikeThomsen opened a pull request: https://github.com/apache/nifi/pull/2113 NIFI-4325 Added new processor that uses the JSON DSL. Thank you for submitting a contribution to Apache NiFi. In order to streamline the review of the contribution we ask you

[GitHub] nifi pull request #2104: NIFI-4307 Added Kotlin 1.1.X support to ExecuteScri...

2017-08-21 Thread MikeThomsen
GitHub user MikeThomsen opened a pull request: https://github.com/apache/nifi/pull/2104 NIFI-4307 Added Kotlin 1.1.X support to ExecuteScript. Thank you for submitting a contribution to Apache NiFi. In order to streamline the review of the contribution we ask you

[GitHub] nifi pull request #2123: NIFI-4345 Added a MongoDB controller service and a ...

2017-09-01 Thread MikeThomsen
GitHub user MikeThomsen opened a pull request: https://github.com/apache/nifi/pull/2123 NIFI-4345 Added a MongoDB controller service and a lookup service. Thank you for submitting a contribution to Apache NiFi. In order to streamline the review of the contribution we ask

[GitHub] nifi pull request #2125: NIFI-4346 Created a LookupService that uses HBase a...

2017-09-02 Thread MikeThomsen
GitHub user MikeThomsen opened a pull request: https://github.com/apache/nifi/pull/2125 NIFI-4346 Created a LookupService that uses HBase as its back end. Thank you for submitting a contribution to Apache NiFi. In order to streamline the review of the contribution we ask

[GitHub] nifi issue #2063: NIFI-4269 Added the ability to serialize Mongo documents t...

2017-08-29 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2063 @mattyb149 Can you take a look? 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 this feature

[GitHub] nifi issue #2063: NIFI-4269 Added the ability to serialize Mongo documents t...

2017-09-05 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2063 @mattyb149 Do you think you'll have a chance to review this this week? ---

[GitHub] nifi issue #2113: NIFI-4325 Added new processor that uses the JSON DSL.

2017-10-08 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2113 I discovered a slick plugin for Maven that lets you cleanly run ElasticSearch in the background during integration tests. I added that to the test setup under a profile. So I have some cleanup

[GitHub] nifi pull request #2180: Added GetMongoAggregation to support running Mongo ...

2017-10-05 Thread MikeThomsen
Github user MikeThomsen commented on a diff in the pull request: https://github.com/apache/nifi/pull/2180#discussion_r143060642 --- Diff: nifi-nar-bundles/nifi-mongodb-bundle/nifi-mongodb-processors/src/main/java/org/apache/nifi/processors/mongodb/AbstractMongoProcessor.java

[GitHub] nifi issue #2113: NIFI-4325 Added new processor that uses the JSON DSL.

2017-10-07 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2113 Thanks. I'll get to these some time today or tomorrow. FYI Elastic has some official statements on the future direction of the Java client here: https://www.elastic.co/blog/state

[GitHub] nifi pull request #2161: NIFI-4333: Providing Docker support of the NiFi Too...

2017-11-28 Thread MikeThomsen
Github user MikeThomsen commented on a diff in the pull request: https://github.com/apache/nifi/pull/2161#discussion_r153581396 --- Diff: nifi-toolkit/nifi-toolkit-assembly/docker/Dockerfile --- @@ -0,0 +1,48 @@ +# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] nifi pull request #2304: NIFI-4644 Fixed LookupService API to allow for more...

2017-11-28 Thread MikeThomsen
GitHub user MikeThomsen opened a pull request: https://github.com/apache/nifi/pull/2304 NIFI-4644 Fixed LookupService API to allow for more than String/Strin… …g lookup pairs. Thank you for submitting a contribution to Apache NiFi. In order to streamline

[GitHub] nifi pull request #2295: NIFI-4479 Added DeleteMongo processor.

2017-11-25 Thread MikeThomsen
GitHub user MikeThomsen opened a pull request: https://github.com/apache/nifi/pull/2295 NIFI-4479 Added DeleteMongo processor. Thank you for submitting a contribution to Apache NiFi. In order to streamline the review of the contribution we ask you to ensure

[GitHub] nifi pull request #2161: NIFI-4333: Providing Docker support of the NiFi Too...

2017-11-24 Thread MikeThomsen
Github user MikeThomsen commented on a diff in the pull request: https://github.com/apache/nifi/pull/2161#discussion_r152943230 --- Diff: nifi-toolkit/nifi-toolkit-assembly/docker/Dockerfile --- @@ -0,0 +1,48 @@ +# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] nifi pull request #2161: NIFI-4333: Providing Docker support of the NiFi Too...

2017-11-24 Thread MikeThomsen
Github user MikeThomsen commented on a diff in the pull request: https://github.com/apache/nifi/pull/2161#discussion_r152944214 --- Diff: nifi-toolkit/nifi-toolkit-assembly/docker/Dockerfile --- @@ -0,0 +1,48 @@ +# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] nifi pull request #2294: NIFI-3538 Added DeleteHBaseRow

2017-11-24 Thread MikeThomsen
GitHub user MikeThomsen opened a pull request: https://github.com/apache/nifi/pull/2294 NIFI-3538 Added DeleteHBaseRow Thank you for submitting a contribution to Apache NiFi. In order to streamline the review of the contribution we ask you to ensure the following steps

[GitHub] nifi issue #2304: NIFI-4644 Fixed LookupService API to allow for more than S...

2017-11-29 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2304 Ok, code review notes should all be addressed now. ---

[GitHub] nifi issue #2246: NIFI-4559: Add non-zero status relationship to ExecuteStre...

2017-11-29 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2246 @mattyb149 As mentioned, I tried this out and got it to work. So I see no reason it cannot be merged. ---

[GitHub] nifi issue #2304: NIFI-4644 Fixed LookupService API to allow for more than S...

2017-11-29 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2304 Thanks @markap14 I'll fix these and push a new version. ---

[GitHub] nifi issue #2304: NIFI-4644 Fixed LookupService API to allow for more than S...

2017-11-29 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2304 @markap14 That's fine. I posted on the mailing list that someone already found a real case where it broke their flow because it converted an integer to a string and MongoDB couldn't do the lookup

[GitHub] nifi pull request #2130: NIFI-3970: Add CSVRecordLookupService

2017-11-29 Thread MikeThomsen
Github user MikeThomsen commented on a diff in the pull request: https://github.com/apache/nifi/pull/2130#discussion_r153848713 --- Diff: nifi-nar-bundles/nifi-standard-services/nifi-lookup-services-bundle/nifi-lookup-services/src/main/java/org/apache/nifi/lookup

[GitHub] nifi pull request #2130: NIFI-3970: Add CSVRecordLookupService

2017-11-29 Thread MikeThomsen
Github user MikeThomsen commented on a diff in the pull request: https://github.com/apache/nifi/pull/2130#discussion_r153841552 --- Diff: nifi-nar-bundles/nifi-standard-services/nifi-lookup-services-bundle/nifi-lookup-services/src/main/java/org/apache/nifi/lookup

[GitHub] nifi pull request #2130: NIFI-3970: Add CSVRecordLookupService

2017-11-29 Thread MikeThomsen
Github user MikeThomsen commented on a diff in the pull request: https://github.com/apache/nifi/pull/2130#discussion_r153854915 --- Diff: nifi-nar-bundles/nifi-standard-services/nifi-lookup-services-bundle/nifi-lookup-services/src/main/java/org/apache/nifi/lookup

[GitHub] nifi pull request #2130: NIFI-3970: Add CSVRecordLookupService

2017-11-29 Thread MikeThomsen
Github user MikeThomsen commented on a diff in the pull request: https://github.com/apache/nifi/pull/2130#discussion_r153855193 --- Diff: nifi-nar-bundles/nifi-standard-services/nifi-lookup-services-bundle/nifi-lookup-services/src/main/java/org/apache/nifi/lookup

[GitHub] nifi pull request #2130: NIFI-3970: Add CSVRecordLookupService

2017-11-29 Thread MikeThomsen
Github user MikeThomsen commented on a diff in the pull request: https://github.com/apache/nifi/pull/2130#discussion_r153848468 --- Diff: nifi-nar-bundles/nifi-standard-services/nifi-lookup-services-bundle/nifi-lookup-services/src/main/java/org/apache/nifi/lookup

[GitHub] nifi pull request #2130: NIFI-3970: Add CSVRecordLookupService

2017-11-29 Thread MikeThomsen
Github user MikeThomsen commented on a diff in the pull request: https://github.com/apache/nifi/pull/2130#discussion_r153845391 --- Diff: nifi-nar-bundles/nifi-standard-services/nifi-lookup-services-bundle/nifi-lookup-services/src/main/java/org/apache/nifi/lookup

[GitHub] nifi pull request #2307: NIFI-4649 Added FlattenJson processor.

2017-11-30 Thread MikeThomsen
GitHub user MikeThomsen opened a pull request: https://github.com/apache/nifi/pull/2307 NIFI-4649 Added FlattenJson processor. Thank you for submitting a contribution to Apache NiFi. In order to streamline the review of the contribution we ask you to ensure

[GitHub] nifi pull request #2292: NIFI-4639: fresh writer for each output record

2017-11-28 Thread MikeThomsen
Github user MikeThomsen commented on a diff in the pull request: https://github.com/apache/nifi/pull/2292#discussion_r153658187 --- Diff: nifi-nar-bundles/nifi-kafka-bundle/nifi-kafka-0-10-processors/src/main/java/org/apache/nifi/processors/kafka/pubsub/PublisherLease.java

[GitHub] nifi pull request #2292: NIFI-4639: fresh writer for each output record

2017-11-28 Thread MikeThomsen
Github user MikeThomsen commented on a diff in the pull request: https://github.com/apache/nifi/pull/2292#discussion_r153658243 --- Diff: nifi-nar-bundles/nifi-kafka-bundle/nifi-kafka-0-10-processors/src/test/java/org/apache/nifi/processors/kafka/pubsub/TestPublisherLease.java

[GitHub] nifi pull request #2294: NIFI-3538 Added DeleteHBaseRow

2017-12-17 Thread MikeThomsen
Github user MikeThomsen commented on a diff in the pull request: https://github.com/apache/nifi/pull/2294#discussion_r157368931 --- Diff: nifi-nar-bundles/nifi-hbase-bundle/nifi-hbase-processors/src/main/java/org/apache/nifi/hbase/DeleteHBaseRow.java --- @@ -0,0 +1,178

[GitHub] nifi pull request #2294: NIFI-3538 Added DeleteHBaseRow

2017-12-17 Thread MikeThomsen
Github user MikeThomsen commented on a diff in the pull request: https://github.com/apache/nifi/pull/2294#discussion_r157368968 --- Diff: nifi-nar-bundles/nifi-hbase-bundle/nifi-hbase-processors/src/main/java/org/apache/nifi/hbase/DeleteHBaseRow.java --- @@ -0,0 +1,178

[GitHub] nifi pull request #2294: NIFI-3538 Added DeleteHBaseRow

2017-12-17 Thread MikeThomsen
Github user MikeThomsen commented on a diff in the pull request: https://github.com/apache/nifi/pull/2294#discussion_r157368551 --- Diff: nifi-nar-bundles/nifi-hbase-bundle/nifi-hbase-processors/src/main/java/org/apache/nifi/hbase/DeleteHBaseRow.java --- @@ -0,0 +1,178

[GitHub] nifi pull request #2294: NIFI-3538 Added DeleteHBaseRow

2017-12-17 Thread MikeThomsen
Github user MikeThomsen commented on a diff in the pull request: https://github.com/apache/nifi/pull/2294#discussion_r157368648 --- Diff: nifi-nar-bundles/nifi-hbase-bundle/nifi-hbase-processors/src/main/java/org/apache/nifi/hbase/DeleteHBaseRow.java --- @@ -0,0 +1,178

[GitHub] nifi issue #2295: NIFI-4479 Added DeleteMongo processor.

2017-12-12 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2295 Just checked in the updates. ---

[GitHub] nifi issue #2304: NIFI-4644 Fixed LookupService API to allow for more than S...

2017-12-13 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2304 @markap14 Have you had a chance to think more this? FWIW, it doesn't seem to break any of the unit tests. ---

[GitHub] nifi pull request #2294: NIFI-3538 Added DeleteHBaseRow

2017-12-17 Thread MikeThomsen
Github user MikeThomsen commented on a diff in the pull request: https://github.com/apache/nifi/pull/2294#discussion_r157383920 --- Diff: nifi-nar-bundles/nifi-hbase-bundle/nifi-hbase-processors/src/main/java/org/apache/nifi/hbase/DeleteHBaseRow.java --- @@ -0,0 +1,178

[GitHub] nifi pull request #2294: NIFI-3538 Added DeleteHBaseRow

2017-12-17 Thread MikeThomsen
Github user MikeThomsen commented on a diff in the pull request: https://github.com/apache/nifi/pull/2294#discussion_r157383902 --- Diff: nifi-nar-bundles/nifi-hbase-bundle/nifi-hbase-processors/src/main/java/org/apache/nifi/hbase/DeleteHBaseRow.java --- @@ -0,0 +1,178

[GitHub] nifi issue #2180: Added GetMongoAggregation to support running Mongo aggrega...

2017-11-10 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2180 @mbolka I think the PMC are backlogged. If you really need this, try asking someone on the dev mailing list to jump in on it. I'll have time in the next day or two to make any updates

[GitHub] nifi issue #2113: NIFI-4325 Added new processor that uses the JSON DSL.

2017-11-12 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2113 @mattyb149 I think everything except the proxy issue is now in there. ---

  1   2   3   4   5   6   7   8   9   10   >