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

cleanup request parsing in RestSearchAction #17156

Merged
merged 1 commit into from
Mar 16, 2016

Conversation

areek
Copy link
Contributor

@areek areek commented Mar 16, 2016

Simplifies _search request parsing logic.

@nik9000
Copy link
Member

nik9000 commented Mar 16, 2016

Looks right to me.
On Mar 16, 2016 7:16 PM, "Areek Zillur" notifications@github.com wrote:

Simplifies _search request parsing logic.

You can view, comment on, or merge this pull request online at:

#17156
Commit Summary

  • cleanup request parsing in RestSearchAction

File Changes

Patch Links:


You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
#17156

@areek areek merged commit 22e12ab into elastic:master Mar 16, 2016
@areek
Copy link
Contributor Author

areek commented Mar 16, 2016

Thanks @nik9000 for the review :)

@clintongormley clintongormley added :Search/Search Search-related issues that do not fall into other categories and removed :Search Refactoring labels Feb 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>non-issue :Search/Search Search-related issues that do not fall into other categories v5.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants