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

[Backport 2.x] Merging Async Query APIs feature branch into main. #2163

Merged
merged 1 commit into from
Sep 27, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport ae10857 from #2159.

* Glue datasource support (#2055)

Signed-off-by: Vamsi Manohar <reddyvam@amazon.com>

* Initial commit of new job APIs (#2050)

Signed-off-by: Vamsi Manohar <reddyvam@amazon.com>

* Create Job API (#2070)

* Create Job API

Signed-off-by: Vamsi Manohar <reddyvam@amazon.com>

* Refactor to Async Query API

Signed-off-by: Vamsi Manohar <reddyvam@amazon.com>

---------

Signed-off-by: Vamsi Manohar <reddyvam@amazon.com>

* Cancel Job API (#2126)

Signed-off-by: Vamsi Manohar <reddyvam@amazon.com>

* Add tags to the emr jobs based on the query types (#2150)

Signed-off-by: Vamsi Manohar <reddyvam@amazon.com>

---------

Signed-off-by: Vamsi Manohar <reddyvam@amazon.com>
(cherry picked from commit ae10857)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@codecov
Copy link

codecov bot commented Sep 27, 2023

Codecov Report

Merging #2163 (fb31d71) into 2.x (d49f0af) will decrease coverage by 0.71%.
The diff coverage is 82.70%.

@@             Coverage Diff              @@
##                2.x    #2163      +/-   ##
============================================
- Coverage     97.32%   96.61%   -0.71%     
- Complexity     4659     4754      +95     
============================================
  Files           411      437      +26     
  Lines         12109    12702     +593     
  Branches        827      861      +34     
============================================
+ Hits          11785    12272     +487     
- Misses          317      420     +103     
- Partials          7       10       +3     
Flag Coverage Δ
sql-engine 96.61% <82.70%> (-0.71%) ⬇️

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

Files Coverage Δ
...ch/sql/datasources/glue/GlueDataSourceFactory.java 100.00% <100.00%> (ø)
...ql/datasources/rest/RestDataSourceQueryAction.java 0.00% <ø> (ø)
...sql/datasources/service/DataSourceServiceImpl.java 100.00% <100.00%> (ø)
...l/datasources/utils/DatasourceValidationUtils.java 100.00% <100.00%> (ø)
...rch/sql/opensearch/setting/OpenSearchSettings.java 100.00% <100.00%> (ø)
...park/asyncquery/AsyncQueryExecutorServiceImpl.java 100.00% <100.00%> (ø)
...OpensearchAsyncQueryJobMetadataStorageService.java 100.00% <100.00%> (ø)
...cquery/exceptions/AsyncQueryNotFoundException.java 100.00% <100.00%> (ø)
...h/sql/spark/asyncquery/model/AsyncQueryResult.java 100.00% <100.00%> (ø)
.../asyncquery/model/S3GlueSparkSubmitParameters.java 100.00% <100.00%> (ø)
... and 24 more

@vamsi-amazon vamsi-amazon merged commit 84843e5 into 2.x Sep 27, 2023
30 of 34 checks passed
@github-actions github-actions bot deleted the backport/backport-2159-to-2.x branch September 27, 2023 21:09
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.

3 participants