Skip to content
This repository has been archived by the owner on Oct 3, 2023. It is now read-only.

guides/integrations/sql/java: dependency errors from unpublished version of ocjdbc #527

Closed
PikkaPikkachu opened this issue Jan 11, 2019 · 2 comments
Closed

guides/integrations/sql/java: dependency errors from unpublished version of ocjdbc #527

PikkaPikkachu opened this issue Jan 11, 2019 · 2 comments
Assignees
@odeke-em

Comments

@PikkaPikkachu
Copy link

The end-to-end example fails to compile, due to unresolved dependencies in pom.xml, specifically:

<dependency>
            <groupId>io.opencensus.integration</groupId>
            <artifactId>opencensus-ocjdbc</artifactId>
            <version>0.0.2</version>
</dependency>

https://mvnrepository.com/artifact/io.opencensus/ocjdbc is still on the older version of the ocjdbc library.
I see census-ecosystem/opencensus-java-jdbc#32 is on hold for the next release.
@odeke-em
@odeke-em
Copy link
Member

Thank you for this report @PikkaPikkachu!

In deed, unfortunately that code release has been stalled for a couple of months /cc @bogdandrutu @songy23 @shahprit. Before the OpenCensus-Java team was concerned about maintaining the JDBC-SQL integration for life but the consensus was that this is an important integration that is core to many Java programs.

However, since this has been stalled, I'll publish it under my group as "io.orijtech.integrations.ocjdbc" like I've done for the Jedis and SpyMemcached integrations. Please give me about 30 minutes or less.

odeke-em added a commit to orijtech/opencensus-java-jdbc that referenced this issue Jan 11, 2019
@odeke-em
all: retrofit under io.orijtech.integrations.ocjdbc
427fef6 
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.
@odeke-em odeke-em mentioned this issue Jan 11, 2019
Merged
@odeke-em odeke-em self-assigned this Jan 11, 2019
odeke-em added a commit that referenced this issue Jan 11, 2019
@odeke-em
integrations/sql: update JDBC import path
9501d75 
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
odeke-em added a commit that referenced this issue Jan 11, 2019
@odeke-em
integrations/sql: update JDBC import path
cd19969 
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 mentioned this issue Jan 11, 2019
Merged
@odeke-em
Copy link
Member

I've mailed out #530 which fixes this issue. The publishing was done a few minutes ago so after merge into the website, please refresh an hour or two later when Maven Central updates.

@odeke-em odeke-em changed the title Guides/Integrations/SQL: Dependency errors guides/integrations/sql/java: dependency errors from unpublished version of ocjdbc Jan 11, 2019
odeke-em added a commit that referenced this issue Jan 12, 2019
@odeke-em
integrations/sql: retouch ocjdbc to ensure working end-to-end
66c484a 
Retouch the OCJDBC integration guide now that it was successfully
published on Sonatype. This retouch ensures that the end-to-end
example fully compiles. The last edits of the actual integration
introduced an unexpected compulsory option which rendered the
previous guide uncompilable.

Updates #527
odeke-em added a commit that referenced this issue Jan 12, 2019
@odeke-em
integrations/sql: retouch ocjdbc to ensure working end-to-end
6a58f7c 
Retouch the OCJDBC integration guide now that it was successfully
published on Sonatype. This retouch ensures that the end-to-end
example fully compiles. The last edits of the actual integration
introduced an unexpected compulsory option which rendered the
previous guide uncompilable.

This change also involved running `./gradlew goJF` to format
the Java source code.

Updates #527
@odeke-em odeke-em mentioned this issue Jan 12, 2019
Merged
odeke-em added a commit that referenced this issue Jan 12, 2019
@odeke-em
integrations/sql: retouch ocjdbc to ensure working end-to-end
7f20f1a 
Retouch the OCJDBC integration guide now that it was successfully
published on Sonatype. This retouch ensures that the end-to-end
example fully compiles. The last edits of the actual integration
introduced an unexpected compulsory option which rendered the
previous guide uncompilable.

This change also involved running `./gradlew goJF` to format
the Java source code.

Updates #527
odeke-em added a commit to orijtech/opencensus-java-jdbc that referenced this issue Mar 1, 2019
@odeke-em
all: retrofit under io.orijtech.integrations.ocjdbc
4ff1d67 
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.
odeke-em added a commit to orijtech/opencensus-java-jdbc that referenced this issue Mar 2, 2019
@odeke-em
all: retrofit under io.orijtech.integrations.ocjdbc
14bbdd0 
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.
odeke-em added a commit to orijtech/opencensus-java-jdbc that referenced this issue May 14, 2019
@odeke-em
all: retrofit under io.orijtech.integrations.ocjdbc
e1860dd 
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.
odeke-em added a commit to orijtech/opencensus-java-jdbc that referenced this issue Sep 3, 2019
@odeke-em
all: retrofit under io.orijtech.integrations.ocjdbc
ec584d1 
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.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@odeke-em odeke-em

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@odeke-em @PikkaPikkachu