markap14 commented on pull request #5042: URL: https://github.com/apache/nifi/pull/5042#issuecomment-843476232
@gresockj thanks for all the review feedback! Now that I haven't looked at my code in a while I did start refresh giving myself a full code review. I think everything looked good except that I found a situation in which the State Manager could be updated in stateless even when a downstream processor throw an Exception. I created a system test to verify it, and then I fixed that. You mind giving the latest commit a once-over also? -- 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: us...@infra.apache.org