chore(deps): bump opentelemetry_sdk from 0.28.0 to 0.31.0 in /engine#6
Closed
dependabot[bot] wants to merge 1 commit intomainfrom
Closed
chore(deps): bump opentelemetry_sdk from 0.28.0 to 0.31.0 in /engine#6dependabot[bot] wants to merge 1 commit intomainfrom
dependabot[bot] wants to merge 1 commit intomainfrom
Conversation
Bumps [opentelemetry_sdk](https://github.com/open-telemetry/opentelemetry-rust) from 0.28.0 to 0.31.0. - [Release notes](https://github.com/open-telemetry/opentelemetry-rust/releases) - [Changelog](https://github.com/open-telemetry/opentelemetry-rust/blob/main/docs/release_0.30.md) - [Commits](open-telemetry/opentelemetry-rust@opentelemetry_sdk-0.28.0...v0.31.0) --- updated-dependencies: - dependency-name: opentelemetry_sdk dependency-version: 0.31.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]>
9277f74 to
83740ff
Compare
Contributor
|
Superseded by #16 |
Contributor
Author
|
OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, let me know by commenting If you change your mind, just re-open this PR and I'll resolve any conflicts on it. |
hugocorreia90
added a commit
that referenced
this pull request
May 1, 2026
… for totalBytesBilled (#330) `stats_from_response` already prefers `statistics.query.totalBytesBilled` over the top-level `totalBytesProcessed` fallback (PR #326), but the synchronous `jobs.query` and `jobs.getQueryResults` endpoints don't include the `statistics` block at all — so every sync query was falling back to processed bytes, which doesn't apply BigQuery's 10 MB per-query minimum-bill floor. This PR adds `BigQueryAdapter::fetch_job_statistics`, an async helper that calls `GET /projects/<p>/jobs/<id>?location=<loc>` for a job ID returned by `run_query`. The full Job resource includes the `statistics` block. `execute_statement_with_stats` now enriches the response with that block before passing it to `stats_from_response`, so cost reporting reflects what the BigQuery console actually charges the user. One extra HTTP roundtrip per query — `jobs.get` is free and returns in tens of milliseconds for a fresh job. Best-effort: if `jobs.get` fails for any reason (transient API error, missing job reference), the path falls back to the existing `totalBytesProcessed` parsing with a `debug!` log so a future "cost looks low" debugging session has the failure reason recorded. Verified end-to-end: - New `#[ignore]` integration test runs a real query against `INFORMATION_SCHEMA.SCHEMATA` and asserts `bytes_scanned >= 10 MiB` — proving the floor is applied (constant queries like `SELECT 1` are exempt and don't trigger the floor, so they aren't useful as a signal here). - The merge and time-interval live smoke tests both have their assertions tightened from `bytes_scanned > 0` to `bytes_scanned >= 10 MiB`. Both now report 20–30 MB across their multi-statement runs (one floor per `jobs.query` call). Drops finding #6 in `live/README.md` — the gap is closed.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Bumps opentelemetry_sdk from 0.28.0 to 0.31.0.
Changelog
Sourced from opentelemetry_sdk's changelog.
... (truncated)
Commits
285dc92chore: Prepare for release v0.31.0 (#3179)9cde968chore: Prepare for release otel-http v0.30.1, Revert part of multi-value key ...5250df2fix: Suppress telemetry emitted inside ofBatchLogProcessor::emit(#3172)9bd2c1bfix: use instrumentation schema URL on scope spans (#3171)159135cfeat: Add is_remote flag in exporter for spans and span links (#3153)b7ff11bfix: Use path+version dependencies for publishing to crates.io otel-http (#3...24da5c9fix: Use path+version dependencies for publishing to crates.io (#3167)6f75c58fix: Addstdfeature to serde to fix CI linting issues (#3165)80b5dcbchore: Bump opentelemetry-proto to v0.30.1 and opentelemetry-otlp to v0.30.1 ...b70771achore: bump otel-proto to v1.8.0 (#3156)