-
Notifications
You must be signed in to change notification settings - Fork 191
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
Update co.elastic to org.opensearch #88
Conversation
* Move folder co.elastic to org.opensearch * Modify co.elastic to org.opensearch * Organize imports * Fix LineLength errors Signed-off-by: Rishab Nahata <rnnahata@amazon.com>
Signed-off-by: Rishab Nahata <rnnahata@amazon.com>
…ch.clients.opensearch Signed-off-by: Rishab Nahata <rnnahata@amazon.com>
…earch-project#8) Unit Tests fix Signed-off-by: Mital Awachat <mitalawachat@users.noreply.github.com>
Signed-off-by: Rishab Nahata <rnnahata@amazon.com>
Co-authored-by: Mital Awachat <awachatm@amazon.com> Signed-off-by: Mital Awachat <mitalawachat@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thanks
@imRishN thanks for this change, I see 99% of the files in this pull request start with
I am wondering, did you regenerate these files or changed them manually? Thank you. |
I changed them manually. The above snippet has come from elasticsearch-java repo which will be removed in subsequent PRs. Thanks. |
Signed-off-by: Rishab Nahata <rnnahata@amazon.com>
Description
This PR updates namespace co.elastic to org.opensearch and fixes checkstyle issues.
Issues Resolved
#12
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.