Skip to content

Commit 6635352

Browse files
committed
Address review comments
1 parent 74050b6 commit 6635352

File tree

3 files changed

+31
-10
lines changed

3 files changed

+31
-10
lines changed

spanner/cloud-client/README.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,5 +31,11 @@ You can then run a given `ClassName` via:
3131
### Running the tutorial
3232
mvn exec:java -Dexec.mainClass=com.example.spanner.SpannerSample -Dexec.args="<command> my-instance my-database"
3333

34+
## Tracing sample
35+
TracingSample.java demonstrates how to export traces generated by client library to StackDriver and to /tracez page.
36+
37+
### Running the tracing sample
38+
mvn exec:java -Dexec.mainClass=com.example.spanner.TracingSample -Dexec.args="my-instance my-database"
39+
3440
## Test
3541
mvn verify -Dspanner.test.instance=<instance id> -Dspanner.sample.database=<new database id> -Dspanner.quickstart.database=<existing database id>

spanner/cloud-client/pom.xml

Lines changed: 14 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,7 @@ limitations under the License.
4646
<maven.compiler.target>1.8</maven.compiler.target>
4747
<maven.compiler.source>1.8</maven.compiler.source>
4848
<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
49+
<opencensus.version>0.11.0</opencensus.version>
4950
</properties>
5051

5152
<dependencies>
@@ -73,27 +74,33 @@ limitations under the License.
7374
<dependency>
7475
<groupId>io.opencensus</groupId>
7576
<artifactId>opencensus-api</artifactId>
76-
<version>0.11.0</version>
77+
<version>${opencensus.version}</version>
7778
</dependency>
7879
<dependency>
7980
<groupId>io.opencensus</groupId>
8081
<artifactId>opencensus-impl</artifactId>
81-
<version>0.11.0</version>
82+
<version>${opencensus.version}</version>
83+
<scope>runtime</scope>
8284
</dependency>
8385
<dependency>
8486
<groupId>io.opencensus</groupId>
8587
<artifactId>opencensus-contrib-zpages</artifactId>
86-
<version>0.11.0</version>
88+
<version>${opencensus.version}</version>
8789
</dependency>
8890
<dependency>
8991
<groupId>io.opencensus</groupId>
90-
<artifactId>opencensus-exporter-trace-logging</artifactId>
91-
<version>0.11.0</version>
92+
<artifactId>opencensus-exporter-trace-stackdriver</artifactId>
93+
<version>${opencensus.version}</version>
9294
</dependency>
9395
<dependency>
9496
<groupId>io.opencensus</groupId>
95-
<artifactId>opencensus-exporter-trace-stackdriver</artifactId>
96-
<version>0.11.0</version>
97+
<artifactId>opencensus-exporter-stats-stackdriver</artifactId>
98+
<version>${opencensus.version}</version>
99+
</dependency>
100+
<dependency>
101+
<groupId>io.opencensus</groupId>
102+
<artifactId>opencensus-contrib-grpc-metrics</artifactId>
103+
<version>${opencensus.version}</version>
97104
</dependency>
98105

99106
<!-- Test dependencies -->

spanner/cloud-client/src/main/java/com/example/spanner/TracingSample.java

Lines changed: 11 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,12 +10,16 @@
1010
import com.google.cloud.spanner.Statement;
1111

1212
import io.opencensus.common.Scope;
13+
import io.opencensus.contrib.grpc.metrics.RpcViews;
1314
import io.opencensus.contrib.zpages.ZPageHandlers;
15+
import io.opencensus.exporter.stats.stackdriver.StackdriverStatsExporter;
1416
import io.opencensus.exporter.trace.stackdriver.StackdriverExporter;
1517
import io.opencensus.trace.Tracing;
1618
import io.opencensus.trace.samplers.Samplers;
1719

1820
public class TracingSample {
21+
22+
private static final String SAMPLE_SPAN = "CloudSpannerSample";
1923

2024
public static void main(String[] args) throws Exception {
2125
if (args.length != 2) {
@@ -27,10 +31,14 @@ public static void main(String[] args) throws Exception {
2731

2832
// Installs a handler for /tracez page.
2933
ZPageHandlers.startHttpServerAndRegisterAll(8080);
30-
// Installs an exporter for stack driver.
34+
// Installs an exporter for stack driver traces.
3135
StackdriverExporter.createAndRegister();
32-
Tracing.getExportComponent().getSampledSpanStore().registerSpanNamesForCollection(Arrays.asList("CloudSpannerSample"));
36+
Tracing.getExportComponent().getSampledSpanStore().registerSpanNamesForCollection(Arrays.asList(SAMPLE_SPAN));
3337

38+
// Installs an exporter for stack driver stats.
39+
StackdriverStatsExporter.createAndRegister();
40+
RpcViews.registerAllCumulativeViews();
41+
3442
// Name of your instance & database.
3543
String instanceId = args[0];
3644
String databaseId = args[1];
@@ -40,7 +48,7 @@ public static void main(String[] args) throws Exception {
4048
options.getProjectId(), instanceId, databaseId));
4149
// Queries the database
4250
try (Scope ss = Tracing.getTracer()
43-
.spanBuilderWithExplicitParent("CloudSpannerSample", null)
51+
.spanBuilderWithExplicitParent(SAMPLE_SPAN, null)
4452
.setSampler(Samplers.alwaysSample())
4553
.startScopedSpan()) {
4654
ResultSet resultSet = dbClient.singleUse().executeQuery(Statement.of("SELECT 1"));

0 commit comments

Comments
 (0)