sql icon indicating copy to clipboard operation
sql copied to clipboard

Bump main to 3.0

Open ps48 opened this issue 3 years ago • 3 comments

Signed-off-by: Shenoy Pratik [email protected]

Description

bump main to 3.0

Issues Resolved

#692

Check List

  • [ ] New functionality includes testing.
    • [ ] All tests pass, including unit test, integration test and doctest
  • [ ] New functionality has been documented.
    • [ ] New functionality has javadoc added
    • [ ] New functionality has user manual doc added
  • [x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license. For more information on following Developer Certificate of Origin and signing off your commits, please check here.

ps48 avatar Jul 28 '22 19:07 ps48

SQL build fails due to missing ml-commons artifact

ps48 avatar Jul 28 '22 19:07 ps48

Codecov Report

Merging #715 (f50a961) into main (5c6fd72) will decrease coverage by 31.98%. The diff coverage is n/a.

@@              Coverage Diff              @@
##               main     #715       +/-   ##
=============================================
- Coverage     94.75%   62.76%   -31.99%     
=============================================
  Files           283       10      -273     
  Lines          7678      658     -7020     
  Branches        560      118      -442     
=============================================
- Hits           7275      413     -6862     
+ 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 Δ
...nsearch/sql/analysis/WindowExpressionAnalyzer.java
...gation/ExpressionAggregationScriptLeafFactory.java
...search/sql/planner/optimizer/pattern/Patterns.java
...ch/sql/expression/aggregation/CountAggregator.java
...arch/sql/opensearch/request/OpenSearchRequest.java
...rg/opensearch/sql/data/model/ExprBooleanValue.java
.../opensearch/sql/data/model/ExprTimestampValue.java
...l/planner/optimizer/rule/MergeFilterAndFilter.java
...ch/sql/opensearch/client/OpenSearchNodeClient.java
...nsearch/sql/analysis/SelectExpressionAnalyzer.java
... and 263 more

Help us with your feedback. Take ten seconds to tell us how you rate us.

codecov-commenter avatar Jul 28 '22 19:07 codecov-commenter

Don't merge this PR until we have agreement on https://github.com/opensearch-project/opensearch-plugins/issues/142#issuecomment-1218627274.

penghuo avatar Aug 18 '22 17:08 penghuo

Closing the PR for now. Will reopen once we have decision here

ps48 avatar Sep 14 '22 19:09 ps48