You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@nifi.apache.org by GitBox <gi...@apache.org> on 2021/01/19 18:25:12 UTC

[GitHub] [nifi] mattyb149 commented on pull request #4763: NIFI-8146: Added RecordPath for Statement Type and to specify where to find the data within the incoming Record. Also performed some minor code refactoring to avoid lots of large lambdas, as it results in code that is more readable and more maintainable. Updated some properties to use dependsOn() notation now that this is available.

mattyb149 commented on pull request #4763:
URL: https://github.com/apache/nifi/pull/4763#issuecomment-763033574


   +1 LGTM, test failures are from Checkstyle which I will fix on merge. Ran contrib-check and tried on a live system with Postgres and Oracle DBs, with the statement type both inside and outside the data path. Thanks for the improvement! Merging to main


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org