Skip to content

Feat sqlserver insight ce#3255

Merged
LordofAvernus merged 2 commits intomainfrom
feat-sqlserver-insight-ce
Apr 16, 2026
Merged

Feat sqlserver insight ce#3255
LordofAvernus merged 2 commits intomainfrom
feat-sqlserver-insight-ce

Conversation

@BugsGuru
Copy link
Copy Markdown
Collaborator

@BugsGuru BugsGuru commented Apr 16, 2026

User description

关联的 issue

https://github.com/actiontech/sqle-ee/issues/2700

描述你的变更

性能洞察功能支持 SQLServer 数据源

确认项(pr提交后操作)

Tip

请在指定复审人之前,确认并完成以下事项,完成后✅


  • 我已完成自测
  • 我已记录完整日志方便进行诊断
  • 我已在关联的issue里补充了实现方案
  • 我已在关联的issue里补充了测试影响面
  • 我已确认了变更的兼容性,如果不兼容则在issue里标记 not_compatible
  • 我已确认了是否要更新文档,如果要更新则在issue里标记 need_update_doc

assign in @LordofAvernus


Description

  • 增加SQLServer性能采集配置

  • 添加SQLServer慢SQL采集信息

  • 新增筛选SQL审计实例逻辑

  • 增加SQLServer启动时间辅助函数


Diagram Walkthrough

flowchart LR
  A["添加SQLServer支持"] --> B["过滤无审核实例"]
  A --> C["新增获取启动时间函数"]
  A --> D["更新国际化消息"]
Loading

File Walkthrough

Relevant files
Enhancement
instance_audit_plan.go
更新实例数据过滤逻辑                                                                                             

sqle/api/controller/v2/instance_audit_plan.go

  • 增加过滤逻辑,仅返回开启SQL审核的实例
+6/-0     
message_zh.go
增加中文国际化SQLServer提示                                                                             

sqle/locale/message_zh.go

  • 新增SQLServer性能采集提示信息
  • 新增SQLServer慢SQL采集提示信息
  • 新增SQLServer活跃会话采集提示信息
+6/-0     
instance_audit_plan.go
新增SQLServer启动时间辅助函数                                                                           

sqle/model/instance_audit_plan.go

  • 新增获取SQLServer启动时间函数
  • 使用最大字符串比较获取时间
+12/-0   
active.en.toml
更新英文国际化SQLServer信息                                                                             

sqle/locale/active.en.toml

  • 增加SQLServer性能采集及慢SQL采集英文提示
  • 新增SQLServer活跃会话英文提示
+6/-0     
active.zh.toml
更新中文国际化SQLServer信息                                                                             

sqle/locale/active.zh.toml

  • 增加SQLServer性能采集及慢SQL采集中文提示
  • 新增SQLServer活跃会话中文提示
+6/-0     

@github-actions
Copy link
Copy Markdown

PR Reviewer Guide 🔍

🎫 Ticket compliance analysis ✅

2700 - Fully compliant

Compliant requirements:

• 增加SQLServer性能采集配置
• 添加SQLServer慢SQL采集信息
• 新增筛选SQL审计实例逻辑
• 增加SQLServer启动时间辅助函数

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

@github-actions
Copy link
Copy Markdown

PR Code Suggestions ✨

Explore these optional code suggestions:

CategorySuggestion                                                                                                                                    Impact
Possible issue
添加空指针检查

建议在访问 inst.SqlQueryConfig.AuditEnabled 前先检查 inst.SqlQueryConfig 是否为
nil,以防止出现空指针解引用的情况,从而避免引发 panic。
通过增加该检查,可以保证只有在配置不为
nil且审核已启用的情况下,才进行后续处理,从而提高代码健壮性。

sqle/api/controller/v2/instance_audit_plan.go [120-121]

-if req.FunctionalModule == v1.FunctionalModuleViewSQLInsight && !inst.SqlQueryConfig.AuditEnabled {
+if req.FunctionalModule == v1.FunctionalModuleViewSQLInsight && (inst.SqlQueryConfig == nil || !inst.SqlQueryConfig.AuditEnabled) {
     continue
 }
Suggestion importance[1-10]: 8

__

Why: The suggestion enhances robustness by adding a nil-check on inst.SqlQueryConfig to prevent potential panics when accessing AuditEnabled, and it correctly identifies the affected lines.

Medium

@LordofAvernus LordofAvernus merged commit 0d4cca6 into main Apr 16, 2026
4 checks passed
@BugsGuru BugsGuru deleted the feat-sqlserver-insight-ce branch April 16, 2026 07:10
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.

2 participants