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

Create UploadGeoJSONRequest content as an object #32

Merged
merged 6 commits into from
Jan 31, 2022

Conversation

VijayanB
Copy link
Member

Signed-off-by: Vijayan Balasubramanian balasvij@amazon.com

Description

Create Request content object to deserialize into an object. This will be used by Transport action
to build logic to index request content as document into an index.

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.

Update request to deserialize content into object.

Signed-off-by: Vijayan Balasubramanian <balasvij@amazon.com>
@VijayanB VijayanB requested a review from a team January 28, 2022 02:19
@VijayanB VijayanB self-assigned this Jan 28, 2022
@codecov-commenter
Copy link

codecov-commenter commented Jan 28, 2022

Codecov Report

Merging #32 (4090a7f) into main (389ca01) will increase coverage by 1.70%.
The diff coverage is 91.17%.

Impacted file tree graph

@@             Coverage Diff              @@
##               main      #32      +/-   ##
============================================
+ Coverage     87.09%   88.80%   +1.70%     
- Complexity       33       44      +11     
============================================
  Files             9       10       +1     
  Lines            93      125      +32     
  Branches          8       12       +4     
============================================
+ Hits             81      111      +30     
- Misses           12       14       +2     
Impacted Files Coverage Δ
...al/action/upload/geojson/UploadGeoJSONRequest.java 100.00% <ø> (ø)
...n/upload/geojson/UploadGeoJSONTransportAction.java 0.00% <0.00%> (ø)
...va/org/opensearch/geospatial/GeospatialParser.java 94.73% <90.90%> (+5.84%) ⬆️
...on/upload/geojson/UploadGeoJSONRequestContent.java 100.00% <100.00%> (ø)

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 389ca01...4090a7f. Read the comment docs.

@VijayanB VijayanB changed the title Add request content model Create UploadGeoJSONRequest content as an object Jan 28, 2022
Signed-off-by: Vijayan Balasubramanian <balasvij@amazon.com>
@VijayanB VijayanB force-pushed the add-request-content branch 2 times, most recently from 79c1a6f to d6ac9af Compare January 28, 2022 19:36
Signed-off-by: Vijayan Balasubramanian <balasvij@amazon.com>
@VijayanB VijayanB force-pushed the add-request-content branch 2 times, most recently from 9229cc5 to d12e392 Compare January 28, 2022 21:48
Signed-off-by: Vijayan Balasubramanian <balasvij@amazon.com>
@VijayanB VijayanB force-pushed the add-request-content branch 2 times, most recently from ace41b3 to 782e7af Compare January 28, 2022 23:30
Copy link
Member

@martin-gaievski martin-gaievski left a comment

Choose a reason for hiding this comment

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

Lgtm, thank you

check whether input has valid index name, geospatial field
and GeoJSON data.

Signed-off-by: Vijayan Balasubramanian <balasvij@amazon.com>
Currently, there are two field types, geo_shape and geo_point.
Allow user to pass this information while creating index.

Signed-off-by: Vijayan Balasubramanian <balasvij@amazon.com>
@VijayanB VijayanB merged commit e8aa4fa into opensearch-project:main Jan 31, 2022
@VijayanB VijayanB deleted the add-request-content branch January 31, 2022 22:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants