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 terminology: Rename Integrator -> Processor #659

Closed
jmacd opened this issue Jun 17, 2020 · 1 comment · Fixed by #347
Closed

Metrics terminology: Rename Integrator -> Processor #659

jmacd opened this issue Jun 17, 2020 · 1 comment · Fixed by #347
Assignees
@jmacd
Labels
area:sdk Related to the SDK priority:p2 Medium priority level release:required-for-ga Must be resolved before GA release, or nice to have before GA spec:metrics Related to the specification/metrics directory

Comments

@jmacd
Copy link
Contributor

jmacd commented Jun 17, 2020

People arrive in OpenTelemetry looking for a metrics processor API. We have such a thing, but it's named "Integrator" in a nod to its mathematical role in summing the output of an Accumulator on behalf of an Exporter.

However, this obfuscates its purpose. The metrics Integrator is a processor API, and it should be named Processor. The naming will be updated in the WIP metrics SDK specification #347.
@jmacd jmacd added the spec:metrics Related to the specification/metrics directory label Jun 17, 2020
@jmacd
Copy link
Contributor Author

jmacd commented Jun 17, 2020

I will factor this decision into #347 when I start working on it again.

@MrAlias MrAlias mentioned this issue Jun 18, 2020
Closed
@jmacd jmacd mentioned this issue Jun 23, 2020
Merged
@carlosalberto carlosalberto added the area:sdk Related to the SDK label Jun 26, 2020
@jmacd jmacd added the release:required-for-ga Must be resolved before GA release, or nice to have before GA label Jun 29, 2020
@carlosalberto carlosalberto added the priority:p2 Medium priority level label Jul 24, 2020
@jmacd jmacd mentioned this issue Aug 19, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jmacd jmacd

Labels
area:sdk Related to the SDK priority:p2 Medium priority level release:required-for-ga Must be resolved before GA release, or nice to have before GA spec:metrics Related to the specification/metrics directory
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Metric SDK specification OUTLINE jmacd/opentelemetry-specification
2 participants
@carlosalberto @jmacd