Skip to content

Conversation

@opensearch-trigger-bot
Copy link
Contributor

Backport 5aef2e6 from #873

Signed-off-by: vamsi-amazon <[email protected]>
(cherry picked from commit 5aef2e6)
@opensearch-trigger-bot opensearch-trigger-bot bot requested a review from a team as a code owner September 30, 2022 16:19
@codecov-commenter
Copy link

Codecov Report

Merging #877 (dea47e3) into main (9e28601) will decrease coverage by 32.06%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##               main     #877       +/-   ##
=============================================
- Coverage     94.83%   62.76%   -32.07%     
=============================================
  Files           291       10      -281     
  Lines          7795      658     -7137     
  Branches        568      119      -449     
=============================================
- Hits           7392      413     -6979     
+ Misses          349      192      -157     
+ Partials         54       53        -1     
Flag Coverage Δ
query-workbench 62.76% <ø> (ø)
sql-engine ?

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

Impacted Files Coverage Δ
...pensearch/sql/planner/physical/DedupeOperator.java
...ch/sql/planner/optimizer/LogicalPlanOptimizer.java
...ssion/operator/arthmetic/MathematicalFunction.java
...rg/opensearch/sql/data/model/ExprMissingValue.java
.../main/java/org/opensearch/sql/planner/Planner.java
...c/main/java/org/opensearch/sql/sql/SQLService.java
.../org/opensearch/sql/ppl/utils/ArgumentFactory.java
...ensearch/sql/analysis/NamedExpressionAnalyzer.java
...l/expression/window/ranking/DenseRankFunction.java
...age/script/filter/lucene/relevance/MatchQuery.java
... and 271 more

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

@penghuo penghuo merged commit e733e7b into main Sep 30, 2022
@github-actions github-actions bot deleted the backport/backport-873-to-main branch September 30, 2022 16:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants