mosermw commented on PR #7929: URL: https://github.com/apache/nifi/pull/7929#issuecomment-1781646495
You make some very thoughtful points @exceptionfactory and it makes sense to provide fine grained error handling if the underlying library provides different error codes. I spot checked some other Fetch processors which only have a single failure relationship, and I imagine their libraries don't enable finer grained handling as easily. How about one last idea before I join you with opposing this? What if we do not include the log message in the attribute but only include the relationship name in the attribute? And change the attribute name to 'fetch.failure.reason' which aligns with the FetchParquet processor. This should enable this PR to be much simpler and easier to maintain. fetch.failure.reason=comms.failure or fetch.failure.reason=not.found or fetch.failure.reason=permission.denied -- 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: issues-unsubscr...@nifi.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
