Github user YolandaMDavis commented on the issue:
https://github.com/apache/nifi/pull/690
@markap14 thanks it looks good to me and no you didn't break anything :).
+1
I'll get this merged in shortly.
---
If your project is set up for it, you can reply to this email
Github user markap14 commented on the issue:
https://github.com/apache/nifi/pull/690
Hey @YolandaMDavis - the changes that were made in the later commit will
not completely prevent the IllegalArgumentException from being thrown in
verifyCreate... it's still possible because after the
Github user YolandaMDavis commented on the issue:
https://github.com/apache/nifi/pull/690
@markap14 retried tests again using a similar setup as before yet included
an input port on the standalone server in addition to the output port.
Performed a variety of tests attempting to
Github user YolandaMDavis commented on the issue:
https://github.com/apache/nifi/pull/690
@markap14 thanks for the update! will retest and confirm.
---
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
Github user markap14 commented on the issue:
https://github.com/apache/nifi/pull/690
@YolandaMDavis I updated the PR with a new commit so that you can see what
changed. It looks like the way to cause it to hit is to drag a connection from
a Processor to the Remote Process Group, and
Github user markap14 commented on the issue:
https://github.com/apache/nifi/pull/690
@YolandaMDavis I will look into the NPE. The test case that you described
will not cause the issue that you are expecting, though. There is no way to
test it without doing some funky stuff with
Github user YolandaMDavis commented on the issue:
https://github.com/apache/nifi/pull/690
@markap14 picked up for review
---
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