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 logRequestFail bad method call #149

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
3 participants
@jdohuutin

jdohuutin commented Nov 4, 2014

add missing duration parameter

@mguillermin

This comment has been minimized.

Show comment
Hide comment
@mguillermin

mguillermin commented Nov 4, 2014

👍

@polyfractal

This comment has been minimized.

Show comment
Hide comment
@polyfractal

polyfractal Nov 4, 2014

Member

LGTM, thanks for the PR! @jdohuutin would you mind signing the Elasticsearch CLA so that I can merge this in?

Member

polyfractal commented Nov 4, 2014

LGTM, thanks for the PR! @jdohuutin would you mind signing the Elasticsearch CLA so that I can merge this in?

@jdohuutin

This comment has been minimized.

Show comment
Hide comment
@jdohuutin

jdohuutin Nov 4, 2014

@polyfractal done, I just sign Elasticsearch CLA

jdohuutin commented Nov 4, 2014

@polyfractal done, I just sign Elasticsearch CLA

@polyfractal

This comment has been minimized.

Show comment
Hide comment
@polyfractal

polyfractal Nov 6, 2014

Member

Merged, thanks!

Member

polyfractal commented Nov 6, 2014

Merged, thanks!

@polyfractal polyfractal closed this Nov 6, 2014

@jdohuutin jdohuutin deleted the jdohuutin:fix-log-request-fail-call branch Nov 7, 2014

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