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

Include DataSketches metrics provider for bookie stats #143

Merged
merged 1 commit into from
Jan 10, 2017

Conversation

merlimat
Copy link
Contributor

@merlimat merlimat commented Dec 8, 2016

Motivation

Include a metrics provider for Bookie stats for standalone and for normal cluster deployment.

We have added a new metrics provider that is based on Data Sketches that has minimal overhead (no allocations) and
dumps the stats periodically in JSON format into a file.

@yahoocla
Copy link

yahoocla commented Dec 8, 2016

CLA is valid!

@merlimat merlimat added the type/enhancement The enhancements for the existing features or docs. e.g. reduce memory usage of the delayed messages label Dec 8, 2016
@merlimat merlimat self-assigned this Dec 8, 2016
@merlimat merlimat added this to the 1.16 milestone Dec 8, 2016
@merlimat merlimat force-pushed the master branch 2 times, most recently from 5b4aae2 to 42abf8c Compare December 21, 2016 18:21
@merlimat
Copy link
Contributor Author

@saandrews @rdhabalia Please take a look

@merlimat
Copy link
Contributor Author

merlimat commented Jan 9, 2017

@saandrews @rdhabalia Ping again

Copy link
Contributor

@rdhabalia rdhabalia left a comment

Choose a reason for hiding this comment

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

👍

@merlimat merlimat merged commit 3aaff93 into apache:master Jan 10, 2017
sijie pushed a commit to sijie/pulsar that referenced this pull request Mar 4, 2018
hangc0276 pushed a commit to hangc0276/pulsar that referenced this pull request May 26, 2021
Support listener either in format: type://:port, e.g.: "SSL://:9093", or in format: type://hostname:port, e.g.: "SSL://hostname:9093",
For the 1st format, need to fill it with advertisedAddress for hostname.
For the 2nd format, need to check the hostname is the same as advertisedAddress.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/enhancement The enhancements for the existing features or docs. e.g. reduce memory usage of the delayed messages
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants