[GitHub] nifi pull request #2890: NIFI-5427: Updating ScrollElasticsearchHttp to use ...

2018-07-13 Thread gresockj
GitHub user gresockj opened a pull request: https://github.com/apache/nifi/pull/2890 NIFI-5427: Updating ScrollElasticsearchHttp to use POST, supporting ES6 Thank you for submitting a contribution to Apache NiFi. In order to streamline the review of the contribution we ask

[GitHub] nifi pull request #2133: NIFI-4361: Fixing early recovery shutdown due to EO...

2017-09-08 Thread gresockj
Github user gresockj commented on a diff in the pull request: https://github.com/apache/nifi/pull/2133#discussion_r137839524 --- Diff: nifi-commons/nifi-write-ahead-log/src/main/java/org/wali/MinimalLockingWriteAheadLog.java --- @@ -474,8 +474,15 @@ private void recoverFromEdits

[GitHub] nifi pull request #1887: NIFI-4014: Correcting Event Type text in S2S proven...

2017-06-03 Thread gresockj
GitHub user gresockj opened a pull request: https://github.com/apache/nifi/pull/1887 NIFI-4014: Correcting Event Type text in S2S provenance reporting task Thank you for submitting a contribution to Apache NiFi. In order to streamline the review of the contribution we ask

[GitHub] nifi issue #1872: NIFI-106: Expose processors' counters in Stats History

2017-05-30 Thread gresockj
Github user gresockj commented on the issue: https://github.com/apache/nifi/pull/1872 If you think it's appropriate to include in this ticket, you could also update the SiteToSiteStatusReportingTask to include the counters in the serialized JSON. --- If your project is set up

[GitHub] nifi issue #1786: NIFI-2199 & NIFI-3112 - Handle propagation of stderr and a...

2017-05-12 Thread gresockj
Github user gresockj commented on the issue: https://github.com/apache/nifi/pull/1786 Salt is easy to set up with this [walkthrough](https://docs.saltstack.com/en/latest/topics/tutorials/walkthrough.html). You can run salt-master and salt-minion on the same nifi box if you want

[GitHub] nifi issue #1786: NIFI-2199 & NIFI-3112 - Handle propagation of stderr and a...

2017-05-12 Thread gresockj
Github user gresockj commented on the issue: https://github.com/apache/nifi/pull/1786 I tried this out, and it does appear to work when I run ssh user@remote-host "sudo service nifi restart". However, when I do the same with a salt command, it still hangs: salt ni

[GitHub] nifi pull request #1671: NIFI-3703: Adding 'Allow Anonymous Credentials' pro...

2017-04-13 Thread gresockj
Github user gresockj closed the pull request at: https://github.com/apache/nifi/pull/1671 --- 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 #1671: NIFI-3703: Adding 'Allow Anonymous Credentials' pro...

2017-04-13 Thread gresockj
GitHub user gresockj opened a pull request: https://github.com/apache/nifi/pull/1671 NIFI-3703: Adding 'Allow Anonymous Credentials' property to AWS processors Thank you for submitting a contribution to Apache NiFi. In order to streamline the review of the contribution we

[GitHub] nifi pull request #1659: [NIFI-3035] use api supported .entries() to iterate...

2017-04-11 Thread gresockj
Github user gresockj commented on a diff in the pull request: https://github.com/apache/nifi/pull/1659#discussion_r110865211 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/canvas/nf-canvas-utils.js --- @@ -412,9 +412,9

[GitHub] nifi pull request #1659: [NIFI-3035] use api supported .entries() to iterate...

2017-04-08 Thread gresockj
Github user gresockj commented on a diff in the pull request: https://github.com/apache/nifi/pull/1659#discussion_r110516800 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/canvas/nf-canvas-utils.js --- @@ -412,9 +412,9

[GitHub] nifi pull request #1659: [NIFI-3035] use api supported .entries() to iterate...

2017-04-08 Thread gresockj
Github user gresockj commented on a diff in the pull request: https://github.com/apache/nifi/pull/1659#discussion_r110516478 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/canvas/nf-canvas-utils.js --- @@ -412,9 +412,9

[GitHub] nifi pull request #1660: NIFI-3674: Implementing SiteToSiteStatusReportingTa...

2017-04-07 Thread gresockj
GitHub user gresockj opened a pull request: https://github.com/apache/nifi/pull/1660 NIFI-3674: Implementing SiteToSiteStatusReportingTask 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 #1592: NIFI-3582: Fixing PersistentMapCache bug that skipp...

2017-03-13 Thread gresockj
GitHub user gresockj opened a pull request: https://github.com/apache/nifi/pull/1592 NIFI-3582: Fixing PersistentMapCache bug that skipped persistent evictions Thank you for submitting a contribution to Apache NiFi. In order to streamline the review of the contribution we

[GitHub] nifi pull request #1454: NIFI-3419: Routing flow files causing ZipException ...

2017-01-30 Thread gresockj
GitHub user gresockj opened a pull request: https://github.com/apache/nifi/pull/1454 NIFI-3419: Routing flow files causing ZipException to failure in MergeContent Thank you for submitting a contribution to Apache NiFi. In order to streamline the review of the contribution

[GitHub] nifi pull request #1398: NIFI-3275: Correcting SocketChannelSender log warni...

2017-01-05 Thread gresockj
GitHub user gresockj opened a pull request: https://github.com/apache/nifi/pull/1398 NIFI-3275: Correcting SocketChannelSender log warning message 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 #1359: NIFI-3253: Correcting documentation in Notify proce...

2016-12-23 Thread gresockj
GitHub user gresockj opened a pull request: https://github.com/apache/nifi/pull/1359 NIFI-3253: Correcting documentation in Notify processor 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 #1358: NIFI-3127: Fixing NPE for empty flow files in Split...

2016-12-23 Thread gresockj
GitHub user gresockj opened a pull request: https://github.com/apache/nifi/pull/1358 NIFI-3127: Fixing NPE for empty flow files in SplitText Thank you for submitting a contribution to Apache NiFi. In order to streamline the review of the contribution we ask you

[GitHub] nifi issue #1329: NIFI-190: Initial commit of Wait and Notify processors

2016-12-16 Thread gresockj
Github user gresockj commented on the issue: https://github.com/apache/nifi/pull/1329 I like the idea about waiting for N signals. I think it can be done by adding a getKeys(Pattern) method to the distributed cache, so maybe it's best captured in a separate ticket. --- If your

[GitHub] nifi pull request #1329: NIFI-190: Initial commit of Wait and Notify process...

2016-12-16 Thread gresockj
Github user gresockj commented on a diff in the pull request: https://github.com/apache/nifi/pull/1329#discussion_r92813543 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/Notify.java --- @@ -0,0 +1,191

[GitHub] nifi pull request #1329: NIFI-190: Initial commit of Wait and Notify process...

2016-12-14 Thread gresockj
GitHub user gresockj opened a pull request: https://github.com/apache/nifi/pull/1329 NIFI-190: Initial commit of Wait and Notify processors 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 issue #1317: NIFI-3186: Fixed NPE that was overlooked

2016-12-13 Thread gresockj
Github user gresockj commented on the issue: https://github.com/apache/nifi/pull/1317 This has been running for a day with lots of flow files and several restarts, and I have not encountered the issue yet. --- If your project is set up for it, you can reply to this email and have

[GitHub] nifi issue #1317: NIFI-3186: Fixed NPE that was overlooked

2016-12-12 Thread gresockj
Github user gresockj commented on the issue: https://github.com/apache/nifi/pull/1317 It looks like it should fix the issue. I have deployed it to my cluster, and I'll monitor it today and let you know tomorrow if I have anymore problems. --- If your project is set up for it, you

[GitHub] nifi issue #1245: Nifi 3066

2016-11-19 Thread gresockj
Github user gresockj commented on the issue: https://github.com/apache/nifi/pull/1245 Hi @markap14 , thanks for submitting this patch. I tested it out on my cluster and it does not appear to resolve the MergeContent problem in my case. I'm hoping I deployed the fix correctly: I

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

2016-10-21 Thread gresockj
Github user gresockj commented on the issue: https://github.com/apache/nifi/pull/ I tested this patch on a NiFi 1.0.0 instance and it resolves NIFI-2848! Thanks Pierre. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

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

2016-10-21 Thread gresockj
Github user gresockj commented on the issue: https://github.com/apache/nifi/pull/ Yes, I will take a look 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 does not have this feature enabled

[GitHub] nifi pull request #733: NIFI-2417: Implementing QueryElasticsearchHttp and S...

2016-09-22 Thread gresockj
Github user gresockj closed the pull request at: https://github.com/apache/nifi/pull/733 --- 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 #733: NIFI-2417: Implementing QueryElasticsearchHttp and ScrollEl...

2016-09-22 Thread gresockj
Github user gresockj commented on the issue: https://github.com/apache/nifi/pull/733 Thanks for the help! --- 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 #733: NIFI-2417: Implementing QueryElasticsearchHttp and S...

2016-09-22 Thread gresockj
Github user gresockj commented on a diff in the pull request: https://github.com/apache/nifi/pull/733#discussion_r80111838 --- Diff: nifi-nar-bundles/nifi-elasticsearch-bundle/nifi-elasticsearch-processors/src/main/java/org/apache/nifi/processors/elasticsearch

[GitHub] nifi pull request #733: NIFI-2417: Implementing QueryElasticsearchHttp and S...

2016-09-22 Thread gresockj
Github user gresockj commented on a diff in the pull request: https://github.com/apache/nifi/pull/733#discussion_r80111550 --- Diff: nifi-nar-bundles/nifi-elasticsearch-bundle/nifi-elasticsearch-processors/src/main/java/org/apache/nifi/processors/elasticsearch

[GitHub] nifi pull request #733: NIFI-2417: Implementing QueryElasticsearchHttp and S...

2016-09-21 Thread gresockj
Github user gresockj commented on a diff in the pull request: https://github.com/apache/nifi/pull/733#discussion_r79914319 --- Diff: nifi-nar-bundles/nifi-elasticsearch-bundle/nifi-elasticsearch-processors/src/main/java/org/apache/nifi/processors/elasticsearch

[GitHub] nifi pull request #733: NIFI-2417: Implementing QueryElasticsearchHttp and S...

2016-09-21 Thread gresockj
Github user gresockj commented on a diff in the pull request: https://github.com/apache/nifi/pull/733#discussion_r79913709 --- Diff: nifi-nar-bundles/nifi-elasticsearch-bundle/nifi-elasticsearch-processors/src/main/java/org/apache/nifi/processors/elasticsearch

[GitHub] nifi issue #733: NIFI-2417: Implementing QueryElasticsearchHttp and ScrollEl...

2016-09-21 Thread gresockj
Github user gresockj commented on the issue: https://github.com/apache/nifi/pull/733 Thanks @mattyb149 ! Good comments, I'll see if I can work on these this week. --- 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 #917: NIFI-2631: Adding 'Commit Mode' and 'Use Versions' to ListS...

2016-09-02 Thread gresockj
Github user gresockj commented on the issue: https://github.com/apache/nifi/pull/917 Thanks for checking that out, gives me some more confidence that it's unrelated to this. For now I'll proceed with the ITListS3 tests when I get a chance. --- If your project is set up for it, you

[GitHub] nifi issue #917: NIFI-2631: Adding 'Commit Mode' and 'Use Versions' to ListS...

2016-09-02 Thread gresockj
Github user gresockj commented on the issue: https://github.com/apache/nifi/pull/917 Yes, it's requesting this: https://localhost:8443/nifi-api/controller/process-groups/66ef7ae5-4bdb-4c50-9811-429be759f6fb/processors/7738f82e-d185-3931-a98e-c1156cd21cbc/state

[GitHub] nifi issue #917: NIFI-2631: Adding 'Commit Mode' and 'Use Versions' to ListS...

2016-09-02 Thread gresockj
Github user gresockj commented on the issue: https://github.com/apache/nifi/pull/917 Hmm, so this is strange. I was testing ListS3 out in NiFi today and I noticed that I cannot see the state in the View State window. However, the processor behaves as if the state is being persisted

[GitHub] nifi pull request #917: NIFI-2631: Adding 'Commit Mode' and 'Use Versions' t...

2016-08-23 Thread gresockj
GitHub user gresockj opened a pull request: https://github.com/apache/nifi/pull/917 NIFI-2631: Adding 'Commit Mode' and 'Use Versions' to ListS3 You can merge this pull request into a Git repository by running: $ git pull https://github.com/gresockj/nifi NIFI-2631-master

[GitHub] nifi pull request #916: NIFI-2631: Adding 'Commit Mode' and 'Use Versions' t...

2016-08-23 Thread gresockj
GitHub user gresockj opened a pull request: https://github.com/apache/nifi/pull/916 NIFI-2631: Adding 'Commit Mode' and 'Use Versions' to ListS3 You can merge this pull request into a Git repository by running: $ git pull https://github.com/gresockj/nifi NIFI-2631

[GitHub] nifi issue #905: NIFI-2611: Fixing bugs in UnpackContent with flow file unpa...

2016-08-22 Thread gresockj
Github user gresockj commented on the issue: https://github.com/apache/nifi/pull/905 @mosermw Go ahead and squash away! --- 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 #905: NIFI-2611: Fixing bugs in UnpackContent with flow fi...

2016-08-22 Thread gresockj
Github user gresockj commented on a diff in the pull request: https://github.com/apache/nifi/pull/905#discussion_r75700388 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/UnpackContent.java --- @@ -182,15

[GitHub] nifi pull request #905: NIFI-2611: Fixing bugs in UnpackContent with flow fi...

2016-08-22 Thread gresockj
Github user gresockj commented on a diff in the pull request: https://github.com/apache/nifi/pull/905#discussion_r75703219 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/UnpackContent.java --- @@ -536,7

[GitHub] nifi pull request #905: NIFI-2611: Fixing bugs in UnpackContent with flow fi...

2016-08-22 Thread gresockj
Github user gresockj commented on a diff in the pull request: https://github.com/apache/nifi/pull/905#discussion_r75703161 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/test/java/org/apache/nifi/processors/standard/TestUnpackContent.java

[GitHub] nifi pull request #733: NIFI-2417: Implementing QueryElasticsearchHttp and S...

2016-07-28 Thread gresockj
GitHub user gresockj opened a pull request: https://github.com/apache/nifi/pull/733 NIFI-2417: Implementing QueryElasticsearchHttp and ScrollElasticsearchHttp I have implemented these processors for my own project, and thought it might be useful to submit them to NiFi