markap14 commented on PR #11164: URL: https://github.com/apache/nifi/pull/11164#issuecomment-5639879817
[GPT-5.6 Sol] @joewitt thanks for the detailed review. The first two findings are addressed in the latest commits: - a87ca6d6852 clears component interrupt status at the framework boundary after each Processor or Reporting Task invocation. The new regression tests failed on the prior code exactly as described—the second invocation never occurred—and now pass. - 300f0c68e06 adds `AUTO` as the default scheduling strategy. It resolves to `STANDARD` below Java 25 and `VIRTUAL` on Java 25 or newer; explicit `STANDARD` and `VIRTUAL` values still override it. The administration guide also warns about Java 21 monitor pinning. Your permit-wait polling note is also accurate. Eliminating that polling requires targeted wake-up of threads waiting for permits without interrupting a Processor currently in `onTrigger`, so I am reviewing that concurrency change separately with Mark. -- 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]
