-
Notifications
You must be signed in to change notification settings - Fork 138
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
REST API for GET,PUT and DELETE #1482
REST API for GET,PUT and DELETE #1482
Conversation
6dca518
to
bba0bc7
Compare
Codecov Report
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more @@ Coverage Diff @@
## main #1482 +/- ##
=========================================
Coverage 98.46% 98.47%
- Complexity 3869 3886 +17
=========================================
Files 345 347 +2
Lines 9603 9649 +46
Branches 616 625 +9
=========================================
+ Hits 9456 9502 +46
Misses 142 142
Partials 5 5
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
60669a2
to
e8a7270
Compare
core/src/main/java/org/opensearch/sql/datasource/DataSourceService.java
Outdated
Show resolved
Hide resolved
integ-test/src/test/java/org/opensearch/sql/datasource/DataSourceAPIsIT.java
Outdated
Show resolved
Hide resolved
opensearch/src/main/java/org/opensearch/sql/opensearch/storage/OpenSearchDataSourceFactory.java
Outdated
Show resolved
Hide resolved
plugin/src/main/java/org/opensearch/sql/plugin/rest/RestDataSourceQueryAction.java
Show resolved
Hide resolved
core/src/main/java/org/opensearch/sql/datasource/DataSourceServiceImpl.java
Outdated
Show resolved
Hide resolved
core/src/main/java/org/opensearch/sql/datasource/DataSourceServiceImpl.java
Outdated
Show resolved
Hide resolved
core/src/main/java/org/opensearch/sql/datasource/DataSourceServiceImpl.java
Outdated
Show resolved
Hide resolved
core/src/main/java/org/opensearch/sql/datasource/DataSourceServiceImpl.java
Show resolved
Hide resolved
core/src/main/java/org/opensearch/sql/datasource/DataSourceServiceImpl.java
Outdated
Show resolved
Hide resolved
core/src/main/java/org/opensearch/sql/datasource/DataSourceServiceImpl.java
Outdated
Show resolved
Hide resolved
core/src/main/java/org/opensearch/sql/datasource/DataSourceServiceImpl.java
Outdated
Show resolved
Hide resolved
core/src/main/java/org/opensearch/sql/storage/DataSourceFactory.java
Outdated
Show resolved
Hide resolved
core/src/test/java/org/opensearch/sql/analysis/AnalyzerTestBase.java
Outdated
Show resolved
Hide resolved
core/src/main/java/org/opensearch/sql/datasource/DataSourceService.java
Outdated
Show resolved
Hide resolved
.../src/main/java/org/opensearch/sql/plugin/datasource/OpenSearchDataSourceMetadataStorage.java
Show resolved
Hide resolved
.../src/main/java/org/opensearch/sql/plugin/datasource/OpenSearchDataSourceMetadataStorage.java
Show resolved
Hide resolved
.../src/main/java/org/opensearch/sql/plugin/datasource/OpenSearchDataSourceMetadataStorage.java
Show resolved
Hide resolved
e73bab8
to
1dd1ee9
Compare
core/src/main/java/org/opensearch/sql/datasource/DataSourceServiceImpl.java
Outdated
Show resolved
Hide resolved
core/src/main/java/org/opensearch/sql/datasource/DataSourceServiceImpl.java
Show resolved
Hide resolved
core/src/main/java/org/opensearch/sql/datasource/DataSourceServiceImpl.java
Show resolved
Hide resolved
core/src/main/java/org/opensearch/sql/datasource/DataSourceServiceImpl.java
Outdated
Show resolved
Hide resolved
core/src/main/java/org/opensearch/sql/datasource/DataSourceServiceImpl.java
Outdated
Show resolved
Hide resolved
9e85e8b
to
2069e97
Compare
core/src/main/java/org/opensearch/sql/datasource/DataSourceServiceImpl.java
Outdated
Show resolved
Hide resolved
2069e97
to
e0dd50c
Compare
ed3c0a9
to
62a9820
Compare
opensearch/src/main/java/org/opensearch/sql/opensearch/storage/OpenSearchDataSourceFactory.java
Outdated
Show resolved
Hide resolved
...rch/src/test/java/org/opensearch/sql/opensearch/storage/OpenSearchDataSourceFactoryTest.java
Outdated
Show resolved
Hide resolved
Signed-off-by: vamsi-amazon <reddyvam@amazon.com>
62a9820
to
c4335af
Compare
Signed-off-by: vamsi-amazon <reddyvam@amazon.com>
* REST API for GET,PUT and DELETE Signed-off-by: vamsi-amazon <reddyvam@amazon.com> * changed permission action names Signed-off-by: vamsi-amazon <reddyvam@amazon.com> --------- Signed-off-by: vamsi-amazon <reddyvam@amazon.com> (cherry picked from commit 3fbbaa4)
* REST API for GET,PUT and DELETE Signed-off-by: vamsi-amazon <reddyvam@amazon.com> * changed permission action names Signed-off-by: vamsi-amazon <reddyvam@amazon.com> --------- Signed-off-by: vamsi-amazon <reddyvam@amazon.com> (cherry picked from commit 3fbbaa4)
* REST API for GET,PUT and DELETE Signed-off-by: vamsi-amazon <reddyvam@amazon.com> * changed permission action names Signed-off-by: vamsi-amazon <reddyvam@amazon.com> --------- Signed-off-by: vamsi-amazon <reddyvam@amazon.com>
Description
These changes introduce REST APIs for GET DATASOURCE, UPDATE DATASOURCE and DELETE DATASOURCE APIS as follow up to CREATE DATASOURCE APIs #1458
Issues Resolved
Rough LLD Diagrams
Modification of datasource is similar to create apis. Below images corresponds to GET ALL and DELETE APIs.
Check List
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.