Colin Patrick McCabe
Colin Patrick McCabe
I reworked this a bit so that we can test removing voters now. The checkstyle issue should be fixed.
> @cmccabe thanks for this patch. > > > In a follow-on PR, we should convert QuorumTestHarness to the new mechanism and remove the futures-based mechanism. > Pardon me, have...
Fixed conflicts.
I merged in trunk. I will take another look at test failures when this build completes. I expect they will boil down to adding more "this test requires this feature...
We added this test in https://github.com/apache/kafka/pull/17582 instead. Closing this PR
We ended up doing this in: #16669
To respond to some of the questions above, overall, - 3.8-IV0 in the 3.8 branch needs to be correct and completely in line with trunk - 3.9-IV0 and 3.9-IV1 in...
Since this changes the behavior of the new tiered storage configurations, we will need to get it into 3.9 if it is to get in at all. I will commit...
I added a test. Thanks, @jsancio .