Steph
Steph
Does the current main branch even pass the integration tests? When I build locally with the tests, they fail on the main branch.
Thanks, I see it now. Looks like there was an integration test that was added in 2.0 that isn't good. The test that's failing looks like it should have picked...
So, here is my conclusion on this. Unfortunately, I'm not sure I have the cycles to do the full re-write that this code probably requires. The main and 1.x support...
Okay, I'm trying one more solution. What I've found is that the main branch has new code where a "registryId" is implemented, but it is always null right now because...
Okay, so after some back and forth here is my final conclusion to this bug: I've submitted two PRs https://github.com/apache/nifi/pull/8572/checks registryId is always null. Now in the main branch, stateless...
Thanks for taking another look :) So, the storageLocation change is still needed to fix the bug and process the flow in stateless. Otherwise, when it loops down to pull...
Ready for re-review.
> Thanks for working on the unit tests @slambrose. The test for the synchronizer class looks like it exercises the method changes, but the `TestRegistryUtil` appears to be running against...
@exceptionfactory Okay, this should be good now.
> Thanks for the latest updates @slambrose, this is looking closer to completion. > > Changing the `InMemoryFlowRegistry` to return `true` makes sense, as it is only used in Stateless...