Skip to content

Conversation

@zzzxl1993
Copy link
Contributor

@zzzxl1993 zzzxl1993 commented Sep 27, 2024

Proposed changes

  1. Set enable_phrase_query_sequential_opt = true to optimize conjunction matching in sequential phrase queries.
  2. For example, match_phrase "赵丽颖 中国 ~20+" ensures that "赵丽颖" appears consecutively, and "中国" also appears consecutively, while maintaining the semantics of sequential phrase queries.

@doris-robot
Copy link

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR

Since 2024-03-18, the Document has been moved to doris-website.
See Doris Document.

@zzzxl1993
Copy link
Contributor Author

run buildall

1 similar comment
@zzzxl1993
Copy link
Contributor Author

run buildall

@zzzxl1993
Copy link
Contributor Author

run buildall

@doris-robot
Copy link

TeamCity be ut coverage result:
Function Coverage: 37.31% (9634/25819)
Line Coverage: 28.72% (79776/277789)
Region Coverage: 28.16% (41261/146546)
Branch Coverage: 24.77% (21016/84828)
Coverage Report: http://coverage.selectdb-in.cc/coverage/9350770103f944e30ae4521d83c6058b7acc7bdf_9350770103f944e30ae4521d83c6058b7acc7bdf/report/index.html

Copy link
Member

@airborne12 airborne12 left a comment

Choose a reason for hiding this comment

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

LGTM

@github-actions
Copy link
Contributor

github-actions bot commented Oct 7, 2024

PR approved by at least one committer and no changes requested.

@github-actions github-actions bot added approved Indicates a PR has been approved by one committer. reviewed labels Oct 7, 2024
@github-actions
Copy link
Contributor

github-actions bot commented Oct 7, 2024

PR approved by anyone and no changes requested.

Copy link
Contributor

@csun5285 csun5285 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
Contributor

@qidaye qidaye left a comment

Choose a reason for hiding this comment

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

LGTM

@qidaye qidaye merged commit f313f1f into apache:master Oct 10, 2024
eldenmoon pushed a commit to eldenmoon/incubator-doris that referenced this pull request Oct 10, 2024
…41432)

1. Set enable_phrase_query_sequential_opt = true to optimize conjunction
matching in sequential phrase queries.
2. For example, match_phrase "赵丽颖 中国 ~20+" ensures that "赵丽颖" appears
consecutively, and "中国" also appears consecutively, while maintaining
the semantics of sequential phrase queries.
cjj2010 pushed a commit to cjj2010/doris that referenced this pull request Oct 12, 2024
…41432)

1. Set enable_phrase_query_sequential_opt = true to optimize conjunction
matching in sequential phrase queries.
2. For example, match_phrase "赵丽颖 中国 ~20+" ensures that "赵丽颖" appears
consecutively, and "中国" also appears consecutively, while maintaining
the semantics of sequential phrase queries.
amorynan pushed a commit to amorynan/doris that referenced this pull request Oct 12, 2024
…41432)

1. Set enable_phrase_query_sequential_opt = true to optimize conjunction
matching in sequential phrase queries.
2. For example, match_phrase "赵丽颖 中国 ~20+" ensures that "赵丽颖" appears
consecutively, and "中国" also appears consecutively, while maintaining
the semantics of sequential phrase queries.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by one committer. reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants