pvillard31 commented on PR #11568:
URL: https://github.com/apache/nifi/pull/11568#issuecomment-5347974240

   Thanks for proposing this alternative @exceptionfactory. I'm OK with this 
approach to at least stabilize the builds and avoid those build failures on the 
pull requests. However this does not solve the underlying problem:
   
   - `StandardRebalancingPartition.stop()` returns before the worker 
necessarily exits at `StandardRebalancingPartition.java:140`
   - A worker that already polled records may distribute them after `stop()` 
returns
   - Because `rebalanceTask` is cleared immediately, a quick restart may launch 
another worker while the previous generation is still running.
   
   However this can definitely be a follow-up, so I'll proceed with closing my 
PR, approve this one and it can be merged assuming checks are green.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to