Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge main into 1.x #473

Merged
merged 125 commits into from
Mar 9, 2022
Merged

Merge main into 1.x #473

merged 125 commits into from
Mar 9, 2022

Conversation

vamsimanohar
Copy link
Member

Description

[Describe what this change achieves]

Issues Resolved

[List any issues this PR will resolve]

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

peterzhuamazon and others added 30 commits June 16, 2021 14:51
…pensearch-project#146)

* Bump OpenSearch version from rc1 to 1.0.0

Signed-off-by: Chen Dai <[email protected]>

* Rename JDBC artifact by removing -rc1

Signed-off-by: Chen Dai <[email protected]>

* Remove rc1 qualifier in build workflow

Signed-off-by: Chen Dai <[email protected]>

* Remove rc1 from build tools version

Signed-off-by: Chen Dai <[email protected]>

* Fix IT failure

Signed-off-by: Chen Dai <[email protected]>

* Rollback build tools to rc1 due to known issue

Signed-off-by: Chen Dai <[email protected]>

* Bump CLI version

Signed-off-by: Chen Dai <[email protected]>

* Bump query workbench version

Signed-off-by: Chen Dai <[email protected]>

* Build against 1.0.0

Signed-off-by: Chen Dai <[email protected]>

* Update release notes drafter

Signed-off-by: Chen Dai <[email protected]>

* Update nodejs to 10.24.1

Signed-off-by: Chen Dai <[email protected]>
* Add release notes

Signed-off-by: Chen Dai <[email protected]>

* Change release date

Signed-off-by: Chen Dai <[email protected]>

* Add bug fixes section

Signed-off-by: Chen Dai <[email protected]>
…nsearch-integtest-demo

 Add Integtest.sh for OpenSearch integtest setups (workbench)
…roject#166)

* Support implicit type conversion for bool and string

Signed-off-by: Chen Dai <[email protected]>

* Fix lucene query pushdown issue

Signed-off-by: Chen Dai <[email protected]>

* Refactor lucene query methods

Signed-off-by: Chen Dai <[email protected]>

* Refactor builtin repo methods

Signed-off-by: Chen Dai <[email protected]>

* Add comparison test

Signed-off-by: Chen Dai <[email protected]>

* Fix comparison test

Signed-off-by: Chen Dai <[email protected]>

* Add doc test for user manual

Signed-off-by: Chen Dai <[email protected]>

* Fix doc test

Signed-off-by: Chen Dai <[email protected]>

* Fix design doc link

Signed-off-by: Chen Dai <[email protected]>

* Fix RST render issue

Signed-off-by: Chen Dai <[email protected]>

* Fix cast function pushdown issue

Signed-off-by: Chen Dai <[email protected]>

* Improve javadoc for PR

Signed-off-by: Chen Dai <[email protected]>

* Upload design doc

Signed-off-by: Chen Dai <[email protected]>

* Add more user manual

Signed-off-by: Chen Dai <[email protected]>

* Add more user manual

Signed-off-by: Chen Dai <[email protected]>

* Fix doctest

Signed-off-by: Chen Dai <[email protected]>
* Support construct AggregationResponseParser during Aggregator build stage (opensearch-project#108)

* Support construct AggregationResponseParser during Aggregator build stage

* modify the doc

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

* support distinct count aggregation

Signed-off-by: chloe-zh <[email protected]>

* fixed tests

Signed-off-by: chloe-zh <[email protected]>

* Merge remote-tracking branch 'upstream/develop' into issue/opensearch-project#100

Signed-off-by: chloe-zh <[email protected]>

# Conflicts:
#	opensearch/src/main/java/org/opensearch/sql/opensearch/storage/script/aggregation/dsl/MetricAggregationBuilder.java

* update

Signed-off-by: chloe-zh <[email protected]>

* updated user doc

Signed-off-by: chloe-zh <[email protected]>

* Update: support only count for distinct aggregations

Signed-off-by: chloe-zh <[email protected]>

* Update doc; removed distinct start

Signed-off-by: chloe-zh <[email protected]>

* Removed unnecessary methods

Signed-off-by: chloe-zh <[email protected]>

* update

Signed-off-by: chloe-zh <[email protected]>

* Impl stddev and variance function in SQL and PPL (opensearch-project#115)

* impl variance frontend and backend

* Support construct AggregationResponseParser during Aggregator build stage

* add var and varp for PPL

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

* add UT

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

* fix UT

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

* fix doc format

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

* fix doc format

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

* fix the doc

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

* add stddev_samp and stddev_pop

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

* fix UT coverage

* address comments

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

* Fix the aggregation filter missing in named aggregators (opensearch-project#123)

* Take the condition expression as property to the named aggregator when wrapping the delegated aggregator

Signed-off-by: chloe-zh <[email protected]>

* update

Signed-off-by: chloe-zh <[email protected]>

* Added test case where filtered agg is not pushed down

Signed-off-by: chloe-zh <[email protected]>

* update

Signed-off-by: chloe-zh <[email protected]>

* update

Signed-off-by: chloe-zh <[email protected]>

* update

Signed-off-by: chloe-zh <[email protected]>

* modified comparison test

Signed-off-by: chloe-zh <[email protected]>

* removed a comparison test and added it to aggregationIT

Signed-off-by: chloe-zh <[email protected]>

* added ppl IT test cases; added window function test cases

Signed-off-by: chloe-zh <[email protected]>

* moved distinct window function test cases to WindowsIT

Signed-off-by: chloe-zh <[email protected]>

* added ut

Signed-off-by: chloe-zh <[email protected]>

* update

Signed-off-by: chloe-zh <[email protected]>

* update

Signed-off-by: chloe-zh <[email protected]>

* addressed comments

Signed-off-by: chloe-zh <[email protected]>

* added test cases to meet the coverage requirement

Signed-off-by: chloe-zh <[email protected]>

* added test cases for distinct count map and array types

Signed-off-by: chloe-zh <[email protected]>

Co-authored-by: Peng Huo <[email protected]>
…project#171)

* Support implicit cast from string to temporal types

Signed-off-by: Chen Dai <[email protected]>

* Add comparison test

Signed-off-by: Chen Dai <[email protected]>

* Add doctest

Signed-off-by: Chen Dai <[email protected]>

* Fix doctest

Signed-off-by: Chen Dai <[email protected]>

* Add more user manual

Signed-off-by: Chen Dai <[email protected]>

* Add more user manual

Signed-off-by: Chen Dai <[email protected]>

* Fix doctest

Signed-off-by: Chen Dai <[email protected]>

* Update user manual

Signed-off-by: Chen Dai <[email protected]>
Use externally-defined OpenSearch version when specified.
…#178)

Bumps [path-parse](https://github.com/jbgutierrez/path-parse) from 1.0.6 to 1.0.7.
- [Release notes](https://github.com/jbgutierrez/path-parse/releases)
- [Commits](https://github.com/jbgutierrez/path-parse/commits/v1.0.7)

---
updated-dependencies:
- dependency-name: path-parse
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…t#202)

* bump workbench versions to 1.1
Signed-off-by: David Cui <[email protected]>

* bump version to 1.1.0.0 for 1.1 release
Signed-off-by: David Cui <[email protected]>

* add release notes for 1.1
Signed-off-by: David Cui <[email protected]>

* bump odbc files to 1.1.0.0
Signed-off-by: David Cui <[email protected]>
* Downscope request to local method

Signed-off-by: Chen Dai <[email protected]>

* Fix checkstyle

Signed-off-by: Chen Dai <[email protected]>
* supported match in sql and ppl where

Signed-off-by: chloe-zh <[email protected]>

* added legacy syntax in new parser

Signed-off-by: chloe-zh <[email protected]>

* added integration test

Signed-off-by: chloe-zh <[email protected]>

* updated user manual

Signed-off-by: chloe-zh <[email protected]>

* updated user manual

Signed-off-by: chloe-zh <[email protected]>

* update

Signed-off-by: chloe-zh <[email protected]>

* added ppl integ test, updated ppl manual

Signed-off-by: chloe-zh <[email protected]>

* update

Signed-off-by: chloe-zh <[email protected]>

* update

Signed-off-by: chloe-zh <[email protected]>

* update

Signed-off-by: chloe-zh <[email protected]>
* Update build to use public Maven repo

Signed-off-by: Abbas Hussain <[email protected]>

* Update runner to use macos-11 for odbc builds

Signed-off-by: Abbas Hussain <[email protected]>
* compatible with opensearch and all versions in odfe with sql plugin

Signed-off-by: chloe-zh <[email protected]>

* address comments

Signed-off-by: chloe-zh <[email protected]>
zhongnansu and others added 24 commits January 25, 2022 10:48
…hsql` (opensearch-project#392)

* rename sql-cli to opensearchsql

Signed-off-by: Zhongnan Su <[email protected]>

* update workflow

Signed-off-by: Zhongnan Su <[email protected]>
* bumping jackson-databind to 2.12.6

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

* bumping jackson-core & dataformat-cbor

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

* updated missed gradle files

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

* updated junit & added resolutions

Signed-off-by: Shenoy Pratik <[email protected]>
…rch-project#417)

* Support combination of group field and span in stats command

Signed-off-by: penghuo <[email protected]>
Add auto backport workflow and documentation.

Signed-off-by: David Cui <[email protected]>
* update backport workflow and add auto delete

Signed-off-by: David Cui <[email protected]>
Add default version as JDK 11 and add CI matrix for JDK 11 and 14

Signed-off-by: David Cui <[email protected]>
Signed-off-by: Guian Gumpac <[email protected]>
Signed-off-by: Yury Fridlyand <[email protected]>

Co-authored-by: Yury Fridlyand <[email protected]>
Co-authored-by: Yury-Fridlyand <[email protected]>
Optimize sort rewrite logic by using first and last parameter in Composite aggregation.

Signed-off-by: penghuo <[email protected]>
Support ISO 8601 (%y-%m-%dT%TZ) in Date Formats. In order to do that we have started supporting plain characters in date format without a % in front.
Signed-off-by: vamsi-amazon <[email protected]>
@codecov-commenter
Copy link

codecov-commenter commented Mar 9, 2022

Codecov Report

Merging #473 (9242b2b) into 1.x (3762023) will decrease coverage by 3.33%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##                1.x     #473      +/-   ##
============================================
- Coverage     99.89%   96.56%   -3.34%     
- Complexity     2512     2722     +210     
============================================
  Files           245      269      +24     
  Lines          5853     7242    +1389     
  Branches        342      527     +185     
============================================
+ Hits           5847     6993    +1146     
- Misses            5      196     +191     
- Partials          1       53      +52     
Flag Coverage Δ
query-workbench 62.91% <ø> (?)
sql-engine 99.92% <100.00%> (?)

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

Impacted Files Coverage Δ
...rch/sql/analysis/ExpressionReferenceOptimizer.java 100.00% <ø> (ø)
...ensearch/sql/analysis/NamedExpressionAnalyzer.java 100.00% <ø> (ø)
...org/opensearch/sql/analysis/QualifierAnalyzer.java 100.00% <ø> (ø)
...nsearch/sql/analysis/SelectExpressionAnalyzer.java 100.00% <ø> (ø)
...a/org/opensearch/sql/analysis/TypeEnvironment.java 100.00% <ø> (ø)
...nsearch/sql/analysis/WindowExpressionAnalyzer.java 100.00% <ø> (ø)
.../org/opensearch/sql/analysis/symbol/Namespace.java 100.00% <ø> (ø)
...rg/opensearch/sql/analysis/symbol/SymbolTable.java 100.00% <ø> (ø)
...search/sql/data/model/AbstractExprNumberValue.java 100.00% <ø> (ø)
...g/opensearch/sql/data/model/AbstractExprValue.java 100.00% <ø> (ø)
... and 252 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3762023...9242b2b. Read the comment docs.

@vamsimanohar vamsimanohar merged commit 5c09444 into opensearch-project:1.x Mar 9, 2022
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.