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

Parse JSON XContent in the RestHelloAction #192

Merged
merged 3 commits into from
Oct 13, 2022

Conversation

dbwiddis
Copy link
Member

@dbwiddis dbwiddis commented Oct 13, 2022

Signed-off-by: Daniel Widdis [email protected]

Companion PR: opensearch-project/OpenSearch#4760

⚠️ Companion PR must be merged first; Jenkins gradle check will fail until this is done.
✅ Check passes locally.

Description

Demonstrates the use of the ExtensionRestRequest object's contentParser() method in the sample extension.

Issues Resolved

Fixes #178

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.

owaiskazi19
owaiskazi19 previously approved these changes Oct 13, 2022
Copy link
Member

@owaiskazi19 owaiskazi19 left a comment

Choose a reason for hiding this comment

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

Few lines for the handler now!

Signed-off-by: Daniel Widdis <[email protected]>
@joshpalis joshpalis self-requested a review October 13, 2022 17:38
@owaiskazi19 owaiskazi19 merged commit 9c58d4b into opensearch-project:main Oct 13, 2022
@dbwiddis dbwiddis deleted the parseContent branch November 16, 2022 17:03
kokibas pushed a commit to kokibas/opensearch-sdk-java that referenced this pull request Mar 17, 2023
* Parse JSON XContent in the RestHelloAction

Signed-off-by: Daniel Widdis <[email protected]>

* Autofix newline and whitespace issues with spotless

Signed-off-by: Daniel Widdis <[email protected]>

* Null check

Signed-off-by: Daniel Widdis <[email protected]>

Signed-off-by: Daniel Widdis <[email protected]>
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.

[FEATURE] Integrate XContent parsing in Hello World
3 participants