-
Notifications
You must be signed in to change notification settings - Fork 107
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
Fixing paths for some APIs #134
Conversation
Signed-off-by: Vacha Shah <vachshah@amazon.com>
I am new to this codebase and don't quite understand what's going on :) Looks like the PR swaps the order of |
Thanks. I suppose we don't have integration tests for this? I'd expect some to fail against OpenSearch 2.x. Open an issue at least? |
Yes currently these APIs are not part of the integration test. I have created #135 to add this issue. |
Signed-off-by: Vacha Shah <vachshah@amazon.com> (cherry picked from commit abf672c)
Signed-off-by: Vacha Shah vachshah@amazon.com
Description
Fixing url paths for some APIs after type removal.
Issues Resolved
Closes #132
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.