Github user asfgit closed the pull request at:
https://github.com/apache/nifi/pull/347
---
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 enable
Github user olegz commented on the pull request:
https://github.com/apache/nifi/pull/347#issuecomment-219489958
merging
---
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 an
Github user pvillard31 commented on the pull request:
https://github.com/apache/nifi/pull/347#issuecomment-218867090
@olegz Thanks Oleg! Not exactly what I had in mind but it does the trick :)
(I was trying to set up an environment with two simple processors sending flow
files to RPG
Github user olegz commented on the pull request:
https://github.com/apache/nifi/pull/347#issuecomment-217000289
@pvillard31 below is the test you can use to validate this issue. Basically
if you execute it on current master it will fail the assertions and then with
your changes it wil
Github user olegz commented on the pull request:
https://github.com/apache/nifi/pull/347#issuecomment-216635676
Sure, I'll give it a shot
---
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
Github user pvillard31 commented on the pull request:
https://github.com/apache/nifi/pull/347#issuecomment-216631894
@olegz I must admit I tried to use your code as mentioned in your link and
got it working but I didn't find a way to validate the change since it seems to
simulate S2S
Github user olegz commented on the pull request:
https://github.com/apache/nifi/pull/347#issuecomment-216612204
@pvillard31 Are you planning to add tests for this? If not I can take a
crack at it. LMK.
---
If your project is set up for it, you can reply to this email and have your
re
Github user olegz commented on the pull request:
https://github.com/apache/nifi/pull/347#issuecomment-213020240
@pvillard31 Actually I totally forgot about the effort I started a while
back. There is actually a branch for it
https://github.com/olegz/nifi/tree/int-test/nifi-integration
Github user olegz commented on the pull request:
https://github.com/apache/nifi/pull/347#issuecomment-212995061
@pvillard31 let me dig some more. Yes you have to have the "other side"
running but I had some other code bits to simulate that. I'll post here once I
find
---
If your pro
Github user pvillard31 commented on the pull request:
https://github.com/apache/nifi/pull/347#issuecomment-212876061
@olegz Thanks for the code. However if I try to run it, it will try to
connect to localhost:8080/nifi and will fail:
594 [NiFi Site-to-Site Connection Pool Mai
Github user olegz commented on the pull request:
https://github.com/apache/nifi/pull/347#issuecomment-211417082
@pvillard31 I was gonna play with it a bit more and possibly create the
test, but getting tied up with other things. Yet I was able to dig up one of my
experiments that I us
Github user olegz commented on the pull request:
https://github.com/apache/nifi/pull/347#issuecomment-211324090
@pvillard31 right, not the simplest thing to do, but let me dig as I
believe I have something. Will let you know
---
If your project is set up for it, you can reply to this
Github user pvillard31 commented on the pull request:
https://github.com/apache/nifi/pull/347#issuecomment-211283811
@olegz I agree a unit test would be nice, but I must admit that I am not
sure to see how to do it. It involves RPG, remote ports, remote processors and
I don't see how
Github user olegz commented on the pull request:
https://github.com/apache/nifi/pull/347#issuecomment-211126901
@pvillard31 Changes look good, but was wondering if you could add some tests
---
If your project is set up for it, you can reply to this email and have your
reply appear on
GitHub user pvillard31 opened a pull request:
https://github.com/apache/nifi/pull/347
NIFI-1755 Fixed remote process group status counts by only considering
connected remote ports
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/
15 matches
Mail list logo
