exceptionfactory commented on PR #9778: URL: https://github.com/apache/nifi/pull/9778#issuecomment-2733409558
Thanks for the contribution @samar121. I agree with @pvillard31 that this would constitute a breaking change as it stands. This is highlighted by the fact that multiple unit tests are failing. I believe this change could be introduced through a new property to control the behavior, although that needs some additional consideration. Given that there are multiple unit tests failing, and this is a breaking change as it stands, I am closing this pull request for now. However, please feel free to submit a new pull request after updating the unit tests and considering a way forward in terms of introducing the changes in a non-breaking way. We can also discuss options on the associated Jira issue if you would like to consider options before moving directly to code changes. -- 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]
