Skip to content

Conversation

@zhangyj21-lh
Copy link
Contributor

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

Snapshots:

Include snapshots for easier review.

Checklist:

  • My code follows the style guidelines of this project
  • I have already rebased the commits and make the commit message conform to the project standard.
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • Any dependent changes have been merged and published in downstream modules

@github-actions github-actions bot added the fix Bug fixes label Dec 11, 2025
@zhangyj21-lh
Copy link
Contributor Author

1、 修复了DBGPT全文存储类型保存的问题(https://github.com/eosphoros-ai/DB-GPT/issues/2941)
2、修复了Elasticsearch关键词检索(https://github.com/eosphoros-ai/DB-GPT/issues/2946)

@zhangyj21-lh
Copy link
Contributor Author

出现的问题:1、DBGPT全文存储配置上之后,上传文档会提示参数数量错误,full_text /elasticsearch.py中aload_document(self, chunks: List[Chunk])函数:这里只给了2个参数,实际上传入3个参数,因此会报错
2、我按照文档连接好了Elasticsearch(测试显示连接成功),在配置文件里面配置了
#FullText
[rag.storage.full_text]
type = "elasticsearch"
host="127.0.0.1"
port=9201(9200端口被占用换了这个)。在创建知识库时选择full text,还是无法启用全文检索,提示2025-12-08 11:30:22 supermicro dbgpt.storage.base[2022238]WARNING Full text search is not supported in this index store.
2025-12-08 11:30:22 supermicro dbgpt_serve.rag.retriever.knowledge_space[2022238]WARNING Full text search is not supported for this stor
age connector.
作以上修改之后,关键词检索成功:
image

@chenliang15405
Copy link
Collaborator

@zhangyj21-lh hi please fix code format, use make fmt and make fmt-check

@zhangyj21-lh
Copy link
Contributor Author

@zhangyj21-lh hi please fix code format, use make fmt and make fmt-check

@chenliang15405 hi, I have committed the code after the check. Please check it.

Copy link
Collaborator

@chenliang15405 chenliang15405 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Collaborator

@Aries-ckt Aries-ckt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks for your contribution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix Bug fixes

Projects

None yet

3 participants