Skip to content

chore(bigquery): remove dev gate flag and beta api for tracing telemetry#12624

Open
ldetmer wants to merge 6 commits intomainfrom
remove-dev-gate
Open

chore(bigquery): remove dev gate flag and beta api for tracing telemetry#12624
ldetmer wants to merge 6 commits intomainfrom
remove-dev-gate

Conversation

@ldetmer
Copy link
Copy Markdown
Contributor

@ldetmer ldetmer commented Apr 2, 2026

Remove dev gate flag and BetaApi tags from big query open telemetry tracing, as we are feature complete.

Note by removing dev gate we will have added additional telemetry may contain sensitive data such as resource names, full URLs, and error messages.

Fixes #12100

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request removes the temporary development gate for HTTP tracing in the BigQuery RPC implementation by deleting the isHttpTracingEnabled() method and its associated system property. As a result, HTTP tracing attributes are now consistently applied whenever OpenTelemetry tracing is enabled. The test suite has been updated to remove system property manipulations and the specific test scenarios for the disabled gate. I have no feedback to provide.

@ldetmer ldetmer added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Apr 2, 2026
@ldetmer ldetmer changed the title chore(bigquery): remove dev gate flag for tracing telemetry chore(bigquery): remove dev gate flag and beta api for tracing telemetry Apr 2, 2026
@ldetmer ldetmer marked this pull request as ready for review April 8, 2026 20:42
@ldetmer ldetmer requested review from a team as code owners April 8, 2026 20:42
*/
@BetaApi
@VisibleForTesting
public class ErrorTypeUtil {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we make this package private? I think we plan to remove it and use the gax one later. However, removing a non-beta public class would be a breaking change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not merge Indicates a pull request not ready for merge, due to either quality or timing.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

observability cleanup: remove HTTP_TRACING_DEV_GATE_PROPERTY

2 participants