NissimShiman commented on PR #9967: URL: https://github.com/apache/nifi/pull/9967#issuecomment-3025225119
I have tested the "Go To" option will not appear for these changes (and looked over the code as well). Thank you @mosermw for resolving that. This now performs as described in the jira (with the known edge case in the comment still present). This lets the user know in many cases that a bundle version change is about to be persisted to registry, but maybe the edge case could be worked out as well. More work would be needed so a process group's versioned snapshot should not be modified from what is in the FlowRegistry to make this edge case work as well, but that is a lot of work. @bbende Is this something that would make sense to be done as part of this PR? or this something that should be left alone? (and, if it should be left alone, is the PR good as is?) -- 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]
