pvillard31 commented on PR #9445: URL: https://github.com/apache/nifi/pull/9445#issuecomment-2438387137
Thanks @turcsanyip, that makes total sense and thanks @greyp9 for testing and discussions on the Slack thread. If we prefer to just use a synchronized list instead, I'm totally OK with this approach but, as I said before, I don't really see the value of maintaining successes and failures lists as we're not really doing anything with it (like logging number of failures vs number of successes, etc). If we prefer to maintain the current approach and change to a synchronized list, I do think a bit of reworking is still needed to expose the logs from FlowFileResult as processor level bulletins - right now, routing flow files to failure or retry would not trigger any bulletin. Happy to file another PR if that's the consensus. -- 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]
