Luke Chen
Luke Chen
Failed tests are unrelated.
@gaurav-narula , the new added KRaft tests are failing in our test env. The logs are [here](https://drive.google.com/file/d/1CbxgH8eswEXX0YDEJpZn7B9Vd4x-gfo4/view?usp=sharing). Could you help check and fix them? Thanks.
> > @gaurav-narula , the new added KRaft tests are failing in our test env. The logs are [here](https://drive.google.com/file/d/1CbxgH8eswEXX0YDEJpZn7B9Vd4x-gfo4/view?usp=sharing). Could you help check and fix them? Thanks. > > I...
Confirmed all tests passed after https://github.com/apache/kafka/pull/15335 merged.
I understand the problem you're trying to solve, but using the server default request timeout doesn't make sense to me. It will break the contract of fetch protocol that `fetch.max.wait.ms`...
Correction: For this: > some users might feel surprised when their fetch doesn't respond in fetch.max.wait.ms time. This is wrong. Even if the remote reading is not completed, yet, the...
@kamalcph , could we move the dynamic config change into another PR? I have some comments to it, but that is separate from the original changes.
Failed tests are unrelated.
@hudeqi , are you still interested in completing this PR? There seems to have some unaddressed comments..=
Will check this this or next week. Thanks.