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

fix thread_pool_patterns path variable definition #21332

Merged
merged 1 commit into from Nov 4, 2016

Conversation

Projects
None yet
4 participants
@apatrida
Copy link
Contributor

commented Nov 4, 2016

The api spec was not correct, the variable name didn't match and the variable definition was in the query parameters instead of the path parameters

@s1monw

This comment has been minimized.

Copy link
Contributor

commented Nov 4, 2016

test this please

@jasontedor

This comment has been minimized.

Copy link
Member

commented Nov 4, 2016

This looks right, truly sorry about this. I will integrate assuming the CI build passes. Thank you for fixing.

@jasontedor

This comment has been minimized.

Copy link
Member

commented Nov 4, 2016

Relates #19721

@jasontedor jasontedor merged commit a978f13 into elastic:master Nov 4, 2016

2 checks passed

CLA Commit author has signed the CLA
Details
elasticsearch-ci Build finished.
Details

jasontedor added a commit that referenced this pull request Nov 4, 2016

Fix cat thread pool API spec
This commit fixes an issue with the cat thread pool API spec. Namely,
the name of the variable for the thread pool patterns parameter was
misnamed in the spec.

Relates #21332

jasontedor added a commit that referenced this pull request Nov 4, 2016

Fix cat thread pool API spec
This commit fixes an issue with the cat thread pool API spec. Namely,
the name of the variable for the thread pool patterns parameter was
misnamed in the spec.

Relates #21332
@jasontedor

This comment has been minimized.

Copy link
Member

commented Nov 4, 2016

Thanks @apatrida.

@clintongormley clintongormley added the >bug label Nov 13, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.