Skip to content

Conversation

@MaxKsyunz
Copy link
Collaborator

Callout distinction between original and current SQL engines.

Signed-off-by: Max Ksyunz [email protected]

Check List

  • 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.

* Callout distinction between original and current SQL engines.

Signed-off-by: Max Ksyunz <[email protected]>
@MaxKsyunz MaxKsyunz requested a review from a team as a code owner August 1, 2022 18:30
@codecov-commenter
Copy link

codecov-commenter commented Aug 1, 2022

Codecov Report

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

@@              Coverage Diff              @@
##               main     #718       +/-   ##
=============================================
- 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 Δ
...rg/opensearch/sql/analysis/symbol/SymbolTable.java
...main/java/org/opensearch/sql/executor/Explain.java
...pensearch/sql/sql/parser/AstExpressionBuilder.java
...rg/opensearch/sql/planner/logical/LogicalSort.java
...earch/sql/opensearch/data/utils/ObjectContent.java
...rch/executor/protector/NoopExecutionProtector.java
...ggregation/ExpressionAggregationScriptFactory.java
...expression/window/frame/CurrentRowWindowFrame.java
...ql/opensearch/request/OpenSearchScrollRequest.java
.../sql/planner/physical/PhysicalPlanNodeVisitor.java
... and 263 more

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

joshuali925
joshuali925 previously approved these changes Aug 1, 2022
dai-chen
dai-chen previously approved these changes Aug 2, 2022
Copy link
Collaborator

@dai-chen dai-chen left a comment

Choose a reason for hiding this comment

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

Minor comment. Thanks for the changes!

Signed-off-by: MaxKsyunz <[email protected]>
@MaxKsyunz MaxKsyunz dismissed stale reviews from dai-chen and joshuali925 via 2cc610c August 3, 2022 20:18
@MaxKsyunz
Copy link
Collaborator Author

Please see #728 -- same change, but with passing DCO check.

@MaxKsyunz MaxKsyunz closed this Aug 5, 2022
@MaxKsyunz MaxKsyunz deleted the integ-doc-sql-ver-note branch February 8, 2023 17:59
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