googleapis / googleapis/google-cloud-java

[java-bigtable] Replace the usage of BaseApiTracer and BaseApiTracerFactory with ApiTracer and ApiTracerFactory

Open
#13,077 0 comments 0 reactions 0 assignees View on GitHub
api: bigtable
Dominant language
Java
Stars
2.1k
Forks
1.2k
Avg merge
1d 23h
Merged PRs (30d)
157

Description

As part of the OpenTelemetry metrics project, we made all methods in `ApiTracer` default in this [PR](https://github.com/googleapis/sdk-platform-java/pull/2403/files#diff-7c0438b5f6b77d8cf3f1bf3eb9c4550c8a47852bbf16c42a367797c48f3f584f). Since `BaseApiTracer` was originally introduced for Java 7 compatibility and we dropped Java 7 support now, we can replace the usage of `BaseApiTracer` and `BaseApiTracerFactory` with `ApiTracer` and `ApiTracerFactory`.
For example, make `BigtableTracer` implements `ApiTracer` instead of [extending from BaseApiTracer](https://github.com/googleapis/java-bigtable/blob/3f37d8da8b51a3fea56d1c462b9500ee2c244973/google-cloud-bigtable/src/main/java/com/google/cloud/bigtable/data/v2/stub/metrics/BigtableTracer.java#L29).

Contributor guide

Open the contributing guide

Research direction

Start with google-cloud-bigtable/src/main/java/com/google/cloud/bigtable/data/v2/stub/metrics/BigtableTracer.java and search the repository for usages of BaseApiTracer and BaseApiTracerFactory. The work is done when those usages are replaced with ApiTracer and ApiTracerFactory throughout the Java Bigtable code.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
api, observability-sre
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.