Skip to content

[LI-CHERRY-PICK] [31d191fc] KAFKA-7904; Add AtMinIsr partition metric…#30

Merged
xiowu0 merged 1 commit into
linkedin:2.0-lifrom
xiowu0:atMinIsr
Jul 16, 2019
Merged

[LI-CHERRY-PICK] [31d191fc] KAFKA-7904; Add AtMinIsr partition metric…#30
xiowu0 merged 1 commit into
linkedin:2.0-lifrom
xiowu0:atMinIsr

Conversation

@xiowu0

@xiowu0 xiowu0 commented Jul 15, 2019

Copy link
Copy Markdown

… and TopicCommand option (KIP-427)

TICKET = KAFKA-7904
LI_DESCRIPTION =
This patch includes broker metrics for AtMinIsr and modified unit tests. However, it doesn't include topiccommand related change from the original cherry-picked commit because of extra dependency.

EXIT_CRITERIA = HASH [31d191f]
ORIGINAL_DESCRIPTION =

  • Add AtMinIsrPartitionCount metric to ReplicaManager
  • Add AtMinIsr metric to Partition
  • Add --at-min-isr-partitions describe TopicCommand option

https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=103089398

Author: Kevin Lu lu.kevin@berkeley.edu
Author: lu.kevin@berkeley.edu kelu@paypal.com

Reviewers: Gwen Shapira

Closes apache#6421 from KevinLiLu/KAFKA-7904

(cherry picked from commit 31d191f)

…and TopicCommand option (KIP-427)

TICKET = KAFKA-7904
LI_DESCRIPTION =
This patch includes broker metrics for AtMinIsr and modified unit tests. However, it doesn't include topiccommand related change from the original cherry-picked commit because of extra dependency.

EXIT_CRITERIA = HASH [31d191f]
ORIGINAL_DESCRIPTION =

- Add `AtMinIsrPartitionCount` metric to `ReplicaManager`
- Add `AtMinIsr` metric to `Partition`
- Add `--at-min-isr-partitions` describe `TopicCommand` option

https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=103089398

Author: Kevin Lu <lu.kevin@berkeley.edu>
Author: lu.kevin@berkeley.edu <kelu@paypal.com>

Reviewers: Gwen Shapira

Closes apache#6421 from KevinLiLu/KAFKA-7904

(cherry picked from commit 31d191f)
@xiowu0 xiowu0 requested review from hzxa21, jjkoshy and jonlee2 July 15, 2019 23:13

@hzxa21 hzxa21 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for picking this up. LGTM.

@xiowu0 xiowu0 merged commit c3b03ea into linkedin:2.0-li Jul 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants