Skip to content

Commit

Permalink
Remove OpenCensus tracing support for Java SDKs (Azure#7167)
Browse files Browse the repository at this point in the history
  • Loading branch information
samvaity authored Jan 10, 2020
1 parent 0213d7c commit dd6c261
Show file tree
Hide file tree
Showing 33 changed files with 13 additions and 2,086 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -137,8 +137,8 @@
<!-- Don't apply custom Checkstyle rules to files under checkstyle package. -->
<suppress checks="com\.azure\.tools\.checkstyle\.checks\..+" files=".*[/\\]tools[/\\]checkstyle[/\\].*"/>

<!-- Identity and OpenCensus tracing are plugin packages and shouldn't be referenced -->
<suppress checks="IllegalImport" files=".*[/\\]com[/\\]azure[/\\]core[/\\]tracing[/\\]opencensus[/\\]*"/>
<!-- Identity and opentelemetry tracing are plugin packages and shouldn't be referenced -->
<suppress checks="IllegalImport" files=".*[/\\]com[/\\]azure[/\\]core[/\\]tracing[/\\]opentelemetry[/\\]*"/>
<suppress checks="IllegalImport" files=".*[/\\]com[/\\]azure[/\\]identity[/\\]*"/>

<!-- Suppress warnings for Event Processor until the usage of "Client" is discussed and resolved: https://github.com/Azure/azure-sdk/issues/321 -->
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ page at http://checkstyle.sourceforge.net/config.html -->
<module name="AvoidStarImport"/>
<module name="IllegalImport">
<property name="regexp" value="true"/>
<property name="illegalPkgs" value="^(com\.)?sun\.\w*, ^io\.opencensus"/>
<property name="illegalPkgs" value="^(com\.)?sun\.\w*, ^io.opentelemetry"/>
</module>
<module name="RedundantImport"/>
<module name="UnusedImports"/>
Expand Down
4 changes: 2 additions & 2 deletions eng/jacoco-test-coverage/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -74,8 +74,8 @@
<!-- </dependency> -->
<dependency>
<groupId>com.azure</groupId>
<artifactId>azure-core-tracing-opencensus</artifactId>
<version>1.0.0-beta.7</version> <!-- {x-version-update;com.azure:azure-core-tracing-opencensus;current} -->
<artifactId>azure-core-tracing-opentelemetry</artifactId>
<version>1.0.0-beta.3</version> <!-- {x-version-update;com.azure:azure-core-tracing-opentelemetry;current} -->
</dependency>
<dependency>
<groupId>com.azure</groupId>
Expand Down
6 changes: 2 additions & 4 deletions eng/spotbugs-aggregate-report/pom.xml
Original file line number Diff line number Diff line change
@@ -1,8 +1,6 @@
<!-- Copyright (c) Microsoft Corporation. All rights reserved.
Licensed under the MIT License. -->
<project xmlns="http://maven.apache.org/POM/4.0.0"
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
<parent>
<artifactId>azure-client-sdk-parent</artifactId>
<groupId>com.azure</groupId>
Expand Down Expand Up @@ -151,4 +149,4 @@
</properties>
</profile>
</profiles>
</project>
</project>
2 changes: 0 additions & 2 deletions eng/versioning/external_dependencies.txt
Original file line number Diff line number Diff line change
Expand Up @@ -35,8 +35,6 @@ io.netty:netty-tcnative;2.0.26.Final
io.netty:netty-tcnative-boringssl-static;2.0.26.Final
io.netty:netty-transport-native-epoll;4.1.42.Final
io.netty:netty-transport-native-unix-common;4.1.42.Final
io.opencensus:opencensus-api;0.24.0
io.opencensus:opencensus-impl;0.24.0
io.projectreactor.netty:reactor-netty;0.9.0.RELEASE
io.projectreactor:reactor-core;3.3.0.RELEASE
io.reactivex:rxjava;1.2.4
Expand Down
1 change: 0 additions & 1 deletion eng/versioning/version_client.txt
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@ com.azure:azure-core-http-netty;1.2.0;1.3.0-beta.1
com.azure:azure-core-http-okhttp;1.1.0;1.2.0-beta.1
com.azure:azure-core-management;1.0.0-beta.8;1.0.0-beta.8
com.azure:azure-core-test;1.1.0;1.2.0-beta.1
com.azure:azure-core-tracing-opencensus;1.0.0-beta.6;1.0.0-beta.7
com.azure:azure-core-tracing-opentelemetry;1.0.0-beta.2;1.0.0-beta.3
com.azure:azure-data-appconfiguration;1.0.0-beta.7;1.0.1
com.azure:azure-e2e;1.0.0-beta.1;1.0.0-beta.1
Expand Down
3 changes: 1 addition & 2 deletions pom.client.xml
Original file line number Diff line number Diff line change
Expand Up @@ -503,7 +503,7 @@
</group>
<group>
<title>Azure Telemetry</title>
<title>Azure Telemetry - OpenCensus</title>
<title>Azure Telemetry - OpenTelemetry</title>
<packages>com.azure.core.tracing*</packages>
</group>
<group>
Expand Down Expand Up @@ -1263,7 +1263,6 @@
<module>sdk/core/azure-core-http-netty</module>
<module>sdk/core/azure-core-http-okhttp</module>
<module>sdk/core/azure-core-test</module>
<module>sdk/core/azure-core-tracing-opencensus</module>
<module>sdk/core/azure-core-tracing-opentelemetry</module>
<module>sdk/e2e</module>
<module>sdk/eventhubs/azure-messaging-eventhubs</module>
Expand Down
7 changes: 0 additions & 7 deletions pom.management.xml
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,6 @@
<netty.version>4.1.42.Final</netty.version>
<okhttp.version>4.2.2</okhttp.version>
<async-http-client.version>2.10.4</async-http-client.version>
<opencensus.version>0.20.0</opencensus.version>
<rxjava.version>1.2.4</rxjava.version>
<guava.version>24.1.1-jre</guava.version>
<azure-annotations.version>1.7.0</azure-annotations.version>
Expand Down Expand Up @@ -365,12 +364,6 @@
<version>${async-http-client.version}</version>
</dependency>

<dependency>
<groupId>io.opencensus</groupId>
<artifactId>opencensus-api</artifactId>
<version>${opencensus.version}</version>
</dependency>

<dependency>
<groupId>io.reactivex</groupId>
<artifactId>rxjava</artifactId>
Expand Down
1 change: 0 additions & 1 deletion samples/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,6 @@ This section lists how-to guides for the most commonly used APIs and most common
#### General How-To Guides
- [How to create **Netty Http Client**](https://github.com/Azure/azure-sdk-for-java/tree/master/sdk/core/azure-core-http-netty#examples)
- [How to create **OK Http Client**](https://github.com/Azure/azure-sdk-for-java/tree/master/sdk/core/azure-core-http-okhttp#examples)
- [How to use **Azure Core Tracing Opencensus**](https://github.com/Azure/azure-sdk-for-java/tree/master/sdk/core/azure-core-tracing-opencensus#examples)
- [How to use **Azure Core Tracing Opentelemetry**](https://github.com/Azure/azure-sdk-for-java/tree/master/sdk/core/azure-core-tracing-opentelemetry#examples)
- How to configure, access, and analyze **logging** information (TODO)

Expand Down
45 changes: 0 additions & 45 deletions sdk/core/azure-core-tracing-opencensus/CHANGELOG.md

This file was deleted.

153 changes: 0 additions & 153 deletions sdk/core/azure-core-tracing-opencensus/README.md

This file was deleted.

66 changes: 0 additions & 66 deletions sdk/core/azure-core-tracing-opencensus/pom.xml

This file was deleted.

Loading

0 comments on commit dd6c261

Please sign in to comment.