ableegoldman opened a new pull request #8963:
URL: https://github.com/apache/kafka/pull/8963


   The current failures we're seeing with this test are due to faulty 
assumptions that it makes and not any real bug in eos-beta (at least, from what 
I've seen so far).
   
   The test relies on tightly controlling the commits, which it does by setting 
the commit interval to MAX_VALUE and manually requesting commits on the 
context. In two phases, the test assumes that any pending data will be 
committed after a rebalance. But we actually take care to avoid unnecessary 
commits -- with eos-alpha, we only commit tasks that are revoked while in 
eos-beta we must commit all tasks if any are revoked, but only if the revoked 
tasks themselves need a commit.
   
   The failure we see occurs when we try to verify the committed data after a 
second client is started and the group rebalances. The already-running client 
has to give up two tasks to the newly started client, but those tasks may not 
need to be committed in which case none of the tasks would be. So we still have 
an open transaction on the partitions where we try to read committed data.
   
   We can use a punctuator to force a commit on the running client 


----------------------------------------------------------------
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.

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


Reply via email to