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

2016-07-19 Thread JPercivall
Github user JPercivall commented on the issue: https://github.com/apache/nifi/pull/642 +1 Visually verified code and any comments were addressed. Ran a contrib check build and verified functionality in a standalone cluster hitting a MySQL DB. Thanks @mattyb149, I will squash

[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 tables fetched

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

2016-07-18 Thread JPercivall
Github user JPercivall commented on the issue: https://github.com/apache/nifi/pull/642 Alright, after giving it some time and a cup of coffee I realize how off I was at first, lol. This processor reaches out to the DB asking for the tables. Then for each table that isn't already store