Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Improve tracing by adding attributes #3576

Merged

Conversation

surbhigarg92
Copy link
Contributor

@surbhigarg92 surbhigarg92 commented Jan 2, 2025

This PR adds db.name, instance.name, db.table attributes for all relevant Spans

@surbhigarg92 surbhigarg92 requested a review from a team as a code owner January 2, 2025 06:59
@product-auto-label product-auto-label bot added size: m Pull request size is medium. api: spanner Issues related to the googleapis/java-spanner API. labels Jan 2, 2025
@surbhigarg92 surbhigarg92 force-pushed the opentelemetry_traces_attributed branch from 6a7f6d5 to 3585ede Compare January 2, 2025 09:35
@surbhigarg92 surbhigarg92 merged commit eee333b into googleapis:main Jan 3, 2025
31 of 32 checks passed
@surbhigarg92 surbhigarg92 deleted the opentelemetry_traces_attributed branch January 3, 2025 05:16
svc-squareup-copybara pushed a commit to cashapp/misk that referenced this pull request Jan 6, 2025
| Package | Type | Package file | Manager | Update | Change |
|---|---|---|---|---|---|
|
[com.google.cloud:google-cloud-spanner](https://github.com/googleapis/java-spanner)
| dependencies | misk/gradle/libs.versions.toml | gradle | minor |
`6.83.0` -> `6.84.0` |
|
[org.assertj:assertj-core](https://assertj.github.io/doc/#assertj-core)
([source](https://github.com/assertj/assertj)) | dependencies |
misk/gradle/libs.versions.toml | gradle | patch | `3.27.1` -> `3.27.2` |
| [software.amazon.awssdk:sdk-core](https://aws.amazon.com/sdkforjava) |
dependencies | misk/gradle/libs.versions.toml | gradle | patch |
`2.29.44` -> `2.29.45` |
|
[software.amazon.awssdk:dynamodb-enhanced](https://aws.amazon.com/sdkforjava)
| dependencies | misk/gradle/libs.versions.toml | gradle | patch |
`2.29.44` -> `2.29.45` |
| [software.amazon.awssdk:dynamodb](https://aws.amazon.com/sdkforjava) |
dependencies | misk/gradle/libs.versions.toml | gradle | patch |
`2.29.44` -> `2.29.45` |
| [software.amazon.awssdk:aws-core](https://aws.amazon.com/sdkforjava) |
dependencies | misk/gradle/libs.versions.toml | gradle | patch |
`2.29.44` -> `2.29.45` |
| [software.amazon.awssdk:bom](https://aws.amazon.com/sdkforjava) |
dependencies | misk/gradle/libs.versions.toml | gradle | patch |
`2.29.44` -> `2.29.45` |
| [software.amazon.awssdk:auth](https://aws.amazon.com/sdkforjava) |
dependencies | misk/gradle/libs.versions.toml | gradle | patch |
`2.29.44` -> `2.29.45` |

---

### Release Notes

<details>
<summary>googleapis/java-spanner
(com.google.cloud:google-cloud-spanner)</summary>

###
[`v6.84.0`](https://github.com/googleapis/java-spanner/blob/HEAD/CHANGELOG.md#6840-2025-01-06)

##### Features

- Add support for ARRAY\<STRUCT> to CloudCilentExecutor
([#&#8203;3544](googleapis/java-spanner#3544))
([6cbaf7e](googleapis/java-spanner@6cbaf7e))
- Add transaction runner for connections
([#&#8203;3559](googleapis/java-spanner#3559))
([5a1be3d](googleapis/java-spanner@5a1be3d))
- Exposing InstanceType in Instance configuration (to define PROVISIONED
or FREE spanner instance)
([8d295c4](googleapis/java-spanner@8d295c4))
- Improve tracing by adding attributes
([#&#8203;3576](googleapis/java-spanner#3576))
([eee333b](googleapis/java-spanner@eee333b))
- **spanner:** Add jdbc support for external hosts
([#&#8203;3536](googleapis/java-spanner#3536))
([801346a](googleapis/java-spanner@801346a))

##### Bug Fixes

- AsyncTransactionManager did not always close the session
([#&#8203;3580](googleapis/java-spanner#3580))
([d9813a0](googleapis/java-spanner@d9813a0))
- Retry specific internal errors
([#&#8203;3565](googleapis/java-spanner#3565))
([b9ce1a6](googleapis/java-spanner@b9ce1a6))
- Update max_in_use_session at 10 mins interval
([#&#8203;3570](googleapis/java-spanner#3570))
([cc1753d](googleapis/java-spanner@cc1753d))

##### Dependencies

- Update opentelemetry.version to v1.45.0
([#&#8203;3531](googleapis/java-spanner#3531))
([78c82ed](googleapis/java-spanner@78c82ed))

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "after 6pm every weekday,before 2am
every weekday" in timezone Australia/Melbourne, 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.

👻 **Immortal**: This PR will be recreated if closed unmerged. Get
[config help](https://github.com/renovatebot/renovate/discussions) if
that's undesired.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Renovate
Bot](https://github.com/renovatebot/renovate).

GitOrigin-RevId: 89058b2cef6e62653b7c19ce971a25a1dff13e6b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: spanner Issues related to the googleapis/java-spanner API. size: m Pull request size is medium.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants