Skip to content

refactor(layers/metrics): rewrite metrics layer using observe layer#5098

Merged
Xuanwo merged 3 commits intoapache:mainfrom
koushiro-contrib:refactor-metrics-layer
Sep 5, 2024
Merged

refactor(layers/metrics): rewrite metrics layer using observe layer#5098
Xuanwo merged 3 commits intoapache:mainfrom
koushiro-contrib:refactor-metrics-layer

Conversation

@koushiro
Copy link
Copy Markdown
Member

@koushiro koushiro commented Sep 5, 2024

Which issue does this PR close?

Related PR: #5064

Rationale for this change

What changes are included in this PR?

  • rewrite metrics layer using observe layer
  • add more labels

Are there any user-facing changes?

API breaking changes

@koushiro koushiro requested a review from Xuanwo as a code owner September 5, 2024 09:11
@github-actions github-actions Bot added the releases-note/refactor The PR does a refactor on code or has a title that begins with "refactor" label Sep 5, 2024
@koushiro
Copy link
Copy Markdown
Member Author

koushiro commented Sep 5, 2024

Because of the design of the metrics, we cannot do following things at the API level of the metrics library

  1. specify the buckets of the histogram metrics
  2. specify labels while modifying the metrics

Copy link
Copy Markdown
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot!

@Xuanwo Xuanwo merged commit dd02e29 into apache:main Sep 5, 2024
@koushiro koushiro deleted the refactor-metrics-layer branch September 5, 2024 11:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

releases-note/refactor The PR does a refactor on code or has a title that begins with "refactor"

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants