Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Backport: Switch partition metricset from client to broker (#3029) #3090

Merged
merged 1 commit into from
Dec 1, 2016

Conversation

ruflin
Copy link
Member

@ruflin ruflin commented Dec 1, 2016

Update kafka broker query

  • Switch paritition metricset from client to broker
  • on connect try to find the broker id (address must match advertised host).
  • check broker is leader before querying offsets
  • query offsets for all replicas
  • remove 'isr' from event, and replace with boolean flag insync_replica
  • replace replicas from event with per event replica-id
  • update sarama to get offset per replica id

(cherry picked from commit 4c4889f)

Update kafka broker query

- Switch paritition metricset from client to broker
- on connect try to find the broker id (address must match advertised host).
- check broker is leader before querying offsets
- query offsets for all replicas
- remove 'isr' from event, and replace with boolean flag `insync_replica`
- replace `replicas` from event with per event `replica`-id
- update sarama to get offset per replica id

(cherry picked from commit 4c4889f)
@tsg tsg merged commit 93b680e into elastic:5.1 Dec 1, 2016
@ruflin ruflin deleted the backport_3029 branch December 1, 2016 08:32
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
…tic#3090)

Update kafka broker query

- Switch paritition metricset from client to broker
- on connect try to find the broker id (address must match advertised host).
- check broker is leader before querying offsets
- query offsets for all replicas
- remove 'isr' from event, and replace with boolean flag `insync_replica`
- replace `replicas` from event with per event `replica`-id
- update sarama to get offset per replica id

(cherry picked from commit 1078319)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants