opentracing-contrib / meta

A meta-repository for OpenTracing contributions
Apache License 2.0
35 stars 16 forks source link

p6spy instrumentation #31

Closed hypnoce closed 6 years ago

hypnoce commented 6 years ago

In opentracing contrib there is already a jdbc instrumentation. I think there is a place for p6spy since people may already have an environment setup with this jdbc proxy. Here is the main repo : https://github.com/hypnoce/opentracing-p6spy Let me know if you have any comments.

I'm missing Bintray and Sonatype access as described here : https://github.com/opentracing-contrib/meta/blob/master/guide_java.md

Thanks for your help !

tedsuo commented 6 years ago

@malafeev Can you review?

This looks useful to me. I'm wondering if there should be an expectation that this produces the same data as https://github.com/opentracing-contrib/java-jdbc, or if there is anything interesting about how they are different.

@bhs can you add bintray/sonatype for this?

malafeev commented 6 years ago

java-jdbc and p6spy instrumentation should produce similar data. I took idea from p6spy when implemented java-jdbc. In case you are using some specific p6spy features it could be useful.

hypnoce commented 6 years ago

@malafeev let me know if you need any change on my side regarding span decoration @bhs my Bintray user is hypnoce

hypnoce commented 6 years ago

Hi all any news ? Maybe you are overloaded :/.

malafeev commented 6 years ago

I'm OK to add it to contrib

hypnoce commented 6 years ago

@malafeev I added the same traceWithActiveSpanOnly option for feature consistency. Let me know if you are ok with this.

malafeev commented 6 years ago

great @hypnoce

malafeev commented 6 years ago

@hypnoce I created https://github.com/opentracing-contrib/java-p6spy but I don't have permissions to grant you access to bintray/sonatype @bhs did it for me.

bhs commented 6 years ago

@hypnoce you should have a bintray invite. Sorry for the delay! I have a crazy inbox :-/

hypnoce commented 6 years ago

@bhs no worries I can understand :)

hypnoce commented 6 years ago

Hi all,

just committed the first implementation : https://github.com/opentracing-contrib/java-p6spy

Thanks very much for your time !

hypnoce commented 6 years ago

version 0.0.1 released : http://search.maven.org/#search%7Cga%7C1%7Copentracing-p6spy Thanks all for your help !

bhs commented 6 years ago

@hypnoce excellent, thank you!