Skip to content

KAFKA-10776: update the doc to add version attribute in RequstsPerSec metrics#9661

Merged
ijuma merged 1 commit into
apache:trunkfrom
showuon:KAFKA-10776
Dec 15, 2020
Merged

KAFKA-10776: update the doc to add version attribute in RequstsPerSec metrics#9661
ijuma merged 1 commit into
apache:trunkfrom
showuon:KAFKA-10776

Conversation

@showuon

@showuon showuon commented Nov 30, 2020

Copy link
Copy Markdown
Member

This is actually a documentation miss, the version attribute is required to match the Kafka version. Thanks.

See https://cwiki.apache.org/confluence/display/KAFKA/KIP-272%3A+Add+API+version+tag+to+broker%27s+RequestsPerSec+metric

Committer Checklist (excluded from commit message)

  • Verify design and implementation
  • Verify test coverage and CI build status
  • Verify documentation (including upgrade notes)

@showuon

showuon commented Nov 30, 2020

Copy link
Copy Markdown
Member Author

@ijuma , please help review this simple PR to update the doc. Thanks.

1 similar comment
@showuon

showuon commented Dec 15, 2020

Copy link
Copy Markdown
Member Author

@ijuma , please help review this simple PR to update the doc. Thanks.

@ijuma ijuma left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, thanks.

@ijuma ijuma merged commit 3717ab0 into apache:trunk Dec 15, 2020
ijuma added a commit to ijuma/kafka that referenced this pull request Dec 15, 2020
…keys

* apache-github/trunk:
  KAFKA-10776: Add version attribute in RequestsPerSec metrics documentation (apache#9661)
  KAFKA-10854: fix flaky testConnectionRatePerIp test (apache#9752)
  KAFKA-10525: Emit JSONs with new auto-generated schema (KIP-673) (apache#9526)
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.

2 participants