GitHub user kevdoran opened a pull request:

    https://github.com/apache/nifi-registry/pull/139

    NIFIREG-198 Fix VersionedRemoteProcessGroup targetUri bug

    VersionedRemoteProcessGroup has two fields: targetUri and targetUris. They 
do not have simple getters, but rather each will fallback to using the other. 
Given how they are described, for certain combinations of targetUri and 
targetUris, they do not work correctly (mainly, when targetUri is not set and 
targetUris contains only a single value).
    
    Furthermore, in NiFi only TargetUris is used/needed. It was a replacement 
for targetUri when the requirement for multiple URIs was added. Therefore, in 
addition to fixing the bug descirbed above, targetUri can be deprecated and 
dropped in future versions of NiFi Registry.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/kevdoran/nifi-registry 
NIFIREG-198-vrpg-targetUri

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/nifi-registry/pull/139.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #139
    
----
commit 79af0dbe771e14b3e38e713c0ae9d860a7e83eb2
Author: Kevin Doran <kdoran@...>
Date:   2018-09-06T23:10:50Z

    NIFIREG-198 Add unit test that reproduces bug

commit bc138f1d8649eba3a7daf4ef8acd3e2e2f733622
Author: Kevin Doran <kdoran@...>
Date:   2018-09-07T00:26:06Z

    NIFIREG-198 Fix VersionedRemoteProcesGroup targetUri

commit 73ac36f678a3d82acdc935e18c5b0cf08b4064f0
Author: Kevin Doran <kdoran@...>
Date:   2018-09-07T14:53:09Z

    NIFIREG-198 Deprecate VersionedRemoteProcessGroup targetUri

----


---

Reply via email to