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.

Sign up for GitHub

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

Jump to bottom

Metrics Processor #121

Closed
rghetia opened this issue Sep 12, 2019 · 2 comments
Closed

Metrics Processor #121

rghetia opened this issue Sep 12, 2019 · 2 comments
Labels
area:metrics Part of OpenTelemetry Metrics pkg:SDK Related to an SDK package

Comments

@rghetia
Copy link
Contributor

rghetia commented Sep 12, 2019

Similar to Span Processor, add metric processor interface for processing metrics.
Provide two metric processor

  1. Simple Processor
  2. Batched Processor
@rghetia rghetia added pkg:SDK Related to an SDK package area:metrics Part of OpenTelemetry Metrics labels Sep 12, 2019
@jmacd
Copy link
Contributor

jmacd commented Sep 12, 2019

I thought @bogdandrutu talked me out of naming this concept. I have an in-progress metrics SDK that calls through to a prometheus client, but there's not an intermediate Processor we could factor out.

@jmacd
Copy link
Contributor

jmacd commented Dec 12, 2019

Closing this as the current SDK includes a relatively well-developed concept of a processing pipeline built of the Batcher, AggregationSelector, Aggregator implementations, and Exporters.

@jmacd jmacd closed this as completed Dec 12, 2019
hstan referenced this issue in hstan/opentelemetry-go Oct 15, 2020
@evantorrie
Slight improvement to doc.go for go-restful to meet docs requirement (#…
e24f778 
…121)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area:metrics Part of OpenTelemetry Metrics pkg:SDK Related to an SDK package
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jmacd @rghetia