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

Backport high level REST client #25086

Merged
merged 1 commit into from
Jun 7, 2017

Conversation

javanna
Copy link
Member

@javanna javanna commented Jun 6, 2017

This PR backports the high level REST client supports the following apis: ping, info, index, bulk, get, delete, update, search, search scroll and clear scroll. Also, the BulkProcessor has been updated so that it can be used with the high level client as well.

Here is a list of PRs that have already been pushed to master and are part of this PR that is targeted to 5.x only:

The high level client supports the following apis: ping, info, index, bulk, get, delete, update, search, search scroll and clear scroll. Also, the BulkProcessor has been updated so that it can be used with the high level client as well.
Copy link
Member

@tlrx tlrx left a comment

Choose a reason for hiding this comment

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

I skimmed through the changes and it's LGTM

@cbuescher
Copy link
Member

I gave this a quick look also, it's a bit large to give any detailed review but I looked at the changes in core and they only seem to be internal to BulkProcessor and tests. So LGTM too.

@javanna javanna merged commit 55f968c into elastic:5.x Jun 7, 2017
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.

3 participants