Skip to content

Conversation

@iwanghc
Copy link
Collaborator

@iwanghc iwanghc commented Nov 7, 2025

User description

关联的 issue

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

描述你的变更

sql调优新增列选择性参数
sql调优任务中断重启功能定义

确认项(pr提交后操作)

Tip

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


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

link #3164


Description

  • 添加 enable_high_analysis 字段到优化结构体

  • 更新 API 与 Swagger 文档描述

  • 新增 SyncOptimizeResult 函数并调用于启动流程


Diagram Walkthrough

flowchart LR
    A["新增enable_high_analysis字段"] -- "更新" --> B["更新API文档"]
    B -- "调整" --> C["更新Swagger文档"]
    A -- "扩展" --> D["新增SyncOptimizeResult函数"]
    D -- "调用" --> E["启动流程调用同步任务"]
Loading

File Walkthrough

Relevant files
Enhancement
optimization.go
优化结构体新增高级分析字段                                                                                       

sqle/api/controller/v2/optimization.go

  • 增加 enable_high_analysis 布尔字段
  • 调整字段顺序与注释
+9/-7     
optimization_server_ce.go
新增 SyncOptimizeResult 空函数实现                                                           

sqle/server/optimization/optimization_server_ce.go

  • 新增空实现的 SyncOptimizeResult 函数
  • 适用于非企业版同步优化处理
+7/-0     
sqled.go
启动流程中新增优化结果同步调用                                                                                   

sqle/sqled.go

  • 在启动流程中调用 SyncOptimizeResult
  • 异步执行同步优化结果任务
+4/-0     
Documentation
docs.go
文档中新增高级分析字段说明                                                                                       

sqle/docs/docs.go

  • 添加 enable_high_analysis 字段描述
  • 更新 JSON 接口文档定义
+4/-0     
swagger.json
更新 Swagger JSON 接口文档                                                                         

sqle/docs/swagger.json

  • 添加 enable_high_analysis 字段描述
  • 更新 Swagger JSON 文档接口定义
+4/-0     
swagger.yaml
更新 Swagger YAML 接口文档                                                                         

sqle/docs/swagger.yaml

  • 增加 enable_high_analysis 字段支持
  • 更新 Swagger YAML API 定义
+3/-0     

@actiontech-bot actiontech-bot requested review from LordofAvernus and removed request for LordofAvernus November 7, 2025 10:59
@github-actions
Copy link

github-actions bot commented Nov 7, 2025

PR Reviewer Guide 🔍

🎫 Ticket compliance analysis 🔶

2572 - Partially compliant

Compliant requirements:

  • 添加 enable_high_analysis 字段到优化结构体
  • 更新 API 与 Swagger 文档描述
  • 新增 SyncOptimizeResult 函数并调用于启动流程

Non-compliant requirements:

Requires further human verification:

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
🧪 PR contains tests
🔒 No security concerns identified
⚡ Recommended focus areas for review

空实现

新增的 SyncOptimizeResult 函数目前实现为空,建议确认该函数是否需要添加具体的业务逻辑或注释说明以便后续实现。

//go:build !enterprise
// +build !enterprise

package optimization

func SyncOptimizeResult() {
}

@github-actions
Copy link

github-actions bot commented Nov 7, 2025

Failed to generate code suggestions for PR

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