Lan Ding comments

Results 15 comments of


                                            Lan Ding

KAFKA-14511: extend AlterIncrementalConfigs API to support group config

@AndrewJSchofield, I've updated the PR. Please take a look again. Thanks.

KAFKA-14511: extend AlterIncrementalConfigs API to support group config

Thanks @AndrewJSchofield for the feedback, I have addressed comments.

KAFKA-14511: extend AlterIncrementalConfigs API to support group config

@dajac, PTAL, thanks in advance.

KAFKA-14511: extend AlterIncrementalConfigs API to support group config

@AndrewJSchofield I've resolved the conflict, PTAL, thanks in advance.

KAFKA-14511: extend AlterIncrementalConfigs API to support group config

@AndrewJSchofield I've updated the PR. Please take a look again. Thanks.

KAFKA-14511: extend AlterIncrementalConfigs API to support group config

@dajac I've updated the PR. Please take a look again. Thanks.

KAFKA-14511: extend AlterIncrementalConfigs API to support group config

@dajac Sorry for the delay, I've updated the PR, PTAL when you get a chance.

KAFKA-14511: extend AlterIncrementalConfigs API to support group config

Hi @dajac. Thanks a lot for the review. I have made the required changes in the last commit, PTAL when you get a chance.

KAFKA-19340: Move DelayedRemoteFetch to the storage module

@frankvicky @apoorvmittal10 PTAL when you get a chance, thanks in advance.

KAFKA-19340: Move DelayedRemoteFetch to the storage module

I compared the performance of this patch with the current version in catch-up read scenarios with tiered storage enabled. Use command: > sh kafka-consumer-perf-test.sh --topic topic-test --group group-test --broker-list 127.0.0.1:9092...