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: support JDK 23 #2064

Merged
merged 3 commits into from
Jan 28, 2025
Merged

feat: support JDK 23 #2064

merged 3 commits into from
Jan 28, 2025

Conversation

diegomarquezp
Copy link
Contributor

@diegomarquezp diegomarquezp commented Jan 28, 2025

Fixes #2063
Thanks to @ldetmer and @burkedavison for the pointers on how to solve this.

The approach is to flush() a first byte in the test pointed out by the issue. That triggers sending the headers and unblocks NetHttpResponse.<init>

@product-auto-label product-auto-label bot added the size: s Pull request size is small. label Jan 28, 2025
@diegomarquezp diegomarquezp marked this pull request as ready for review January 28, 2025 21:01
@diegomarquezp diegomarquezp requested a review from a team as a code owner January 28, 2025 21:01
@diegomarquezp diegomarquezp requested a review from ldetmer January 28, 2025 21:02
@diegomarquezp diegomarquezp merged commit b6a3616 into main Jan 28, 2025
17 of 19 checks passed
@diegomarquezp diegomarquezp deleted the support-jdk-23 branch January 28, 2025 21:07
svc-squareup-copybara pushed a commit to cashapp/misk that referenced this pull request Feb 7, 2025
| Package | Type | Package file | Manager | Update | Change |
|---|---|---|---|---|---|
|
[com.google.cloud:google-cloud-datastore](https://github.com/googleapis/java-datastore)
| dependencies | misk/gradle/libs.versions.toml | gradle | patch |
`2.26.0` -> `2.26.1` |
|
[com.google.http-client:google-http-client-jackson2](https://github.com/googleapis/google-http-java-client)
| dependencies | misk/gradle/libs.versions.toml | gradle | minor |
`1.45.3` -> `1.46.0` |
|
[com.google.http-client:google-http-client](https://github.com/googleapis/google-http-java-client)
| dependencies | misk/gradle/libs.versions.toml | gradle | minor |
`1.45.3` -> `1.46.0` |
| [software.amazon.awssdk:sdk-core](https://aws.amazon.com/sdkforjava) |
dependencies | misk/gradle/libs.versions.toml | gradle | patch |
`2.30.14` -> `2.30.15` |
|
[software.amazon.awssdk:dynamodb-enhanced](https://aws.amazon.com/sdkforjava)
| dependencies | misk/gradle/libs.versions.toml | gradle | patch |
`2.30.14` -> `2.30.15` |
| [software.amazon.awssdk:dynamodb](https://aws.amazon.com/sdkforjava) |
dependencies | misk/gradle/libs.versions.toml | gradle | patch |
`2.30.14` -> `2.30.15` |
| [software.amazon.awssdk:aws-core](https://aws.amazon.com/sdkforjava) |
dependencies | misk/gradle/libs.versions.toml | gradle | patch |
`2.30.14` -> `2.30.15` |
| [software.amazon.awssdk:bom](https://aws.amazon.com/sdkforjava) |
dependencies | misk/gradle/libs.versions.toml | gradle | patch |
`2.30.14` -> `2.30.15` |
| [software.amazon.awssdk:auth](https://aws.amazon.com/sdkforjava) |
dependencies | misk/gradle/libs.versions.toml | gradle | patch |
`2.30.14` -> `2.30.15` |

---

### Release Notes

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

###
[`v2.26.1`](https://github.com/googleapis/java-datastore/blob/HEAD/CHANGELOG.md#2261-2025-02-05)

##### Bug Fixes

- **deps:** Update the Java code generator (gapic-generator-java) to
2.52.0
([9594024](googleapis/java-datastore@9594024))

##### Dependencies

- Update dependency com.google.cloud:gapic-libraries-bom to v1.51.0
([#&#8203;1726](googleapis/java-datastore#1726))
([89f31a8](googleapis/java-datastore@89f31a8))

</details>

<details>
<summary>googleapis/google-http-java-client
(com.google.http-client:google-http-client-jackson2)</summary>

###
[`v1.46.0`](https://github.com/googleapis/google-http-java-client/blob/HEAD/CHANGELOG.md#1460-2025-02-06)

##### Features

- Support JDK 23
([#&#8203;2064](googleapis/google-http-java-client#2064))
([b6a3616](googleapis/google-http-java-client@b6a3616))
- Update and adapt to GraalVM for JDK 23
([#&#8203;2069](googleapis/google-http-java-client#2069))
([7a0fab3](googleapis/google-http-java-client@7a0fab3))

##### Dependencies

- Update actions/upload-artifact action to v4.6.0
([#&#8203;2056](googleapis/google-http-java-client#2056))
([1e70d04](googleapis/google-http-java-client@1e70d04))
- Update dependency io.grpc:grpc-context to v1.70.0
([#&#8203;2068](googleapis/google-http-java-client#2068))
([7a580bf](googleapis/google-http-java-client@7a580bf))
- Update project.appengine.version to v2.0.32
([#&#8203;2065](googleapis/google-http-java-client#2065))
([2285bb1](googleapis/google-http-java-client@2285bb1))

</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**: Never, 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: 9507906cb3cb7366a85164991512d76c540932bf
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size: s Pull request size is small.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

NetHttpTransportTest.testDisconnectShouldNotWaitToReadResponse fails since JDK 23
2 participants