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(spanner): add jdbc support for external hosts #3536

Merged
merged 5 commits into from
Dec 16, 2024

Conversation

sagnghos
Copy link
Contributor

@sagnghos sagnghos commented Dec 9, 2024

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

  • Make sure to open an issue as a bug/issue before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> ☕️

If you write sample code, please follow the samples format.

@sagnghos sagnghos requested a review from a team as a code owner December 9, 2024 05:53
@product-auto-label product-auto-label bot added size: s Pull request size is small. api: spanner Issues related to the googleapis/java-spanner API. labels Dec 9, 2024
@olavloite olavloite added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Dec 10, 2024
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Dec 10, 2024
@sagnghos sagnghos requested a review from a team as a code owner December 11, 2024 11:43
@product-auto-label product-auto-label bot added size: l Pull request size is large. and removed size: s Pull request size is small. labels Dec 11, 2024
Copy link

Warning: This pull request is touching the following templated files:

  • .github/workflows/ci.yaml
  • .kokoro/presubmit/java11.cfg

@product-auto-label product-auto-label bot added size: m Pull request size is medium. and removed size: l Pull request size is large. labels Dec 11, 2024
@sagnghos sagnghos force-pushed the sanghos-jdbc-support branch from 6f3cb54 to a24b6f0 Compare December 11, 2024 11:51
@sagnghos sagnghos force-pushed the sanghos-jdbc-support branch from a24b6f0 to f2827f6 Compare December 13, 2024 12:34
@sagnghos sagnghos requested a review from olavloite December 13, 2024 12:49
@olavloite olavloite added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Dec 13, 2024
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Dec 13, 2024
@rahul2393 rahul2393 added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Dec 16, 2024
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Dec 16, 2024
@rahul2393 rahul2393 merged commit 801346a into googleapis:main Dec 16, 2024
32 checks passed
@sagnghos sagnghos deleted the sanghos-jdbc-support branch December 16, 2024 08:23
surbhigarg92 pushed a commit to surbhigarg92/java-spanner that referenced this pull request Dec 26, 2024
* feat(spanner): add jdbc support for external hosts

* feat(spanner): added default port value and unit tests

* feat(spanner): fixed redundant class name typo
gagangupt16 pushed a commit to gagangupt16/java-spanner that referenced this pull request Dec 27, 2024
* feat(spanner): add jdbc support for external hosts

* feat(spanner): added default port value and unit tests

* feat(spanner): fixed redundant class name typo
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.

4 participants