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

Posted to issues@nifi.apache.org by "exceptionfactory (via GitHub)" <gi...@apache.org> on 2023/05/24 00:10:31 UTC

[GitHub] [nifi] exceptionfactory commented on pull request #6816: NIFI-9206: Add RemoveRecordField processor and implement the ability to remove fields from records

exceptionfactory commented on PR #6816:
URL: https://github.com/apache/nifi/pull/6816#issuecomment-1560279105

   Thanks for the summary @ChrisSamo632.
   
   Looking over the current PR, there are a number changes that apply to other Processors. Although they are useful in themselves, they should be split off to separate PRs where possible.
   
   This new PR also includes a very large number of test files. Do you think all of those are necessary? The size and scope seem larger than needed at a glance, but I have not yet given the tests a thorough review.


-- 
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.

To unsubscribe, e-mail: issues-unsubscribe@nifi.apache.org

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