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

Adding wrapper query, requirement for spring reactive #342

Closed
wants to merge 1 commit into from

Conversation

ls-urs-keller
Copy link
Contributor

Description

I'm trying to port spring reactive to opensearch for one of our projects, it uses wrapper queries.
This is missing from the opensearch-java library.

Issues Resolved

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.

@ls-urs-keller
Copy link
Contributor Author

ls-urs-keller commented Jan 22, 2023

If someone would be so kind to explain how to modify the "source" of the generated code I couldn't find it, or point me in the right direction. Thank you.

@reta
Copy link
Collaborator

reta commented Jan 22, 2023

@ls-urs-keller thanks for the fix, please add the test case for it (probably java-client/src/test/java/org/opensearch/client/opensearch/integTest/AbstractRequestIT.java is the best place right now), thank you

@VachaShah
Copy link
Collaborator

@ls-urs-keller Thank you for the changes! Want to finish this PR?

The code is not generated anymore and the related comments have been removed from the code. You can merge in the latest changes from the main branch.

@VachaShah
Copy link
Collaborator

Done via #630

@VachaShah VachaShah closed this Sep 22, 2023
@ls-urs-keller ls-urs-keller deleted the v2.1.0a branch September 25, 2023 07:21
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