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

Implement SynchronousMeasurementConsumer #2388

Merged
merged 5 commits into from
Jan 19, 2022
Merged

Implement SynchronousMeasurementConsumer #2388

merged 5 commits into from
Jan 19, 2022

Conversation

aabmass
Copy link
Member

@aabmass aabmass commented Jan 18, 2022

Description

  • Implemented the SynchronousMeasurementConsumer from the design.
  • Added a stub implementation and TODO for implementing the MetricReaderStorage (Implement MetricReaderStorage #2378)
  • Added a small (internal) SdkConfiguration dataclass for passing through SDK configuration from MeterProvider. This avoids having to access private methods/variables on the MeterProvider class.

Fixes #2328

Type of change

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Added test case mocking out dependencies.

Does This PR Require a Contrib Repo Change?

  • No.

Checklist:

  • Followed the style guidelines of this project
  • Unit tests have been added

@aabmass aabmass added sdk Affects the SDK package. metrics labels Jan 18, 2022
@aabmass aabmass force-pushed the 2328-measurementconsumer branch 2 times, most recently from 933c339 to 1b41260 Compare January 18, 2022 18:46
@aabmass aabmass force-pushed the 2328-measurementconsumer branch from 1b41260 to 4ba0f61 Compare January 18, 2022 18:54
@aabmass aabmass force-pushed the 2328-measurementconsumer branch from 4ba0f61 to 7609465 Compare January 18, 2022 18:54
@aabmass aabmass marked this pull request as ready for review January 18, 2022 18:54
@aabmass aabmass requested a review from a team January 18, 2022 18:55
@aabmass aabmass added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Jan 18, 2022
@aabmass aabmass requested review from ocelotl and lzchen January 18, 2022 21:20
@ocelotl ocelotl mentioned this pull request Jan 18, 2022
Closed
@ocelotl ocelotl added the Approve Public API check This label shows that the public symbols added or changed in a PR are strictly necessary label Jan 19, 2022
@lzchen lzchen merged commit fb60538 into open-telemetry:main Jan 19, 2022
@aabmass aabmass deleted the 2328-measurementconsumer branch January 19, 2022 17:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lzchen lzchen lzchen approved these changes

@ocelotl ocelotl ocelotl approved these changes

Assignees
No one assigned
Labels
Approve Public API check This label shows that the public symbols added or changed in a PR are strictly necessary metrics sdk Affects the SDK package. Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Implement DefaultMeasurementConsumer
3 participants
@aabmass @ocelotl @lzchen