Skip to content

[fix] Implementing match_phrase_edge without index query method (#41658) - #42098

Merged
airborne12 merged 1 commit into
apache:branch-2.1from
zzzxl1993:branch-2.1.202410101051
Oct 22, 2024
Merged

[fix] Implementing match_phrase_edge without index query method (#41658)#42098
airborne12 merged 1 commit into
apache:branch-2.1from
zzzxl1993:branch-2.1.202410101051

Conversation

@zzzxl1993

@zzzxl1993zzzxl1993 commented Oct 18, 2024

Copy link
Copy Markdown
Contributor

@doris-robot

Copy link
Copy Markdown

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
Copy Markdown
ContributorAuthor

run buildall

@github-actions

Copy link
Copy Markdown
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@zzzxl1993

Copy link
Copy Markdown
ContributorAuthor

run buildall

@zzzxl1993
zzzxl1993force-pushed the branch-2.1.202410101051 branch from 5ccac51 to 0ff8be4CompareOctober 22, 2024 04:16
@github-actions

Copy link
Copy Markdown
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@zzzxl1993

Copy link
Copy Markdown
ContributorAuthor

run buildall

…he#41658)
1. Supports match_phrase_edge query without creating an inverted index.
@zzzxl1993
zzzxl1993force-pushed the branch-2.1.202410101051 branch from 0ff8be4 to 808e4a7CompareOctober 22, 2024 05:05
@zzzxl1993

Copy link
Copy Markdown
ContributorAuthor

run buildall

@github-actions

Copy link
Copy Markdown
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@doris-robot

Copy link
Copy Markdown

TeamCity be ut coverage result:
Function Coverage: 36.01% (9347/25958)
Line Coverage: 27.59% (76821/278479)
Region Coverage: 26.39% (39460/149519)
Branch Coverage: 23.18% (20086/86666)
Coverage Report: http://coverage.selectdb-in.cc/coverage/808e4a70d78495fc8a632ca1b82a0406398777da_808e4a70d78495fc8a632ca1b82a0406398777da/report/index.html

@airborne12airborne12 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@github-actionsgithub-actionsBot added the approved Indicates a PR has been approved by one committer. label Oct 22, 2024
@github-actions

Copy link
Copy Markdown
Contributor

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

@github-actions

Copy link
Copy Markdown
Contributor

PR approved by anyone and no changes requested.

@airborne12
airborne12 merged commit e2bdac3 into apache:branch-2.1Oct 22, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approvedIndicates a PR has been approved by one committer.reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@zzzxl1993@doris-robot@airborne12