Skip to content

Conversation

@opensearch-trigger-bot
Copy link
Contributor

Backport b919ba0 from #1166

Added Tests And Implementation For Match_Query, Match_Phrase, and Multi_Match Functions

Signed-off-by: GabeFernandez310 <[email protected]>

Signed-off-by: GabeFernandez310 <[email protected]>
(cherry picked from commit b919ba0)
@opensearch-trigger-bot opensearch-trigger-bot bot requested a review from a team as a code owner January 6, 2023 18:37
@dai-chen dai-chen added the enhancement New feature or request label Jan 6, 2023
@codecov-commenter
Copy link

codecov-commenter commented Jan 6, 2023

Codecov Report

Merging #1236 (b919ba0) into 2.x (de79815) will decrease coverage by 0.07%.
The diff coverage is 100.00%.

❗ Current head b919ba0 differs from pull request most recent head c7b90ec. Consider uploading reports for the commit c7b90ec to get more accurate results

@@             Coverage Diff              @@
##                2.x    #1236      +/-   ##
============================================
- Coverage     98.32%   98.24%   -0.08%     
+ Complexity     3555     3389     -166     
============================================
  Files           346      332      -14     
  Lines          8768     8382     -386     
  Branches        554      513      -41     
============================================
- Hits           8621     8235     -386     
  Misses          142      142              
  Partials          5        5              
Flag Coverage Δ
sql-engine 98.24% <100.00%> (-0.08%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...expression/function/RelevanceFunctionResolver.java 100.00% <ø> (ø)
...ch/sql/planner/optimizer/LogicalPlanOptimizer.java 100.00% <ø> (ø)
...rg/opensearch/sql/analysis/ExpressionAnalyzer.java 100.00% <100.00%> (ø)
...a/org/opensearch/sql/data/model/ExprDateValue.java 100.00% <100.00%> (ø)
...g/opensearch/sql/data/model/ExprDatetimeValue.java 100.00% <100.00%> (ø)
...a/org/opensearch/sql/data/model/ExprTimeValue.java 100.00% <100.00%> (ø)
.../opensearch/sql/data/model/ExprTimestampValue.java 100.00% <100.00%> (ø)
.../org/opensearch/sql/data/model/ExprValueUtils.java 100.00% <100.00%> (ø)
...c/main/java/org/opensearch/sql/expression/DSL.java 100.00% <100.00%> (ø)
...arch/sql/expression/datetime/DateTimeFunction.java 100.00% <100.00%> (ø)
... and 44 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Copy link
Collaborator

@Yury-Fridlyand Yury-Fridlyand left a comment

Choose a reason for hiding this comment

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

CI failed

@GabeFernandez310 GabeFernandez310 mentioned this pull request Jan 6, 2023
6 tasks
@YANG-DB YANG-DB closed this in #1241 Jan 7, 2023
@Yury-Fridlyand Yury-Fridlyand deleted the backport/backport-1166-to-2.x branch January 7, 2023 00:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants