markobean commented on PR #10231:
URL: https://github.com/apache/nifi/pull/10231#issuecomment-3214107683

   > Thanks for highlighting this issue @markobean. I believe it has come up in 
the past, but there was some hesitation around making changes due to impact on 
extension behavior.
   > 
   > However, given the the `details` field is an open-ended `String`, it seems 
best to consider this change and highlight it in the migration guide for the 
next release version.
   > 
   > Although it is not possible to address the impact on external components, 
have you reviewed usage within the project to determine if there are any 
changes needed?
   
   Currently, there are no calls to the the updated method in the NiFi 
codebase. (This is probably why it has gone unnoticed.) There should be no ill 
effects, and no other changes needed. As you said, there is no way to evaluate 
external components in custom NARs, so I like the idea of highlighting this in 
the release notes and/or migration guidance. 


-- 
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]

Reply via email to