pgyori commented on PR #9182: URL: https://github.com/apache/nifi/pull/9182#issuecomment-2293531390
Thank you for your review @EndzeitBegins ! It turns out this might not be what we want after all. This yields every time None is returned and we might want to be able to yield from the Python code instead. So I'm reiterating the idea. As for unit tests: I could not find a way to insert a ProcessContext mock to check for yield() call, apparently it isn't tested anywhere, so I figured it's ok to leave it out. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
