Skip to content

fix(memory): lock Elasticsearch metadata.tags mapping - #103

Open
openjiuwen-sync-bot[bot] wants to merge 2 commits into
openJiuwen-ai:mem2.0from
openjiuwenai:sync/pr-188
Open

fix(memory): lock Elasticsearch metadata.tags mapping#103
openjiuwen-sync-bot[bot] wants to merge 2 commits into
openJiuwen-ai:mem2.0from
openjiuwenai:sync/pr-188

Conversation

@openjiuwen-sync-bot

@openjiuwen-sync-bot openjiuwen-sync-bot Bot commented Aug 7, 2026

Copy link
Copy Markdown

Paired: GitHub #103GitCode !188

修复内容

修复 Elasticsearch metadata.tags 动态 mapping 冲突:日期格式标签不再被推断为 date,显式固定为 keyword

变更范围

  • 在新索引 mapping 中声明 metadata.tagskeyword
  • 对已有但尚未声明该字段的索引通过 put_mapping 补齐
  • 增加新旧索引 mapping 回归测试

验证

  • test_dyn_ext_003.py 通过,真实抽取产生 到期时间:2025-12-31 标签并成功写入 ES
  • test_dyn_ext_009.py 通过,日期标签和普通标签连续写入成功
  • 相关 storage/construction 单测通过

已错误映射为 date 的历史索引需要新建 index 后从 KV 真源重建,代码不会自动删除线上索引。

Refs: #​110

Linked Closing Issues:

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


flyman986250 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@openjiuwen-collaboration-bot

Copy link
Copy Markdown

head_sha: 2d15e535d5d09bbe06c5ea62e83a51f751d966ad

任务名称 结果 日志操作
静态检查 ✅SUCCESS 点此跳转
防投毒检查 ✅SUCCESS 点此跳转
开源合规检查 ✅SUCCESS 点此跳转
UT测试 ❌FAILED 点此跳转
ST测试 N/A N/A
build 编译包 N/A N/A
ruff codecheck ✅SUCCESS N/A

@openjiuwen-collaboration-bot

Copy link
Copy Markdown

head_sha: 2d15e535d5d09bbe06c5ea62e83a51f751d966ad

check-pr

@openjiuwen-collaboration-bot

Copy link
Copy Markdown

head_sha: 607ec1f1a727773c0aa5a82758036abecbb9b742

任务名称 结果 日志操作
静态检查 ✅SUCCESS 点此跳转
防投毒检查 ✅SUCCESS 点此跳转
开源合规检查 ✅SUCCESS 点此跳转
UT测试 ❌FAILED 点此跳转
ST测试 N/A N/A
build 编译包 N/A N/A
ruff codecheck ✅SUCCESS N/A

@openjiuwen-collaboration-bot

Copy link
Copy Markdown

head_sha: 2d15e535d5d09bbe06c5ea62e83a51f751d966ad

任务名称 结果 日志操作
静态检查 ✅SUCCESS 点此跳转
防投毒检查 ✅SUCCESS 点此跳转
开源合规检查 ✅SUCCESS 点此跳转
UT测试 ❌FAILED 点此跳转
ST测试 N/A N/A
build 编译包 N/A N/A
ruff codecheck ✅SUCCESS N/A

@openjiuwen-collaboration-bot

Copy link
Copy Markdown

head_sha: 2d15e535d5d09bbe06c5ea62e83a51f751d966ad

check-pr

@openjiuwen-collaboration-bot

Copy link
Copy Markdown

head_sha: 2d15e535d5d09bbe06c5ea62e83a51f751d966ad

任务名称 结果 日志操作
静态检查 ✅SUCCESS 点此跳转
防投毒检查 ✅SUCCESS 点此跳转
开源合规检查 ✅SUCCESS 点此跳转
UT测试 ✅SUCCESS 点此跳转
ST测试 N/A N/A
build 编译包 N/A N/A
ruff codecheck ✅SUCCESS N/A

# ES 的 date_detection 会在 dynamic_templates 匹配前把
# "2025-12-31" 这类标签识别为 date,导致同一字段的
# keyword/date mapping 冲突。
"properties": {"tags": {"type": "keyword"}},

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

head_sha: 2d15e535d5d09bbe06c5ea62e83a51f751d966ad

metadata内只有tags字段吗?是否存在其他字段会出现类似的问题,如果存在,建议统一关闭date_detection

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant