Skip to content

fix(ai-proxy): avoid reserved key column in setting query#6759

Merged
erda-bot merged 4 commits intoerda-project:masterfrom
sfwn:codex/ai-proxy-setting-query-key
Apr 13, 2026
Merged

fix(ai-proxy): avoid reserved key column in setting query#6759
erda-bot merged 4 commits intoerda-project:masterfrom
sfwn:codex/ai-proxy-setting-query-key

Conversation

@sfwn
Copy link
Copy Markdown
Member

@sfwn sfwn commented Apr 9, 2026

What this PR does / why we need it:

AI-Proxy avoid MySQL reserved key column issue in setting query.

Which issue(s) this PR fixes:

Specified Reviewers:

ChangeLog

Language Changelog
🇺🇸 English AI-Proxy avoid reserved key column issue in setting query
🇨🇳 中文 AI-Proxy 修复 setting 查询中 key 保留字导致的 SQL 问题

Need cherry-pick to release versions?

Add comment like /cherry-pick release/1.0 when this PR is merged.

@erda-bot erda-bot added the bugfix label Apr 9, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 15.69%. Comparing base (976ab00) to head (3a513ff).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #6759   +/-   ##
=======================================
  Coverage   15.69%   15.69%           
=======================================
  Files        3988     3988           
  Lines      327396   327396           
=======================================
+ Hits        51393    51396    +3     
  Misses     270148   270148           
+ Partials     5855     5852    -3     
Flag Coverage Δ
by-github-actions 15.69% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
internal/apps/ai-proxy/models/setting/dbclient.go 31.77% <100.00%> (ø)

... and 4 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@luobily
Copy link
Copy Markdown
Member

luobily commented Apr 13, 2026

/approve

@erda-bot erda-bot merged commit 30d9bf7 into erda-project:master Apr 13, 2026
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

3 participants