Skip to content

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

Merged
ijuma merged 1 commit intoapache: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 intoapache:trunkfrom
showuon:KAFKA-10776

Conversation

@showuon
Copy link
Copy Markdown
Member

@showuon showuon commented Nov 30, 2020

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
Copy link
Copy Markdown
Member Author

showuon commented Nov 30, 2020

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

1 similar comment
@showuon
Copy link
Copy Markdown
Member Author

showuon commented Dec 15, 2020

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

Copy link
Copy Markdown
Member

@ijuma ijuma left a comment

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