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

Rust port of 'metrics' for libextra #6810

Closed
graydon opened this issue May 29, 2013 · 9 comments
Closed

Rust port of 'metrics' for libextra #6810

graydon opened this issue May 29, 2013 · 9 comments
Labels
C-enhancement Category: An issue proposing an enhancement or a PR with one.

Comments

@graydon
Copy link
Contributor

graydon commented May 29, 2013

A clone of the 'metrics' package would be useful for instrumentation. We have enough stuff we want to monitor that it'd be good to standardize the interface.

https://github.com/codahale/metrics
https://github.com/rcrowley/go-metrics

Currently I'm growing some ad-hoc pieces of this in extra::stats, the std::test benchmarking code, and the log-bin statistics counters in the new GC. Consolidating that and copying best practice structure (eg. properly separating collecting, aggregating and reporting) from the designs of other metrics libraries would be good.

Depends on #7458

@graydon graydon mentioned this issue May 29, 2013
@graydon
Copy link
Contributor Author

graydon commented May 31, 2013

Note: there are also new stat counters in @brson's scheduler work. And rustc has a number of internal counters for phase timing and size measurements.

@graydon
Copy link
Contributor Author

graydon commented May 31, 2013

@graydon
Copy link
Contributor Author

graydon commented Jun 28, 2013

Also note that @tkuehn has been hacking up a metrics library for servo over in https://github.com/mozilla/servo/blob/master/src/components/util/time.rs so we should coordinate work with requirements there.

@brson
Copy link
Contributor

brson commented Sep 18, 2013

Not 0.8

@flaper87
Copy link
Contributor

Triage bump, still lot to do here and a valid bug.

@flaper87
Copy link
Contributor

Also, since libextra is being split into standalone packages, this metrics work should probably live in libmetrics or elsewhere.

@graydon graydon removed their assignment Jun 16, 2014
@evnm
Copy link

evnm commented Sep 14, 2014

What's the status of this issue? I notice that stats.rs now resides in libtest, but the domain-specific counter and timing support code in rustc is still disjointed.

Maybe this is ok? Even with the current state of affairs, you could imagine someone building as a crate an application metrics library atop test::stats, integrating ideas from Dropwizard's Metrics, HdrHistogram, etc. Would this be undesirable? It's unclear whether that kind of full-featured metrics library was the intended outcome of this issue or rather a collection of traits and utilities with which to write one.

@steveklabnik
Copy link
Member

I'm pulling a massive triage effort to get us ready for 1.0. As part of this, I'm moving stuff that's wishlist-like to the RFCs repo, as that's where major new things should get discussed/prioritized.

This issue has been moved to the RFCs repo: rust-lang/rfcs#843

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-enhancement Category: An issue proposing an enhancement or a PR with one.
Projects
None yet
Development

No branches or pull requests

5 participants