joewitt commented on PR #11164: URL: https://github.com/apache/nifi/pull/11164#issuecomment-5648896619
ok @markap14 this was a lot of fun to review and experiment with. While I dont feel confident enough abou the true code itself and would like someone such as @exceptionfactory to give it a final review - i am +1 at this point after reasonably detailed testing and experimentation. Various results shared about. The first large markdown dump is from going beyond this and suggesting some follow-on changes. The second big block is from really focusing on behavior of scheduling/thread starvation/timing concerns. It holds up well. I did my testing on Java 25. I didn't focus on Java 21. Since we are default off on Java 21 and default on with Java 25 I think that is fine. I also ran the entire test suite/portfolio over the span of a copule hours and everything came up green. -- 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]
