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

Posted to issues@nifi.apache.org by "mh013370 (via GitHub)" <gi...@apache.org> on 2023/03/08 17:25:37 UTC

[GitHub] [nifi] mh013370 commented on pull request #7016: [NIFI-10792] Fixed bug to allow for processing files larger than 10MB…

mh013370 commented on PR #7016:
URL: https://github.com/apache/nifi/pull/7016#issuecomment-1460537814

   > @exceptionfactory I did not end up including the unit test I had as it was a unit which tested with a 20MB file. I would have thought there should be a unit test to exercise the change I made. Please advise.
   
   Anecdotally, the NiFi mock framework will read the entire FF contents into memory (I discovered this in #6369) so I think you're correct in not including unit tests testing larger FFs.


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