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

metrics 3.0.0 is available #19

Closed
miner opened this issue Jul 23, 2013 · 7 comments
Closed

metrics 3.0.0 is available #19

miner opened this issue Jul 23, 2013 · 7 comments
Assignees
Labels

Comments

@miner
Copy link
Contributor

miner commented Jul 23, 2013

http://metrics.codahale.com/about/release-notes/

v3.0.0: June 10 2013

@miner
Copy link
Contributor Author

miner commented Jul 23, 2013

It looks like there are major changes in Metrics v3.0.0. That makes the current Metrics documentation confusing for metrics-clojure users. If you want to link to the Metrics 2.2.0 doc (corresponding to metrics-clojure), there's a copy archived here:

http://web.archive.org/web/20120925003800/http://metrics.codahale.com/manual/core/

@michaelklishin
Copy link
Collaborator

Upgrading is a good idea but doesn't seem to be trivial. Do you use Metrics docs often? Anything major missing from metrics-clojure's own docs?

@miner
Copy link
Contributor Author

miner commented Jul 23, 2013

I'm new to Metrics and metrics-clojure so mostly I was just trying understand the code. The immediate concern I had was the global nature of Metrics/defaultRegistry. I was more confused when I looked at the latest Metrics doc until I realized that metrics-clojure was based on v2.2. A link to the old Metrics doc might help someone else who's trying to understand the Clojure code.

As far as the upgrade to Metrics v3.0 goes, it sounds like a bit of work. I assume there are improvements but I don't have any direct knowledge of the benefits versus the costs.

@michaelklishin
Copy link
Collaborator

I'm looking into this. There indeed are assorted changes and some mean breaking API changes for metrics-clojure. That's why the version that uses Metrics 3.0 will be 2.0. I will report more specific findings later.

@michaelklishin
Copy link
Collaborator

OK, so the biggest change of all is: there is no more Metrics/defaultRegistry. Which leaves us
with two options:

  • Require an explicit argument, which is a good thing
  • Maintain a var

Actually, there is a third option that I think will work well with the kind of API metrics-clojure core has: do both by using multiple arities for every function. I'm investigating this 3rd option, so far
it looks promising.

@michaelklishin
Copy link
Collaborator

@miner please take a look at #22.

@michaelklishin
Copy link
Collaborator

Fixed in master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants