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

all: retrofit under io.orijtech.integrations.ocjdbc #1

Merged
merged 4 commits into from
Oct 17, 2019

Conversation

odeke-em
Copy link
Member

Retrofit this package under
"io.orijtech.integrations.ocjdbc"
instead of
"io.opencensus.integration.jdbc"

because publishing changes under the official OpenCensus
artifact group is a source of contention due to the OpenCensus Java
team not wanting to have to maintain integrations for life but also
the process of publishing is highly bureaucratic.
Also we've waited for more than 3 months for a single release which
has stalled usage, promotion, feedback and marketing of this integration
as per:

This package to be published is maintained under good faith for the entire OpenCensus
community and the licenses all stand; just that the artifact is now
being published under "io.orijtech.integrations.ocjdbc" which guts out
on the previously mentioned bureaucracy, indefinite stalls in publishing
and puts this integration into the hands of users.

odeke-em added a commit to census-instrumentation/opencensus-website that referenced this pull request Jan 11, 2019
Update the JDBC integration path from
   "io.opencensus.integration.ocjdbc"
to
   "io.orijtech.integrations.ocjdbc"

because the old publishing was a little bureaucratic
and stalled for more than 3 months; publishing it
under "io.orijtech.integrations.ocjdbc" is just a flexible
way for maintaining the integration but also for putting
it into the hands of users without quarter long stalls.

Fixes #527
Retrofit by orijtech/opencensus-java-jdbc#1
@odeke-em odeke-em force-pushed the retrofit-for-publishing branch 2 times, most recently from 4ff1d67 to 14bbdd0 Compare March 2, 2019 00:11
odeke-em added a commit to orijtech/opencensus-website that referenced this pull request May 14, 2019
Following suit with:
* census-ecosystem/opencensus-java-jdbc#59
* orijtech/opencensus-java-jdbc#1
initiated by @dmichel1

Upgrade to updated ocjdbc version v0.0.4 which
also uses OpenCensus-Java v0.21.0 having taken
in a bug fix in OpenCensus v0.18.0 that reduced
high CPU usage for low QPS services.
Retrofit this package under
    "io.orijtech.integrations.ocjdbc"
instead of
    "io.opencensus.integration.jdbc"

because publishing changes under the official OpenCensus
artifact group is a source of contention due to the OpenCensus Java
team not wanting to have to maintain integrations for life but also
the process of publishing is highly bureaucratic.
Also we've waited for more than 3 months for a single release which
has stalled usage, promotion, feedback and marketing of this integration
as per:
* census-ecosystem#32
* census-ecosystem#56
* census-instrumentation/opencensus-website#527

This package to be published is maintained under good faith for the entire OpenCensus
community and the licenses all stand; just that the artifact is now
being published under "io.orijtech.integrations.ocjdbc" which guts out
on the previously mentioned bureaucracy, indefinite stalls in publishing
and puts this integration into the hands of users.
Adaptation of @dmichel1's PR
census-ecosystem#59

which upgraded to OpenCensus v0.21.0 so as to use a bug fix from v0.18.0
for high CPU usage for low QPS services.

This commit also takes into consideration new updates to v0.21.0
that require a TagMap TTL.
@odeke-em odeke-em merged commit e88d15d into master Oct 17, 2019
@odeke-em odeke-em deleted the retrofit-for-publishing branch October 17, 2019 01:21
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.

1 participant