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

Repo split changes 2.x #1240

Closed
wants to merge 89 commits into from

Conversation

YANG-DB
Copy link
Member

@YANG-DB YANG-DB commented Jan 6, 2023

Description

SQL (branch 2.x) was split into the following repos:

See

Issues Resolved

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.

Yury-Fridlyand and others added 30 commits September 28, 2022 13:59
…nize-workflows (opensearch-project#864)

Reorganize GitHub workflows.

(cherry picked from commit 10e44ee)
Signed-off-by: Yury-Fridlyand <[email protected]>
Signed-off-by: vamsi-amazon <[email protected]>
(cherry picked from commit 5aef2e6)

Co-authored-by: vamsi-amazon <[email protected]>
…en-links (opensearch-project#932)

Fix broken links

(cherry picked from commit 4b50a01)
Signed-off-by: Yury-Fridlyand <[email protected]>
* Bump version to 3.0

Signed-off-by: Peng Huo <[email protected]>
…for sql and workbench (opensearch-project#962)

* Merge pull request opensearch-project#907 from derek-ho/enable-windows-macos

feat: enable ci for windows and macos
(cherry picked from commit 0699baa)
Signed-off-by: Derek Ho <[email protected]>

* always and os==ubuntu

Signed-off-by: Derek Ho <[email protected]>

Signed-off-by: Derek Ho <[email protected]>
Signed-off-by: Chen Dai <[email protected]>
(cherry picked from commit 9944f2e)

Co-authored-by: Chen Dai <[email protected]>
 - sql-cli
 - sql-odbc
 - sql-jdbc
 - dashboards-query-workbench

Signed-off-by: YANG-DB <[email protected]>
 - sql-cli
 - sql-odbc
 - sql-jdbc
 - dashboards-query-workbench

Signed-off-by: YANG-DB <[email protected]>
Signed-off-by: Peng Huo <[email protected]>
 - move the BI tests to jdbc/odbc
 - move the sql-cli outside and clone for the doctest build

Signed-off-by: YANGDB <[email protected]>
 - move the BI tests to jdbc/odbc
 - move the sql-cli outside and clone for the doctest build

Signed-off-by: YANGDB <[email protected]>
* Fix precedence by reordering grammar rule

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

* Fix precedence in PPL

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

Signed-off-by: Chen Dai <[email protected]>
…arch-project#1187)

* Add between grammar and in-memory impl

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

* Add comparison test for between

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

* Add doctest for between

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

* Add not between support

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

* Fix doctest failure

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

* Refactor to rewrite to basic comparison expression

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

* Clean up unused code

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

* Prepare to publish PR

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

Signed-off-by: Chen Dai <[email protected]>
(cherry picked from commit 6c0af83)

Co-authored-by: Chen Dai <[email protected]>
* Updated UTC TIME Functions for like now FunctionProperties.

Signed-off-by: MitchellGale-BitQuill <[email protected]>

* Finished updates to tests.

Signed-off-by: MitchellGale-BitQuill <[email protected]>

* Removed unused import.

Signed-off-by: MitchellGale-BitQuill <[email protected]>

* Addressed PR comments.

Signed-off-by: MitchellGale-BitQuill <[email protected]>

* Removed extra import.

Signed-off-by: MitchellGale-BitQuill <[email protected]>

* removed `DateTimeFunction` used within class.

Signed-off-by: MitchellGale-BitQuill <[email protected]>

Signed-off-by: MitchellGale-BitQuill <[email protected]>
…pensearch-project#1067)

Change relevance functions that query fields to throw a SemanticCheckException when a field is queried that does not exist.

Signed-off-by: forestmvey <[email protected]>
…earch-project#1171)

* Add table write operator and builder

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

* Add UT for all new classes

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

* Rename child field

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

* Add columns field

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

* Update javadoc to prepare PR

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

Signed-off-by: Chen Dai <[email protected]>
(cherry picked from commit c494d92)

Co-authored-by: Chen Dai <[email protected]>
* Deprecated Spring IoC and using Guice instead

Signed-off-by: Peng Huo <[email protected]>

* fix checkstyle issue

Signed-off-by: Peng Huo <[email protected]>

* Pass injector to action

Signed-off-by: Peng Huo <[email protected]>

* update

Signed-off-by: Peng Huo <[email protected]>

Signed-off-by: Peng Huo <[email protected]>
@codecov-commenter
Copy link

codecov-commenter commented Jan 6, 2023

Codecov Report

Merging #1240 (b994240) into 2.x (fa1083b) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##                2.x    #1240      +/-   ##
============================================
- Coverage     98.35%   98.33%   -0.02%     
- Complexity     3599     3601       +2     
============================================
  Files           346      344       -2     
  Lines          8915     8928      +13     
  Branches        562      562              
============================================
+ Hits           8768     8779      +11     
- Misses          142      143       +1     
- Partials          5        6       +1     
Flag Coverage Δ
sql-engine 98.33% <100.00%> (-0.02%) ⬇️

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

Impacted Files Coverage Δ
...pensearch/sql/expression/function/FunctionDSL.java 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%> (ø)
...pensearch/sql/sql/parser/AstExpressionBuilder.java 99.05% <0.00%> (-0.95%) ⬇️
...rg/opensearch/sql/sql/config/SQLServiceConfig.java

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

GabeFernandez310 and others added 26 commits January 6, 2023 12:06
Signed-off-by: GabeFernandez310 <[email protected]>
Signed-off-by: YANGDB <[email protected]>

# Conflicts:
#	build.gradle
#	core/src/main/java/org/opensearch/sql/expression/DSL.java
#	core/src/main/java/org/opensearch/sql/expression/datetime/DateTimeFunction.java
#	core/src/main/java/org/opensearch/sql/expression/function/BuiltinFunctionName.java
#	core/src/test/java/org/opensearch/sql/expression/datetime/DateTimeFunctionTest.java
#	integ-test/src/test/java/org/opensearch/sql/sql/DateTimeFunctionIT.java
#	plugin/build.gradle
#	sql/src/main/antlr/OpenSearchSQLParser.g4
#	sql/src/main/java/org/opensearch/sql/sql/parser/AstExpressionBuilder.java
Signed-off-by: GabeFernandez310 <[email protected]>
# Conflicts:
#	sql-cli/MAINTAINERS.md
#	sql-jdbc/MAINTAINERS.md
#	sql-odbc/MAINTAINERS.md
Signed-off-by: GabeFernandez310 <[email protected]>
Signed-off-by: YANGDB <[email protected]>

# Conflicts:
#	sql-cli/MAINTAINERS.md
#	sql-jdbc/MAINTAINERS.md
#	sql-odbc/MAINTAINERS.md
Signed-off-by: YANGDB <[email protected]>

# Conflicts:
#	integ-test/build.gradle
# Conflicts:
#	sql/src/test/java/org/opensearch/sql/sql/parser/AstExpressionBuilderTest.java
…roject#1229)

* Resolve sub object field in search hit source

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

* Rename to unexpanded object

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

* Update IT with where condition

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

* Fix test index mapping

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

Signed-off-by: Chen Dai <[email protected]>
…nctions (opensearch-project#1205)

* Added long fix for CEIL, CEILING and FLOOR functions using LONG instead of INT for RETURN.

Signed-off-by: MitchellGale-BitQuill <[email protected]>

* Revert changes to JDBCConnection.java.

Signed-off-by: MitchellGale-BitQuill <[email protected]>

* Addressed PR comments.

Signed-off-by: MitchellGale-BitQuill <[email protected]>

* Made fixes to rst files.

Signed-off-by: MitchellGale-BitQuill <[email protected]>

* Simplify docs according to PR feedback.

Signed-off-by: Yury-Fridlyand <[email protected]>

Signed-off-by: MitchellGale-BitQuill <[email protected]>
Signed-off-by: Yury-Fridlyand <[email protected]>
Co-authored-by: Yury-Fridlyand <[email protected]>
Signed-off-by: YANGDB <[email protected]>

# Conflicts:
#	docs/user/dql/functions.rst
…ect#199) (opensearch-project#1223)

Added Support And Tests For Time Type in day_of_year, week_of_year, month_of_year Functions
Signed-off-by: GabeFernandez310 <[email protected]>
@YANG-DB
Copy link
Member Author

YANG-DB commented Jan 10, 2023

closing - duplicate with #1264

@YANG-DB YANG-DB closed this Jan 10, 2023
@YANG-DB YANG-DB deleted the repo-split-changes-2.x branch January 10, 2023 22:49
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.