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

Add elasticsearch instrumentation #859

Merged

Conversation

SimunKaracic
Copy link
Contributor

@SimunKaracic SimunKaracic commented Sep 22, 2020

Add elasticsearch instrumentation for the official java High and Low level REST client (current version, 7.9).

Simple implementation, records a span for each sync and async request with 2 tags (endpoint, method),
and creates histogram of request sizes.

@SimunKaracic SimunKaracic force-pushed the add-elasticsearch-instrumentation branch from 39234d6 to 3674063 Compare September 25, 2020 07:50
@SimunKaracic SimunKaracic changed the title [WIP] Add elasticsearch instrumentation Add elasticsearch instrumentation Sep 25, 2020
@SimunKaracic SimunKaracic force-pushed the add-elasticsearch-instrumentation branch 4 times, most recently from d86e2ee to 07041ac Compare September 25, 2020 08:43
@SimunKaracic SimunKaracic marked this pull request as ready for review September 25, 2020 11:34
@SimunKaracic SimunKaracic force-pushed the add-elasticsearch-instrumentation branch from 1200bdf to 2da6d1b Compare September 25, 2020 11:36
@SimunKaracic
Copy link
Contributor Author

SimunKaracic commented Sep 25, 2020

kamon-mongo tests failing.
I'm gonna go out on a limb and say that that has nothing to do with this PR :)
They pass locally 💻

build.sbt Outdated Show resolved Hide resolved
@SimunKaracic SimunKaracic force-pushed the add-elasticsearch-instrumentation branch from 967cc41 to 501c355 Compare October 12, 2020 09:24
@SimunKaracic SimunKaracic force-pushed the add-elasticsearch-instrumentation branch from 501c355 to ebef1f2 Compare October 12, 2020 10:00
@SimunKaracic SimunKaracic merged commit e085a5f into kamon-io:master Oct 12, 2020
@SimunKaracic SimunKaracic deleted the add-elasticsearch-instrumentation branch October 12, 2020 10:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants