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: attempt DirectPath by default #770

Merged
merged 6 commits into from Jan 6, 2021

Conversation

mohanli-ml
Copy link
Contributor

Update client to attempt DirectPath by default.

Note that it doesn't mean that after this change client will just use DirectPath, but will call the DirectPath codepath by default.

The actually enablement of DirectPath is controlled by service owner via ACL config. For now, after this change, although all users will attempt DirectPath, but they will all just fallback to the original CFE path.

For testing, we still use the DP test endpoint aa423245250f2bbf.sandbox.googleapis.com to override the default endpoint.

@mohanli-ml mohanli-ml requested a review from a team as a code owner January 5, 2021 20:41
@product-auto-label product-auto-label bot added the api: spanner Issues related to the googleapis/java-spanner API. label Jan 5, 2021
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Jan 5, 2021
@codecov
Copy link

codecov bot commented Jan 6, 2021

Codecov Report

Merging #770 (6329b45) into master (90ad9d6) will increase coverage by 0.01%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #770      +/-   ##
============================================
+ Coverage     85.00%   85.01%   +0.01%     
+ Complexity     2562     2560       -2     
============================================
  Files           143      143              
  Lines         14007    14005       -2     
  Branches       1338     1337       -1     
============================================
  Hits          11906    11906              
+ Misses         1538     1537       -1     
+ Partials        563      562       -1     
Impacted Files Coverage Δ Complexity Δ
...m/google/cloud/spanner/spi/v1/GapicSpannerRpc.java 81.55% <100.00%> (+0.20%) 80.00 <0.00> (-1.00) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 90ad9d6...6329b45. Read the comment docs.

@thiagotnunes thiagotnunes changed the title feat: attemp DirectPath by default feat: attempt DirectPath by default Jan 6, 2021
Copy link
Contributor

@thiagotnunes thiagotnunes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@thiagotnunes thiagotnunes merged commit dc02244 into googleapis:master Jan 6, 2021
thiagotnunes pushed a commit that referenced this pull request May 6, 2021
* chore: add DirectPath fallback integration test

* feat: attemp DirectPath by default

* feat: attemp DirectPath by default
ansh0l pushed a commit to ansh0l/java-spanner that referenced this pull request Nov 10, 2022
This is an auto-generated regeneration of the .pb.go files by
cloud.google.com/go/internal/gapicgen. Once this PR is submitted, genbot will
update the corresponding PR to depend on the newer version of go-genproto, and
assign reviewers. Whilst this or any regen PR is open in go-genproto, genbot
will not create any more regeneration PRs. If all regen PRs are closed,
gapicgen will create a new set of regeneration PRs once per night.

If you have been assigned to review this PR, please:

- Ensure that CI is passing. If it's failing, it requires your manual attention.
- Approve and submit this PR if you believe it's ready to ship. That will prompt
genbot to assign reviewers to the google-cloud-go PR.

Corresponding google-cloud-go PR: googleapis/google-cloud-go#5714

Changes:

chore: regenerate API index

  Source-Link: googleapis/googleapis@4b97f62

feat: The experiment has ended and the API is deleted
  PiperOrigin-RevId: 432000693
  Source-Link: googleapis/googleapis@4fd97e7

docs(artifactregistry): more details for ListFilesRequest parent field
  PiperOrigin-RevId: 431979329
  Source-Link: googleapis/googleapis@8febda4

docs(bigquery/storage): improve documentation for write client feat: update default timeout/retry information feat: update parent annotation for BatchCommitWriteStreamsRequest feat: expose additional StorageError enum values
  PiperOrigin-RevId: 431973595
  Source-Link: googleapis/googleapis@25d691b

feat!: modified descriptions of validation rules on create entity endpoints * Commas disallowed * Backslash allowed
  BREAKING CHANGE: Because of a bug commas have been accepted as part of individual IDs. Fixing this bug is a breaking change but we don't expect users to be impacted as we have detected no individual IDs being generated with commas. Commas are still OK to use as delimiters for lists of multiple IDs.

  PiperOrigin-RevId: 431967940
  Source-Link: googleapis/googleapis@c392eb0

chore(retail): remove unused imports in google/cloud/retail/v2
  PiperOrigin-RevId: 431915215
  Source-Link: googleapis/googleapis@d4f3468

chore: regenerate API index

  Source-Link: googleapis/googleapis@4344efd

feat(securitycenter): Add BigQuery export APIs that help you enable writing new/updated findings from  Security Command Center to a BigQuery table in near-real time. You can then integrate the data into existing workflows and create custom analyses. You can enable this feature at the organization, folder, and project levels to export findings based on your requirements
  PiperOrigin-RevId: 431905699
  Source-Link: googleapis/googleapis@4c1b56f
rajatbhatta pushed a commit to rajatbhatta/java-spanner that referenced this pull request Nov 17, 2022
…cies to v2.8.0 (googleapis#770)

[![WhiteSource Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [com.google.cloud:google-cloud-shared-dependencies](https://togithub.com/googleapis/java-shared-dependencies) | `2.7.0` -> `2.8.0` | [![age](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-shared-dependencies/2.8.0/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-shared-dependencies/2.8.0/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-shared-dependencies/2.8.0/compatibility-slim/2.7.0)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-shared-dependencies/2.8.0/confidence-slim/2.7.0)](https://docs.renovatebot.com/merge-confidence/) |

---

### Release Notes

<details>
<summary>googleapis/java-shared-dependencies</summary>

### [`v2.8.0`](https://togithub.com/googleapis/java-shared-dependencies/blob/HEAD/CHANGELOG.md#&#8203;280-httpsgithubcomgoogleapisjava-shared-dependenciescomparev270v280-2022-03-02)

[Compare Source](https://togithub.com/googleapis/java-shared-dependencies/compare/v2.7.0...v2.8.0)

##### Dependencies

-   update dependency com.google.api-client:google-api-client-bom to v1.33.2 ([#&#8203;602](https://togithub.com/googleapis/java-shared-dependencies/issues/602)) ([85b132f](https://togithub.com/googleapis/java-shared-dependencies/commit/85b132f5830772646025b6a9fbbb970a941b86b5))
-   update dependency com.google.api:api-common to v2.1.4 ([#&#8203;605](https://togithub.com/googleapis/java-shared-dependencies/issues/605)) ([ca7d49a](https://togithub.com/googleapis/java-shared-dependencies/commit/ca7d49aee26b6d90abd2afd61c20861d2307fe9c))
-   update dependency com.google.auth:google-auth-library-bom to v1.5.3 ([#&#8203;614](https://togithub.com/googleapis/java-shared-dependencies/issues/614)) ([e6413f3](https://togithub.com/googleapis/java-shared-dependencies/commit/e6413f3b1be78473ace7085c344eda2d78cdf01a))
-   update dependency com.google.cloud:google-cloud-core to v2.5.6 ([#&#8203;621](https://togithub.com/googleapis/java-shared-dependencies/issues/621)) ([3d5669f](https://togithub.com/googleapis/java-shared-dependencies/commit/3d5669f215689dd2df71200ffe37d5a9d385cda8))
-   update dependency com.google.code.gson:gson to v2.9.0 ([#&#8203;611](https://togithub.com/googleapis/java-shared-dependencies/issues/611)) ([f59c28d](https://togithub.com/googleapis/java-shared-dependencies/commit/f59c28d8cd30ada1237f7722135ba148dce6315e))
-   update dependency com.google.http-client:google-http-client-bom to v1.41.4 ([#&#8203;608](https://togithub.com/googleapis/java-shared-dependencies/issues/608)) ([f9a4f23](https://togithub.com/googleapis/java-shared-dependencies/commit/f9a4f23d3a32148ac59e53eaae6558ccbceca12c))
-   update dependency com.google.oauth-client:google-oauth-client-bom to v1.33.1 ([#&#8203;606](https://togithub.com/googleapis/java-shared-dependencies/issues/606)) ([3882494](https://togithub.com/googleapis/java-shared-dependencies/commit/3882494770d48fcc02ed19088aa06612a8e440eb))
-   update dependency com.google.protobuf:protobuf-bom to v3.19.4 ([#&#8203;593](https://togithub.com/googleapis/java-shared-dependencies/issues/593)) ([1e155bf](https://togithub.com/googleapis/java-shared-dependencies/commit/1e155bfc957bbb7e25d2e0994cdecaa81843bdc5))
-   update dependency io.grpc:grpc-bom to v1.44.1 ([#&#8203;613](https://togithub.com/googleapis/java-shared-dependencies/issues/613)) ([3038a2c](https://togithub.com/googleapis/java-shared-dependencies/commit/3038a2c86cd20c91b65f2d7926eeb739147a68db))
-   update dependency junit:junit to v4.13.2 ([#&#8203;607](https://togithub.com/googleapis/java-shared-dependencies/issues/607)) ([987e617](https://togithub.com/googleapis/java-shared-dependencies/commit/987e61796c7a093e005fe8832cf39275c391b2c1))
-   update dependency org.checkerframework:checker-qual to v3.21.3 ([#&#8203;596](https://togithub.com/googleapis/java-shared-dependencies/issues/596)) ([ac5083c](https://togithub.com/googleapis/java-shared-dependencies/commit/ac5083cd010924dc128f041c2cdbab20166a6bf0))
-   update gax.version to v2.12.2 ([#&#8203;592](https://togithub.com/googleapis/java-shared-dependencies/issues/592)) ([713ff8d](https://togithub.com/googleapis/java-shared-dependencies/commit/713ff8dd94f939c417524616bc47c771a3fbe0cf))
-   update google.common-protos.version to v2.7.4 ([#&#8203;603](https://togithub.com/googleapis/java-shared-dependencies/issues/603)) ([bb9b4c7](https://togithub.com/googleapis/java-shared-dependencies/commit/bb9b4c7e39552cc73b3b9d4c826b26e8cb74459f))
-   update google.core.version ([#&#8203;599](https://togithub.com/googleapis/java-shared-dependencies/issues/599)) ([327d0df](https://togithub.com/googleapis/java-shared-dependencies/commit/327d0df9b57203c0d4f426de0380770d3d7910d6))
-   update iam.version to v1.2.6 ([#&#8203;598](https://togithub.com/googleapis/java-shared-dependencies/issues/598)) ([2801439](https://togithub.com/googleapis/java-shared-dependencies/commit/280143964c7c3b93a8d7f67215ba2cc74ffce761))

</details>

---

### Configuration

📅 **Schedule**: 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.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

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

---

This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/java-spanner-jdbc).
rajatbhatta pushed a commit to rajatbhatta/java-spanner that referenced this pull request Nov 17, 2022
🤖 I have created a release *beep* *boop*
---


### [2.6.1](googleapis/java-spanner-jdbc@v2.6.0...v2.6.1) (2022-03-02)


### Dependencies

* update dependency com.google.cloud:google-cloud-shared-dependencies to v2.8.0 ([googleapis#770](googleapis/java-spanner-jdbc#770)) ([4f32b2d](googleapis/java-spanner-jdbc@4f32b2d))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
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. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants