[GitHub] nifi issue #2945: NIFI-4517: Added ExecuteSQLRecord and QueryDatabaseTableRe...

2018-10-06 Thread pvillard31
Github user pvillard31 commented on the issue: https://github.com/apache/nifi/pull/2945 Gave it another try, all LGTM, merging to master, thanks much @mattyb149 ---

[GitHub] nifi issue #2945: NIFI-4517: Added ExecuteSQLRecord and QueryDatabaseTableRe...

2018-10-04 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/2945 @pvillard31 @MikeThomsen Think we can get the final review done in time for 1.8.0? Folks have been asking for these processors for a long time :) ---

[GitHub] nifi issue #2945: NIFI-4517: Added ExecuteSQLRecord and QueryDatabaseTableRe...

2018-09-25 Thread pvillard31
Github user pvillard31 commented on the issue: https://github.com/apache/nifi/pull/2945 I've been trying the two processors in my usual test workflows replacing ExecuteSQL with the record version (and similarly with QueryDatabaseTable) and it works fine. Once Mike's comments are addre