markap14 commented on PR #11677: URL: https://github.com/apache/nifi/pull/11677#issuecomment-5730791650
[grok-4.6] I have moved this PR to draft. My last commit (d3035f27dce) was a mistake and should not be merged. In response to @bobpaulin's request for revert coverage, I changed `StandardConnectorNode` so that any stored property or configuration step the current Connector version does not declare is silently dropped after `migrateProperties`. That is wrong. Deleting a user's configured values is destructive and is not a decision the framework should make on its own, and it was well outside the test coverage that was actually asked for. I am reverting that behavior and working out the correct approach for what should happen when a version change leaves behind values the target version does not declare. I will update this PR when that is settled. -- 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]
