feat: querier add table in_process_metrics

[xiaochaoren1]

This PR is for:

[xiaochaoren1]

@claude

[xiaochaoren1]

@claude 审核代码，中文回答

[claude]

Claude finished @xiaochaoren1's task —— View job

---

代码审查任务

我已经完成了对这个PR的代码审查。以下是我的任务清单：

• 阅读仓库的CLAUDE.md文件了解开发指南
• 分析所有10个变更文件的内容
• 检查代码质量、安全性和性能问题
• 验证编码规范和最佳实践
• 提供详细的中文代码审查反馈

代码审查结果

📋 功能概述

本PR为querier模块新增了in_process_metrics表的支持，主要用于处理进程内指标数据的查询。

✅ 代码质量评估

优点:

1. 结构清晰: 新增的配置文件结构遵循了现有的约定，包含了中英文双语配置
2. 一致性良好: Go代码变更保持了与现有代码的一致性，使用了相同的常量定义模式
3. 逻辑合理: 在合适的位置添加了新表的支持，避免了重复代码

🔍 详细技术审查

1. 配置文件 (新增的6个配置文件)

• 指标配置文件 (in_process_metrics, .ch, .en):

  • 正确定义了profile_value计数器类型指标
  • 添加了row字段用于统计行数
  • 权限配置合理(111)

• 标签配置文件 (tag/profile/in_process_metrics*):

  • 包含了完整的通用标签定义，与现有profile表保持一致
  • 中英文描述完整，翻译准确
  • 标签类型和权限配置正确

2. Go代码变更

• common/const.go:38-39: 新增常量定义符合命名规范 ✅
• common/const.go:62: 正确将新表添加到DB_TABLE_MAP中 ✅
• clickhouse.go:1324-1327: 为in_process_metrics表设置1s数据源，逻辑正确 ✅
• metrics/metrics.go:353,667: 复用了in_process的指标处理逻辑，避免代码重复 ✅
• tag/description.go:771,939: 正确处理了标签描述逻辑的条件判断 ✅

⚠️ 需要关注的问题

1. 英文配置文件不一致 (轻微问题)

• tag/profile/in_process_metrics.en:52-53: 中文字符出现在英文配置中

profile_event_type         , 剖析类型                       ,
profile_language_type      , 语言类型                       ,

建议: 应该使用英文描述，如:

profile_event_type         , Profile Event Type            ,
profile_language_type      , Profile Language Type         ,

2. 数据源硬编码 (设计考虑)

• clickhouse.go:1325: 硬编码了"1s"数据源
• 建议: 考虑是否应该通过配置或常量来管理数据源，提高可维护性

📊 测试建议

建议验证以下场景：

1. 新表的查询功能是否正常工作
2. 中英文显示是否正确
3. 权限控制是否按预期工作
4. 与现有in_process表的兼容性

🎯 总结

这是一个实现质量较高的PR，代码变更合理且遵循了现有的架构模式。除了英文配置文件中的小问题外，整体实现良好，可以合并。

评分: 8/10 - 功能实现正确，代码质量好，仅有轻微的配置文件问题需要修复。

---

[xiaochaoren1]

@claude 再看看

[claude]

Claude encountered an error —— View job

---

代码审查任务 - 第二轮

正在进行第二轮代码审查...

📋 任务清单

• 阅读仓库开发指南 (CLAUDE.md)
• 重新检查所有12个变更文件
• 对比第一轮审查结果，验证问题是否已修复
• 查找任何新的问题或改进建议
• 提供更新的中文代码审查反馈

---

[xiaochaoren1]

@claude 再看看

[claude]

Claude finished @xiaochaoren1's task —— View job

---

代码审查任务 - 第三轮

📋 任务清单

• 阅读仓库开发指南 (CLAUDE.md)
• 检查所有12个变更文件
• 分析配置文件的正确性
• 检查Go代码变更
• 验证中英文配置一致性
• 检查之前发现的问题是否已修复
• 提供详细的中文代码审查反馈

---

🔍 详细代码审查结果

📋 功能概述

本PR为querier模块新增了in_process_metrics表的支持，这是一个用于处理进程内指标数据查询的新表。实现包括配置文件、Go代码常量定义和查询逻辑的更新。

✅ 代码质量评估

优点：

1. 架构设计合理：新表复用了现有in_process表的大部分标签和指标处理逻辑，减少了代码重复
2. 命名一致性好：遵循了现有的命名约定，使用TABLE_NAME_IN_PROCESS_METRICS常量
3. 配置文件完整：包含了中英文双语配置，权限设置合理
4. 数据源配置正确：为新表设置了1s数据源，符合profile类型数据的特点

🔧 技术实现分析

1. 配置文件 (6个新增文件)

• 指标配置：正确定义了profile_value计数器类型和row统计字段
• 标签配置：复制了完整的通用标签定义，与in_process表保持一致
• 权限配置：所有字段设置为111权限，允许完全访问

2. Go代码变更 (6个文件修改)

• const.go:38-39：新增TABLE_NAME_IN_PROCESS_METRICS常量定义 ✅
• const.go:63：正确添加到DB_TABLE_MAP的profile数据库配置中 ✅
• clickhouse.go:1324-1326：为新表设置1s数据源，逻辑正确 ✅
• metrics.go:353,667：复用in_process的指标处理逻辑，避免重复代码 ✅
• description.go:771,939：更新了标签描述逻辑的条件判断 ✅

🎯 问题分析对比

已修复的问题：
✅ 第一轮审查发现的英文配置问题已完全修复

• 之前在in_process_metrics.en文件中出现的中文字符已被正确的英文翻译替换
• profile_event_type现在显示为"Profile Type"
• profile_language_type现在显示为"Language Type"

新发现的改进点：

1. 配置一致性 (轻微优化建议)

• in_process_metrics.en文件第3行的row字段单位列为空，而中文版有"个"，建议英文版添加单位如"Count"

2. 代码逻辑合理性验证

• description.go:771的条件判断正确排除了in_process_metrics表的tag字段处理
• description.go:939的外部字段查询逻辑也正确排除了新表

📊 测试建议

建议验证以下场景：

1. 新表的查询API是否正常响应
2. 中英文字段显示名称是否正确
3. 1s数据源的查询性能
4. 与现有in_process表的功能隔离性

⚡ 性能考虑

• 复用现有指标处理逻辑，不会增加额外的性能开销
• 1s数据源设置合理，符合profile数据的时间粒度要求

🎯 最终评估

代码质量：9/10

• 实现正确且完整
• 之前发现的问题已全部修复
• 遵循了现有的代码规范和架构模式
• 中英文配置现在完全一致

建议操作：✅ 可以合并
这是一个高质量的PR，所有核心功能实现正确，之前的问题已修复，可以安全合并到主分支。

可选优化： 可以考虑为英文版的row字段添加单位"Count"，但这不是阻塞性问题。

---