slambrose commented on PR #8536:
URL: https://github.com/apache/nifi/pull/8536#issuecomment-2017834831

   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 branches currently do not work to process stateless flows that have 
sub-versioned flows. It's **broken** in current state. My two small changes fix 
both the main and 1.x support branches in terms of processing the flows (tested 
and confirmed); however, there are 9 integration tests that appear to be 
_trying_ to test this exact use case. Clearly, these these 9 tests were bad to 
begin with because they don't catch the bug at all. I'm not sure what actually 
changed in the code base such that these 9 tests fail in the main branch but 
not the 1.x support branch. I see many places where this concept of a 
"registryId" is not fully programmed out, so making a require non-null on it 
does not work. 
   I'm proposing an acceptance of these merge requests with the broken tests 
commented out to fix processing in production, with a continued ticket to 
investigate coding this out completely and fixing the integration tests at a 
later time. Otherwise, the product remains broken indefinitely.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to