Reduced API - Githubissues

eclipse / microprofile-metrics

microprofile-metrics

Apache License 2.0

100 stars 66 forks source link

Reduced API #664

Closed Channyboy closed 2 years ago

Channyboy commented 2 years ago

Visualization of possible reduced API for MP Metrics 5.0 See discussions on MicroProfile Metrics meeting minutes: https://docs.google.com/document/d/1HJOg-NOVpcE5X7qhELnyBa_A7OaVTwbOPCsAOp6F1Zc/edit

Remove: SimpleTimer and related annotation Meter and related annotation ConcurrentGauge (no related anno) Metered interface ... This affects Timer metrics w.r.t rate related calculations

Channyboy commented 2 years ago

Only API builds (for dev). TCK will be followed up later.

jgallimore commented 2 years ago

I've been using this PR for quite a while now, and think we should get it merged in. A couple of minor comments:

src/main/java/org/eclipse/microprofile/metrics/annotation/package-info.java still references Metered, ConcurrentGauge and SimplyTimed
src/main/java/org/eclipse/microprofile/metrics/test/MetricAppBean.java has some constants related to the metric types that have been removed