[
https://issues.apache.org/jira/browse/KAFKA-7837?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16752923#comment-16752923
]
ASF GitHub Bot commented on KAFKA-7837:
---
junrao commented on pull request #6202: KAFKA-7837: Ensure
[
https://issues.apache.org/jira/browse/KAFKA-7837?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16752790#comment-16752790
]
ASF GitHub Bot commented on KAFKA-7837:
---
dhruvilshah3 commented on pull request #6202: KAFKA-7837:
[
https://issues.apache.org/jira/browse/KAFKA-7837?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16751766#comment-16751766
]
Jun Rao commented on KAFKA-7837:
[~dhruvilshah], the following is the description of the iterator of
[
https://issues.apache.org/jira/browse/KAFKA-7837?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16750805#comment-16750805
]
Dhruvil Shah commented on KAFKA-7837:
-
[~junrao] [~lindong] having looked a bit into this, it seems
[
https://issues.apache.org/jira/browse/KAFKA-7837?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16746588#comment-16746588
]
Dhruvil Shah commented on KAFKA-7837:
-
I could take a stab at fixing this.
> maybeShrinkIsr may not
[
https://issues.apache.org/jira/browse/KAFKA-7837?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16746520#comment-16746520
]
Jun Rao commented on KAFKA-7837:
@lindong, yes, the approach that you described sounds good.
>
[
https://issues.apache.org/jira/browse/KAFKA-7837?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16745832#comment-16745832
]
Dong Lin commented on KAFKA-7837:
-
[~junrao] Currently `partition.maybeShrinkIsr(...)` reads
[
https://issues.apache.org/jira/browse/KAFKA-7837?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16745680#comment-16745680
]
Jun Rao commented on KAFKA-7837:
One way to improve this is that in maybeShrinkIsr(), for each iterated