Open
Conversation
added 8 commits
November 5, 2024 14:43
1. 重命名和重构现有指标类,添加新的实现如DefaultLongCounter和DefaultLongHistogram 2. 引入OpenTelemetry依赖,添加NOP实现类用于默认行为 3. 更新Counter、Gauge和Histogram接口,使其与OpenTelemetry兼容 4. 创建EventBridgeMetricsManager用于统一管理指标
1. 将指标相关类从metrics模块移至infrastructure模块 2. 更新指标类以支持OpenTelemetry,包括Counter、Gauge、Histogram等 3. 添加MetricsCollectorFactory和MetricConfig用于配置和初始化指标收集器
1. 新增EventBridgeMetricsConstant类定义指标标签常量 2. 更新DoubleGauge、DoubleHistogram等类,添加metricName字段 3. 在EventBridgeMetricsManager中定义具体的指标实例
1. 更新依赖从common到infrastructure模块 2. 添加Prometheus指标导出功能 3. 在事件处理流程中添加指标收集点(PutEvents、Filter、Transform、Trigger) 4. 配置自定义指标桶和标签 DevPilot: I've analyzed the changes and created a commit message that summarizes the key modifications. The message highlights the addition of metrics collection throughout the EventBridge components, the dependency updates, and the configuration of Prometheus for metrics export. The message is in Chinese as requested and follows best practices for commit messages.
Member
|
some others submit files without license header, so that your PR-checker not pass, can you help it ? |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
add metrics