Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

surbhigarg92
Copy link
Contributor

@surbhigarg92 surbhigarg92 commented May 8, 2025

Currently the Java implementation only parses the integer part.

According to the w3c standard, this should be a float (https://www.w3.org/TR/server-timing/#duration-attribute).

@surbhigarg92 surbhigarg92 requested review from a team as code owners May 8, 2025 08:54
@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 May 8, 2025
@surbhigarg92 surbhigarg92 force-pushed the server_timing_float branch from 080eabb to 448061e Compare May 8, 2025 08:59
@rahul2393 rahul2393 force-pushed the server_timing_float branch from c3b452a to 1b80ffe Compare May 8, 2025 10:20
@surbhigarg92 surbhigarg92 added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label May 8, 2025
@rahul2393 rahul2393 removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label May 9, 2025
@rahul2393 rahul2393 merged commit da8dd8d into main May 9, 2025
37 checks passed
@rahul2393 rahul2393 deleted the server_timing_float branch May 9, 2025 05:01
surbhigarg92 added a commit that referenced this pull request May 9, 2025
* fix: change server timing duration attribute to float as per w3c

* fix clirr

* remove deprecated warning

---------

Co-authored-by: rahul yadav <[email protected]>
Co-authored-by: rahul2393 <[email protected]>
svc-squareup-copybara pushed a commit to cashapp/misk that referenced this pull request May 9, 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.92.0` -> `6.93.0` |

---

### Release Notes

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

###
[`v6.93.0`](https://github.com/googleapis/java-spanner/blob/HEAD/CHANGELOG.md#6930-2025-05-09)

##### Features

- Enable AFE and gRPC metrics for DP
([#&#8203;3852](googleapis/java-spanner#3852))
([203baae](googleapis/java-spanner@203baae))

##### Bug Fixes

- Change server timing duration attribute to float as per w3c
([#&#8203;3851](googleapis/java-spanner#3851))
([da8dd8d](googleapis/java-spanner@da8dd8d))
- **deps:** Update the Java code generator (gapic-generator-java) to
2.57.0
([23b985c](googleapis/java-spanner@23b985c))
- Non-ASCII Unicode characters in code
([#&#8203;3844](googleapis/java-spanner#3844))
([85a0820](googleapis/java-spanner@85a0820))
- Only close and return sessions once
([#&#8203;3846](googleapis/java-spanner#3846))
([32b2373](googleapis/java-spanner@32b2373))

</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**: Enabled.

β™» **Rebasing**: Never, 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 [Renovate
Bot](https://github.com/renovatebot/renovate).

GitOrigin-RevId: bcf762c4aff746876d98c5fcd10f568fa9f00651
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