Skip to content

Correcting api name #24924

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
May 29, 2017
Merged

Correcting api name #24924

merged 1 commit into from
May 29, 2017

Conversation

@propulkit
Copy link
Contributor

@propulkit propulkit commented May 28, 2017

As per REST request signature for reroute, API has no underscore.

Thanks!

  • Have you signed the contributor license agreement?
  • Have you followed the contributor guidelines?
  • If submitting code, have you built your formula locally prior to submission with gradle check?
  • If submitting code, is your pull request against master? Unless there is a good reason otherwise, we prefer pull requests against master and will backport as needed.
  • If submitting code, have you checked that your submission is for an OS that we support?
  • If you are submitting this code for a class then read our policy for that.

TCorrecting api name
d4d096e 
As per REST request signature for reroute, API has no underscore.
@elasticmachine
Copy link
Collaborator

elasticmachine commented May 28, 2017

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

@clintongormley clintongormley added the >docs General docs changes label May 29, 2017
@clintongormley clintongormley merged commit 0d56652 into elastic:5.4 May 29, 2017
@clintongormley
Copy link
Contributor

clintongormley commented May 29, 2017

thanks @propulkit - merged

clintongormley pushed a commit that referenced this pull request May 29, 2017
@clintongormley
TCorrecting api name (#24924)
e8b1290 
As per REST request signature for reroute, API has no underscore.
clintongormley pushed a commit that referenced this pull request May 29, 2017
@clintongormley
TCorrecting api name (#24924)
2551686 
As per REST request signature for reroute, API has no underscore.
jasontedor added a commit to jasontedor/elasticsearch that referenced this pull request May 29, 2017
@jasontedor
Merge branch 'master' into primary-promotion-transition
4f5f53d 
* master:
  Add a second refresh to concurrent relocation test
  Add a dummy_index to upgrade tests to ensure we recover fine with replicas (elastic#24937)
  Rework bwc snapshot projects to build up to two bwc versions (elastic#24870)
  Move the IndexDeletionPolicy to be engine internal (elastic#24930)
  [Tests] Harden InternalExtendedStatsTests (elastic#24934)
  TCorrecting api name (elastic#24924)
  Add search method to high level REST client (elastic#24796)
  Add fromXContent method to ClearScrollResponse (elastic#24909)
  ClearScrollRequest to implement ToXContentObject (elastic#24907)
  SearchScrollRequest to implement ToXContentObject (elastic#24906)
  Fix bug in weight computation for query cache
@propulkit
Copy link
Contributor Author

propulkit commented May 30, 2017

@clintongormley Pleasure :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

>docs General docs changes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@propulkit @elasticmachine @clintongormley