Skip to content

fix: align SQL manage list and statistics filter_error_priority - #901

Merged
LordofAvernus merged 2 commits into
zjrc_3.2408from
dev-zjrc-sql-manage-error-priority-pagination-20260929
Sep 30, 2026
Merged

LordofAvernus merged 2 commits into
zjrc_3.2408from
dev-zjrc-sql-manage-error-priority-pagination-20260929

Conversation

@LordofAvernus

Copy link
Copy Markdown
Collaborator

关联的 issue

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

描述你的变更

  • SqlManage 客户端类型显式声明 filter_error_priority(列表与统计)
  • SQLEEIndex 去掉交叉扩展,按类型传递优先级筛选
  • 请求测锁定列表/统计同参、告警 omit、统计失败保留总数

确认项(pr提交后操作)

Tip

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


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

LordofAvernus and others added 2 commits September 30, 2026 12:46
Align list/statistics params with the statistics API contract so the
frontend can pass P0/P1 without cross-type holes.

Co-authored-by: Cursor <cursoragent@cursor.com>
Use typed list params for error priority, and add request tests for
P0/P1 co-pass, omit-on-alert, and retain total when statistics fails.

Co-authored-by: Cursor <cursoragent@cursor.com>
@github-actions

Copy link
Copy Markdown

Coverage report

St.❔
Category Percentage Covered / Total
🟢 Statements 90.07% 15436/17138
🟡 Branches 73.08% 6497/8890
🟢 Functions 90.41% 5373/5943
🟢 Lines 90.58% 15172/16749

Test suite run success

2325 tests passing in 629 suites.

Report generated by 🧪jest coverage report action from 82a6ff5

@LordofAvernus
LordofAvernus merged commit f4a2d9d into zjrc_3.2408 Sep 30, 2026
9 checks passed
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.

1 participant