Skip to content

Implement MetricReader default aggregation controls#2638

Merged
lzchen merged 3 commits intoopen-telemetry:mainfrom
ocelotl:issue_2635
Apr 27, 2022
Merged

Implement MetricReader default aggregation controls#2638
lzchen merged 3 commits intoopen-telemetry:mainfrom
ocelotl:issue_2635

Conversation

@ocelotl
Copy link
Copy Markdown
Contributor

@ocelotl ocelotl commented Apr 22, 2022

Fixes #2635

@ocelotl ocelotl self-assigned this Apr 22, 2022
@ocelotl ocelotl changed the title WIP Implement MetricReader default aggregation controls Apr 22, 2022
@ocelotl ocelotl marked this pull request as ready for review April 22, 2022 22:24
@ocelotl ocelotl requested a review from a team April 22, 2022 22:24
@ocelotl ocelotl added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Apr 22, 2022
@ocelotl ocelotl force-pushed the issue_2635 branch 2 times, most recently from 24d85f3 to 9e1308f Compare April 22, 2022 23:28
@srikanthccv
Copy link
Copy Markdown
Member

@ocelotl I think it's good add a CHANGELOG for this and other PRs since they are important features.

@ocelotl
Copy link
Copy Markdown
Contributor Author

ocelotl commented Apr 25, 2022

@ocelotl I think it's good add a CHANGELOG for this and other PRs since they are important features.

Added an entry 👍

@ocelotl ocelotl removed the Skip Changelog PRs that do not require a CHANGELOG.md entry label Apr 25, 2022
@lzchen lzchen merged commit a4b4c45 into open-telemetry:main Apr 27, 2022
@ocelotl ocelotl deleted the issue_2635 branch April 27, 2022 20:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement MetricReader default aggregation controls #2627

3 participants