Github user markap14 commented on the issue:
https://github.com/apache/nifi/pull/1208
@ijokarumawak thanks for the update. The new commit perfectly addresses the
concern that I had with the Builder. I do think we should update the
RemoteProcessGroup & DTO as well. The DTO could still have a getTargetUri()
that would return the first value entered, just as the builder does now, and
then also have a getTargetUris() that returns all of them. Along those lines,
it may make sense to not event deprecated the getUri() method and simply
document that it returns the first one entered by the user and the getUris()
method returns all of them. The DTO may also require having a setTargetUri()
and setTargetUris(). I'm just very hesitant to return a comma-separated list of
URI's when we have a method named getUri(), etc. This would also have the added
benefit of keeping the UI clean, showing a single URI if we want to. @mcgilman
do you have any thoughts on how the RPG DTO should change (or not)?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
