The GitHub Actions job "Check Documentation" on fluss.git/rocksmetrics has 
succeeded.
Run started by GitHub user platinumhamburg (triggered by platinumhamburg).

Head commit for run:
afc353255bff67f3ae79ac8b512014381592908a / ocean.wy <[email protected]>
重构: 优化 RocksDB 指标管理架构

主要变更:
1. 重命名 RocksDBMetrics -> RocksDBStatistics,明确其职责为统计收集而非指标注册
2. 将 RocksDBStatistics 管理权从 TableMetricGroup 移交给 BucketMetricGroup,实现更清晰的封装
3. 简化注册流程:Replica 直接向 BucketMetricGroup 注册,无需跨层级调用
4. 完善生命周期管理:
   - 注册:KvTablet 创建后在 Replica.initKvTablet() 中调用
   - 注销:KvTablet 销毁前在 Replica.dropKv() 中显式调用
   - 新增 BucketMetricGroup.unregisterRocksDBStatistics() 确保成对调用
5. 动态聚合优化:
   - 移除 TableMetricGroup.rocksDBStatisticsMap 冗余数据结构
   - 新增 allRocksDBStatistics() 方法动态从 BucketMetricGroup 获取
6. 提高并发安全性:将 TableMetricGroup.buckets 改为 ConcurrentHashMap

收益:
- 代码更简洁:减少 50+ 行冗余代码
- 架构更清晰:RocksDBStatistics 归属明确
- 资源更安全:注册/注销成对,防止泄漏
- 维护性更好:单一职责,修改影响面小

测试验证:
- ✅ RocksDBResourceContainerTest: 7/7 通过
- ✅ KvTabletTest: 13/13 通过
- ✅ 编译通过,无 linter 错误

Report URL: https://github.com/apache/fluss/actions/runs/20713023440

With regards,
GitHub Actions via GitBox

Reply via email to