Skip to content

feat(Spanner)!: Update cloud-spanner to minimum Node version of 22 - #9291

Merged
alkatrivedi merged 2 commits into
mainfrom
node-version-upgrade-22
Sep 18, 2026
Merged

alkatrivedi merged 2 commits into
mainfrom
node-version-upgrade-22

Conversation

@alkatrivedi

@alkatrivedi alkatrivedi commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Upgrade the Node version from 18 to 22.

#8985

@alkatrivedi
alkatrivedi requested a review from a team as a code owner September 11, 2026 14:32

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

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.

Code Review

This pull request upgrades OpenTelemetry dependencies to v2, bumps the Node engine requirement to >=22, removes the deprecated @google-cloud/opentelemetry-resource-util package in favor of @opentelemetry/resource-detector-gcp and a local MonitoredResource interface, and updates metric view configurations and resource instantiations. The reviewer pointed out that in OpenTelemetry JS SDK v2, View and Aggregation are still classes rather than plain objects/options, and suggested reverting the plain object view configurations back to using new View and Aggregation.ExplicitBucketHistogram to prevent TypeScript compilation and runtime errors.

Comment thread handwritten/spanner/src/metrics/constants.ts
Comment thread handwritten/spanner/src/metrics/constants.ts
@alkatrivedi
alkatrivedi force-pushed the node-version-upgrade-22 branch 5 times, most recently from bc904b5 to ac440ca Compare September 17, 2026 06:32
@alkatrivedi alkatrivedi changed the title feat!: Update cloud-spanner to minimum Node version of 22 feat(Spanner)!: Update cloud-spanner to minimum Node version of 22 Sep 17, 2026
Comment thread handwritten/spanner/package.json
Comment thread handwritten/spanner/package.json
@alkatrivedi
alkatrivedi force-pushed the node-version-upgrade-22 branch 2 times, most recently from 46d5ed2 to 817f389 Compare September 18, 2026 08:15
@alkatrivedi
alkatrivedi requested a review from a team as a code owner September 18, 2026 08:15
@alkatrivedi
alkatrivedi force-pushed the node-version-upgrade-22 branch 2 times, most recently from fd8b54c to 85444ba Compare September 18, 2026 08:28
@alkatrivedi
alkatrivedi force-pushed the node-version-upgrade-22 branch from 85444ba to 0ae6482 Compare September 18, 2026 09:42
@alkatrivedi
alkatrivedi merged commit ecd5bfd into main Sep 18, 2026
50 checks passed
@alkatrivedi
alkatrivedi deleted the node-version-upgrade-22 branch September 18, 2026 13:08
alkatrivedi pushed a commit that referenced this pull request Sep 18, 2026
🤖 I have created a release *beep* *boop*
---


##
[9.0.0](spanner-v8.13.0...spanner-v9.0.0)
(2026-09-18)


### ⚠ BREAKING CHANGES

* **Spanner:** Update cloud-spanner to minimum Node version of 22
([#9291](#9291))
* **Spanner:** upgrade OpenTelemetry to v2
([#9329](#9329))

### Features

* **Spanner:** Update cloud-spanner to minimum Node version of 22
([#9291](#9291))
([ecd5bfd](ecd5bfd))
* **Spanner:** Upgrade OpenTelemetry to v2
([#9329](#9329))
([ed3a44e](ed3a44e))


### Performance Improvements

* **spanner:** Bypass gRPC message interception in metrics interceptor
([#9306](#9306))
([a44180d](a44180d))
* **spanner:** Disable gRPC channelz by default
([#9320](#9320))
([11c257b](11c257b))
* **spanner:** Optimize codec.getType and reuse type singletons
([#9366](#9366))
([1b1b411](1b1b411))
* **spanner:** Remove rest-parameter allocations in wrappedRequestFn
([#9365](#9365))
([7085e03](7085e03))
* **spanner:** Synchronous multiplexed session hand-off for non-stream
callers
([#9362](#9362))
([dbbc55c](dbbc55c))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants