forked from census-ecosystem/opencensus-java-jdbc
-
Notifications
You must be signed in to change notification settings - Fork 1
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
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
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
force-pushed
the
retrofit-for-publishing
branch
2 times, most recently
from
March 2, 2019 00:11
4ff1d67
to
14bbdd0
Compare
odeke-em
force-pushed
the
retrofit-for-publishing
branch
from
May 14, 2019 21:50
14bbdd0
to
a1258f4
Compare
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
force-pushed
the
retrofit-for-publishing
branch
from
September 3, 2019 21:27
a1258f4
to
67b4ec7
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.