pvillard31 commented on PR #10393: URL: https://github.com/apache/nifi/pull/10393#issuecomment-3411538994
Thanks for the review @rfellows - you're right that if the verify method is not implemented and the configuration is correct (based on the validators on the properties), it'd say that this is successful even though it may not true. I agree that this could be confusing but I do think this is aligned with what we have right now: a processor without a verify method would show successful at verification if configuration is valid but could very well throw errors when the processor is started. I think this is OK. Also the only implementation that is not git based in Apache NiFi is the NiFi Registry implementation for which we could also implement the verify method in a follow up improvement. -- 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]
