Skip to content

Updated snippets with blob metadata - #1635

Closed
gridcellcoder wants to merge 1 commit into
googleapis:masterfrom
gridcellcoder:patch-2
Closed

Updated snippets with blob metadata#1635
gridcellcoder wants to merge 1 commit into
googleapis:masterfrom
gridcellcoder:patch-2

Conversation

@gridcellcoder

Copy link
Copy Markdown

Use of correct map and return of the updated blob

Use of correct map and return of the updated blob
@googlebot

Copy link
Copy Markdown

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.


  • If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
  • If you signed the CLA as a corporation, please let us know the company's name.

@googlebotgooglebot added the cla: no This human has *not* signed the Contributor License Agreement. label Feb 20, 2017
@gridcellcoder
gridcellcoder deleted the patch-2 branch February 20, 2017 14:17
chingor13 pushed a commit that referenced this pull request Jan 22, 2026
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
suztomo pushed a commit to suztomo/google-cloud-java that referenced this pull request Mar 23, 2026
meltsufin pushed a commit that referenced this pull request Apr 29, 2026
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
meltsufin pushed a commit that referenced this pull request May 1, 2026
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
meltsufin pushed a commit that referenced this pull request May 2, 2026
* feat: Add FirestoreOpenTelemetryOptions to FirestoreOptions. (#1531)
* feat: Add FirestoreOpenTelemetryOptions to FirestoreOptions.
* Address code review feedback.
* feat: Add com.google.cloud.firestore.telemetry package. (#1533)
* feat: Add FirestoreOpenTelemetryOptions to FirestoreOptions.
* feat: Add com.google.cloud.firestore.telemetry package.
* Address code review feedback.
* Factor out the otel version in pom.xml.
* fix: Remove OpenCensus tracing code. (#1589)
* feat: Add FirestoreOpenTelemetryOptions to FirestoreOptions.
* feat: Add com.google.cloud.firestore.telemetry package.
* fix: Remove OpenCensus tracing code.
* feat: tracing for aggregate queries, bulkwriter, partition queries, a… (#1590)
* feat: Add FirestoreOpenTelemetryOptions to FirestoreOptions.
* feat: Add com.google.cloud.firestore.telemetry package.
* fix: Remove OpenCensus tracing code.
* feat: tracing for aggregate queries, bulkwriter, partition queries, and listDocuments.
* Address code review feedback.
* Address feedback.
* don't use wildcard imports.
* feat: trace instrumentation for DocumentReference methods. (#1591)
* feat: Add FirestoreOpenTelemetryOptions to FirestoreOptions.
* feat: Add com.google.cloud.firestore.telemetry package.
* fix: Remove OpenCensus tracing code.
* feat: tracing for aggregate queries, bulkwriter, partition queries, and listDocuments.
* feat: trace instrumentation for DocumentReference methods.
* feat: trace instrumentation for queries and transactions. (#1592)
* feat: Add FirestoreOpenTelemetryOptions to FirestoreOptions.
* feat: Add com.google.cloud.firestore.telemetry package.
* fix: Remove OpenCensus tracing code.
* feat: tracing for aggregate queries, bulkwriter, partition queries, and listDocuments.
* feat: trace instrumentation for DocumentReference methods.
* feat: trace instrumentation for queries and transactions.
* test: Adding first e2e client-tracing test w/ Custom Root Span (#1621)
* test: Adding first e2e client-tracing test w/ Custom Root Span
* Roll back E2E tests commit.
* Address feedback.
* Address feedback (better event log message).
* Address feedback.
---------
Co-authored-by: Jimit J Shah <57637300+jimit-j-shah@users.noreply.github.com>
* test: End-to-End Integration Test for Client-side Tracing in Firestore Java Server SDK using OpenTelemetry SDK and Cloud Trace Exporter against Cloud Trace. (#1635)
* Adding first e2e client-tracing test w/ Custom Root Span
* test: Adding first e2e client-tracing test w/ Custom Root Span
* Fixing test dependencies and use default GCP testing project.
Fixing
* Fixing test dependencies and use default GCP testing project.
* Fixing formatting
* Add aggregationQueryGet Test
* Add bulkWriterCommitTrace Test
* Fixing running multiple-tests
* Add partitionQuery Test
* Add collectionListDocumentsTrace Test
* Add docRef*Trace Tests
* Add docRefUpdate*Trace and docRefDelete*Trace Tests
* Fixing Trace fetching using retries for missing or incomplete traces due to eventual consistency of Cloud Trace
* Add get/query Trace Tests
* Add Transaction test
* Added TraceContainer to be able to test transaction test-cases
* test: Adding Transaction tests
* test: Adding Transaction tests
* test: Adding TestParameterInjector to run the test for global and non-global opentelemetry SDK instances
* test: formatting and cleanup
* test: Adding first e2e client-tracing test w/ Custom Root Span
* test: Add aggregationQueryGet Test
* test: Add bulkWriterCommitTrace Test and fixed running multiple-tests
* test: Add partitionQuery Test
* test: Add collectionListDocumentsTrace Test
* test: Add docRefUpdate*Trace and docRefDelete*Trace Tests and fixed Trace fetching using retries for missing or incomplete traces due to eventual consistency of Cloud Trace
* test: Add get/query Trace Tests
* test: Added Transaction tests using TraceContainer to verify traces for Transaction ops (BeginTransaction, Rollback etc)
* test: Adding TestParameterInjector to run the test for global and non-global opentelemetry SDK instances
* test: Formatting and cleanup
* test: review comments
* test: fixing dfs to handle case where the compareTo callstack may be shorter than the trace callstack - don't need to throw an exception in that case
* test: Consolidating verification methods
* test: review comments
* fix: Make telemetry-related fields transient. (#1638)
* fix: Rename 'enabled' to 'tracingEnabled'. (#1639)
* fix: Rename 'enabled' to 'tracingEnabled'.
In the future, FirestoreOpenTelemetryOptions will support enabling/disabling
Logging and Metrics as well. So we should use a better name for this field.
* address feedback.
* fix: Minor improvement to the ITE2ETracingTest. (#1637)
* fix: Minor improvement to the ITE2ETracingTest.
* revert the numExpectedSpans change.
* feat: Add 'isTransactional' attribute. (#1657)
* fix: Necessary test improvements for CI environments. (#1673)
* fix: Necessary test improvements for CI environments.
* Address feedback.
* feat: Disable the tracing feature and remove public APIs.
* 🦉 Updates from OwlBot post-processor
See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
* Add the Firestore SDK version to the attributes.
* 🦉 Updates from OwlBot post-processor
See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
* Update the "test" dependency versions.
* Address feedback related to attributes.
* Add 'project_id' attribute.
* 🦉 Updates from OwlBot post-processor
See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
* Revert a736fcc ("Disable the tracing feature and remove public APIs").
* GlobalOtel reset for test must happen in `before`, not `after`.
* Address feedback.
* 🦉 Updates from OwlBot post-processor
See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
---------
Co-authored-by: Jimit J Shah <57637300+jimit-j-shah@users.noreply.github.com>
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
benjaminp pushed a commit to benjaminp/google-cloud-java that referenced this pull request Jul 13, 2026
… to v2.19.1 (googleapis#1635)
[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)
This PR contains the following updates:
| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [com.google.cloud:google-cloud-bigtable](https://togithub.com/googleapis/java-bigtable) | `2.19.0` -> `2.19.1` | [![age](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-bigtable/2.19.1/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-bigtable/2.19.1/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-bigtable/2.19.1/compatibility-slim/2.19.0)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-bigtable/2.19.1/confidence-slim/2.19.0)](https://docs.renovatebot.com/merge-confidence/) |
---
### Release Notes
<details>
<summary>googleapis/java-bigtable</summary>
### [`v2.19.1`](https://togithub.com/googleapis/java-bigtable/blob/HEAD/CHANGELOG.md#&#8203;2191-httpsgithubcomgoogleapisjava-bigtablecomparev2190v2191-2023-02-16)
[Compare Source](https://togithub.com/googleapis/java-bigtable/compare/v2.19.0...v2.19.1)
##### Bug Fixes
- Change the return type of Heartbeat::getEstimatedLowWatermark to long ([#&googleapis#8203;1631](https://togithub.com/googleapis/java-bigtable/issues/1631)) ([547acea](https://togithub.com/googleapis/java-bigtable/commit/547acea43606b4ba312f8143d32a85f9538f7775))
- Fix connectivity error count calculation ([#&googleapis#8203;1632](https://togithub.com/googleapis/java-bigtable/issues/1632)) ([38f4141](https://togithub.com/googleapis/java-bigtable/commit/38f4141c1cdfa8ff7859e7c68a1f85edeb099b83))
- **test:** Fix flaky test ([#&googleapis#8203;1633](https://togithub.com/googleapis/java-bigtable/issues/1633)) ([b754e85](https://togithub.com/googleapis/java-bigtable/commit/b754e857c305ebc4b77a09cf0dc38f936000f2fc))
##### Dependencies
- Update dependency org.graalvm.buildtools:junit-platform-native to v0.9.20 ([#&googleapis#8203;1626](https://togithub.com/googleapis/java-bigtable/issues/1626)) ([b1e7b91](https://togithub.com/googleapis/java-bigtable/commit/b1e7b91aa51b86594ab9c767e0e0f42236fc336c))
- Update dependency org.graalvm.buildtools:native-maven-plugin to v0.9.20 ([#&googleapis#8203;1627](https://togithub.com/googleapis/java-bigtable/issues/1627)) ([1676bbb](https://togithub.com/googleapis/java-bigtable/commit/1676bbb5b0b4563d23b6ae4e94841bc4750de701))
</details>
---
### Configuration
📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.
♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 **Ignore**: Close this PR and you won't be reminded about this update again.
---
- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box
---
This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/java-bigtable).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNC4xMjUuMSIsInVwZGF0ZWRJblZlciI6IjM0LjEyNS4xIn0=-->
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: noThis human has *not* signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@gridcellcoder@googlebot