Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix the problem of Tag selector "=" filter can not match the whole word #84

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

it-linnan
Copy link

@it-linnan it-linnan commented Apr 6, 2020

Motivation:

Tag选择器的“=”筛选,不能实现全字匹配。
比如jvm.threads.daemon{app="foo"},这样查询
实际结果是,能够查询到foo和gateway等多个应用的时序数据
预期结果是,只查询到foo应用的时序数据

Modification:

使用ES的keyword特性,阻止ES进行分词

Result:

Fixes #83 .

@sofastack-bot sofastack-bot bot changed the title 修复Tag选择器"="筛选不能全字匹配的问题 Fix the problem of Tag selector "=" filter can not match the whole word Apr 6, 2020
@sofastack-bot
Copy link

sofastack-bot bot commented Apr 6, 2020

Hi @it-linnan, we detect non-English characters in the issue. This comment is an auto translation by @sofastack-robot to help other users to understand this issue.

We encourage you to describe your issue in English which is more friendly to other users.

Motivation: "=" filter of ag selector cannot match whole words. For example, jvm.threads.daemon {app = "foo"}, so that the actual result of the query is that it can query the time series data of multiple applications such as foo and gateway. The expected result is that only the time series data of the foo application can be queried Use ES's keyword feature to prevent ES from tokenizing ### Result: Fixes # 83.

@sofastack-bot
Copy link

sofastack-bot bot commented Apr 6, 2020

Hi @it-linnan, Please sign Contributor License Agreement!

After you signed CLA, we will automatically sync the status of this pull request in 3 minutes.

@sofastack-bot sofastack-bot bot added bug Something isn't working cla:no First-time contributor question Further information is requested size/XS labels Apr 6, 2020
@sofastack-bot sofastack-bot bot added cla:yes and removed cla:no labels Jun 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working cla:yes First-time contributor question Further information is requested size/XS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Tag selector [=] Filter
1 participant