search

census-ecosystem / opencensus-java-jdbc

OpenCensus instrumented JDBC wrapper for tracing and metrics
Apache License 2.0
7 stars 7 forks source link

Update the group id of this artifact. #16

Closed songy23 closed 6 years ago

songy23 commented 6 years ago

Instead of using "io.opencensus", consider "io.opencensus.contrib" or "io.opencensus.integrations".

/cc @bogdandrutu

songy23 commented 6 years ago

Since we're planning to do a v0.0.2 release soon, I think it's a good time to update the group id now.

odeke-em commented 6 years ago

I think io.opencensus.integrations would be a match since io.opencensus.contrib is where grpc, log-correlation, zpages which are core to the Java OpenCensus project are put.

bogdandrutu commented 6 years ago

I agree with io.opencensus.integrations or io.opencensus.integration (no plural). Also you may change the artifact to be opencensus-ocjdbc to be consistent with all opencensus artifacts.

songy23 commented 6 years ago

io.opencensus.integration sounds good - updated in #17.