Skip to content

Comments

refactor(knowledge_base): remove FULLTEXT index from content field#3209

Merged
winfredLIN merged 1 commit intomainfrom
fix-2640
Feb 6, 2026
Merged

refactor(knowledge_base): remove FULLTEXT index from content field#3209
winfredLIN merged 1 commit intomainfrom
fix-2640

Conversation

@LordofAvernus
Copy link
Collaborator

@LordofAvernus LordofAvernus commented Feb 6, 2026

User description

关联的 issue

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

描述你的变更

移除FULLTEXT index

确认项(pr提交后操作)

Tip

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


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


Description

  • 移除 sqle/model/knowledge_base.goContent 字段的 FULLTEXT 索引

  • 保持 Content 字段类型为 text


Diagram Walkthrough

flowchart LR
  A["删除 Content FULLTEXT 索引"] --> B["保留 text 类型"]
Loading

File Walkthrough

Relevant files
Enhancement
knowledge_base.go
删除 FULLTEXT 索引配置                                                                                 

sqle/model/knowledge_base.go

  • 删除 Content 字段的 FULLTEXT 索引声明
  • 保持 Content 字段类型为 text,不再包含索引配置
+1/-1     

@github-actions
Copy link

github-actions bot commented Feb 6, 2026

PR Reviewer Guide 🔍

🎫 Ticket compliance analysis 🔶

2640 - Partially compliant

Compliant requirements:

  • 移除 FULLTEXT 索引
  • 保持 Content 字段类型为 text

Non-compliant requirements:

[]

Requires further human verification:

[]

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

字段修改确认

请确保移除 FULLTEXT 索引后不会对文本搜索或数据库性能产生负面影响,确认相关逻辑在系统中得到妥善处理。

Content     string        `gorm:"type:text" json:"content"`                // 内容

@github-actions
Copy link

github-actions bot commented Feb 6, 2026

PR Code Suggestions ✨

No code suggestions found for the PR.

@winfredLIN winfredLIN merged commit 494a0a9 into main Feb 6, 2026
4 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.

2 participants