markap14 commented on PR #10742:
URL: https://github.com/apache/nifi/pull/10742#issuecomment-3724804498

   Thanks @mcgilman I think for those cases, calling `getFlow` with or without 
the Connector ID argument is probably OK because they are requesting specific 
ID's that they've obtained from other groups, so they won't ever request a 
Group unless it was handed the group from a Connector. But I think it probably 
makes sense to explicitly use `null` from FlowAnalyzingRegistryClient because 
that should never be used in relation to a Connector. But it would potentially 
make sense to use FlowDifferenceFilters to check if a Connector's flow has 
changed, so let's leave it without the explicit `null`.


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