exceptionfactory commented on PR #6650: URL: https://github.com/apache/nifi/pull/6650#issuecomment-1315434084
Thanks for putting together this improvement @nandorsoma! Following the note in the summary, introducing the new `defaultValue(DescribedValue)` method signature would create potential compatibility problems for custom components that currently use `defaultValue(null)` in a `PropertyDescriptor.Builder`. As this is part of `nifi-api`, compatibility should be maintained, so it would be best to remove that addition and back out the removal of `defaultValue(null)` from the various components. -- 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]
