feat: Mark getDefaultEndpoint() with @ObsoleteApi#2347
Merged
Conversation
blakeli0
reviewed
Jan 5, 2024
| AnnotationNode.builder() | ||
| .setType(FIXED_TYPESTORE.get(ObsoleteApi.class.getSimpleName())) | ||
| .setDescription( | ||
| "See https://github.com/googleapis/sdk-platform-java/issues/2346 for more information") |
Contributor
There was a problem hiding this comment.
Can we provide a brief information in the description first? So that customers don't have to click through the link when they hover over on the method? Something like "There is no concept of a default endpoint anymore with the introduction of universeDomain, please use ... instead".
In addition, maybe we can add the annotation only until the alternative getResolvedEndpoint() is introduced later?
Member
Author
There was a problem hiding this comment.
Sure, I'll can add a description and hold off on adding the annotation until a future date.
until the alternative getResolvedEndpoint() is introduced
I don't think getResolvedEndpoint() is going to be a 100% true alternative to getDefaultEndpoint(). Based on the previous configurations (prior to EndpointContext), the default endpoint may not be the resolved endpoint used by the client. We're no longer supporting a concept of a default endpoint so there really should not be an alternative to getDefaultEndpoint().
Contributor
There was a problem hiding this comment.
I'm not a fan of having github links in comments, I think they should be used only if we have no other way. They could become invalid for a lot of reasons, repo access changes, Github URL format change etc., or maybe the source of truth is moved elsewhere. Can we add more info in the annotation description itself?
lqiu96
added
the
do not merge
Member
Author
|
Removing the |
lqiu96
force-pushed
the
mark-getDefaultEndpoint-obsoleteApi
branch
from
7cf57ca to
9ea0253
Compare
lqiu96
removed
the
do not merge
blakeli0
approved these changes
Jul 12, 2024
|
|
lqiu96
pushed a commit
that referenced
this pull request
Jul 25, 2024
🤖 I have created a release *beep* *boop* --- <details><summary>2.43.0</summary> ## [2.43.0](v2.42.0...v2.43.0) (2024-07-25) ### Features * add `transport` option to `generation_config.yaml` ([#3052](#3052)) ([3b1a915](3b1a915)) * get released version from versions.txt to render `README.md` ([#3007](#3007)) ([99bb2b3](99bb2b3)) * Introduce java.time to Gax-Java ([#1872](#1872)) ([308aeaf](308aeaf)) * Mark `getDefaultEndpoint()` with @ObsoleteApi ([#2347](#2347)) ([e46648f](e46648f)) * parse `BUILD.bzel` to determine whether a commit that only changed `BUILD.bazel` is a qualified commit ([#2937](#2937)) ([502f801](502f801)) ### Bug Fixes * Fix: ([d996c2d](d996c2d)) * `BaseApiTracer` to noop on attemptFailed via overloaded method call ([#3016](#3016)) ([2fc938a](2fc938a)) * Generator to skip generation for empty services. ([#3051](#3051)) ([ff2c485](ff2c485)) * restore hermetic build image publication ([#2952](#2952)) ([97a6d67](97a6d67)) ### Dependencies * update dependency com.fasterxml.jackson:jackson-bom to v2.17.2 ([#3028](#3028)) ([d16f9d1](d16f9d1)) * update dependency com.google.cloud.opentelemetry:detector-resources-support to v0.30.0 ([#2975](#2975)) ([b3ec93f](b3ec93f)) * update dependency com.google.cloud.opentelemetry:detector-resources-support to v0.31.0 ([#3044](#3044)) ([6bd07dc](6bd07dc)) * update dependency com.google.errorprone:error_prone_annotations to v2.29.2 ([#3058](#3058)) ([8ea0868](8ea0868)) * update dependency com.google.errorprone:error_prone_annotations to v2.29.2 ([#3059](#3059)) ([81b23dc](81b23dc)) * update dependency com.google.guava:guava to v33.2.1-jre ([#3027](#3027)) ([12ee456](12ee456)) * update dependency commons-codec:commons-codec to v1.17.1 ([#3049](#3049)) ([58d94b7](58d94b7)) * update dependency dev.cel:cel to v0.6.0 ([#3050](#3050)) ([bc332d9](bc332d9)) * update dependency net.bytebuddy:byte-buddy to v1.14.18 ([#3029](#3029)) ([8799cf6](8799cf6)) * update dependency org.apache.commons:commons-lang3 to v3.15.0 ([#3060](#3060)) ([2538334](2538334)) * update dependency org.checkerframework:checker-qual to v3.45.0 ([#2988](#2988)) ([4edd216](4edd216)) * update google api dependencies ([#2951](#2951)) ([c16f6c9](c16f6c9)) * update google auth library dependencies to v1.24.0 ([#3039](#3039)) ([98b5bd7](98b5bd7)) * update googleapis/java-cloud-bom digest to 47c5dbc ([#2974](#2974)) ([57623f0](57623f0)) * update grpc dependencies to v1.65.1 ([#3061](#3061)) ([27497e2](27497e2)) * update junit5 monorepo to v5.10.3 ([#2963](#2963)) ([bc55fe1](bc55fe1)) * update netty dependencies to v4.1.112.final ([#3057](#3057)) ([5af127b](5af127b)) * update opentelemetry-java monorepo to v1.40.0 ([#3035](#3035)) ([5c31c42](5c31c42)) * Use Gapic-Showcase v0.35.1 ([#3018](#3018)) ([43773f0](43773f0)) ### Documentation * add support option to 'new issue' choices ([#3055](#3055)) ([6a2a17d](6a2a17d)) </details> --- 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>
chingor13
pushed a commit
to googleapis/google-cloud-java
that referenced
this pull request
Mar 24, 2026
🤖 I have created a release *beep* *boop* --- <details><summary>2.43.0</summary> ## [2.43.0](googleapis/sdk-platform-java@v2.42.0...v2.43.0) (2024-07-25) ### Features * add `transport` option to `generation_config.yaml` ([#3052](googleapis/sdk-platform-java#3052)) ([2e6c95c](googleapis/sdk-platform-java@2e6c95c)) * get released version from versions.txt to render `README.md` ([#3007](googleapis/sdk-platform-java#3007)) ([204e73b](googleapis/sdk-platform-java@204e73b)) * Introduce java.time to Gax-Java ([#1872](googleapis/sdk-platform-java#1872)) ([300026a](googleapis/sdk-platform-java@300026a)) * Mark `getDefaultEndpoint()` with @ObsoleteApi ([#2347](googleapis/sdk-platform-java#2347)) ([311d4ed](googleapis/sdk-platform-java@311d4ed)) * parse `BUILD.bzel` to determine whether a commit that only changed `BUILD.bazel` is a qualified commit ([#2937](googleapis/sdk-platform-java#2937)) ([254bf37](googleapis/sdk-platform-java@254bf37)) ### Bug Fixes * Fix: ([8d11e25](googleapis/sdk-platform-java@8d11e25)) * `BaseApiTracer` to noop on attemptFailed via overloaded method call ([#3016](googleapis/sdk-platform-java#3016)) ([41532b4](googleapis/sdk-platform-java@41532b4)) * Generator to skip generation for empty services. ([#3051](googleapis/sdk-platform-java#3051)) ([1af3059](googleapis/sdk-platform-java@1af3059)) * restore hermetic build image publication ([#2952](googleapis/sdk-platform-java#2952)) ([2a5d36d](googleapis/sdk-platform-java@2a5d36d)) ### Dependencies * update dependency com.fasterxml.jackson:jackson-bom to v2.17.2 ([#3028](googleapis/sdk-platform-java#3028)) ([3c41fc4](googleapis/sdk-platform-java@3c41fc4)) * update dependency com.google.cloud.opentelemetry:detector-resources-support to v0.30.0 ([#2975](googleapis/sdk-platform-java#2975)) ([10fe657](googleapis/sdk-platform-java@10fe657)) * update dependency com.google.cloud.opentelemetry:detector-resources-support to v0.31.0 ([#3044](googleapis/sdk-platform-java#3044)) ([283c889](googleapis/sdk-platform-java@283c889)) * update dependency com.google.errorprone:error_prone_annotations to v2.29.2 ([#3058](googleapis/sdk-platform-java#3058)) ([f8bd80c](googleapis/sdk-platform-java@f8bd80c)) * update dependency com.google.errorprone:error_prone_annotations to v2.29.2 ([#3059](googleapis/sdk-platform-java#3059)) ([062add9](googleapis/sdk-platform-java@062add9)) * update dependency com.google.guava:guava to v33.2.1-jre ([#3027](googleapis/sdk-platform-java#3027)) ([e305208](googleapis/sdk-platform-java@e305208)) * update dependency commons-codec:commons-codec to v1.17.1 ([#3049](googleapis/sdk-platform-java#3049)) ([7934920](googleapis/sdk-platform-java@7934920)) * update dependency dev.cel:cel to v0.6.0 ([#3050](googleapis/sdk-platform-java#3050)) ([ae465dc](googleapis/sdk-platform-java@ae465dc)) * update dependency net.bytebuddy:byte-buddy to v1.14.18 ([#3029](googleapis/sdk-platform-java#3029)) ([e5382c4](googleapis/sdk-platform-java@e5382c4)) * update dependency org.apache.commons:commons-lang3 to v3.15.0 ([#3060](googleapis/sdk-platform-java#3060)) ([d62f6da](googleapis/sdk-platform-java@d62f6da)) * update dependency org.checkerframework:checker-qual to v3.45.0 ([#2988](googleapis/sdk-platform-java#2988)) ([2b524e5](googleapis/sdk-platform-java@2b524e5)) * update google api dependencies ([#2951](googleapis/sdk-platform-java#2951)) ([46ea538](googleapis/sdk-platform-java@46ea538)) * update google auth library dependencies to v1.24.0 ([#3039](googleapis/sdk-platform-java#3039)) ([55e9db2](googleapis/sdk-platform-java@55e9db2)) * update googleapis/java-cloud-bom digest to 47c5dbc ([#2974](googleapis/sdk-platform-java#2974)) ([f3f4036](googleapis/sdk-platform-java@f3f4036)) * update grpc dependencies to v1.65.1 ([#3061](googleapis/sdk-platform-java#3061)) ([e587e23](googleapis/sdk-platform-java@e587e23)) * update junit5 monorepo to v5.10.3 ([#2963](googleapis/sdk-platform-java#2963)) ([4a8eb0a](googleapis/sdk-platform-java@4a8eb0a)) * update netty dependencies to v4.1.112.final ([#3057](googleapis/sdk-platform-java#3057)) ([6b96a35](googleapis/sdk-platform-java@6b96a35)) * update opentelemetry-java monorepo to v1.40.0 ([#3035](googleapis/sdk-platform-java#3035)) ([564e4cb](googleapis/sdk-platform-java@564e4cb)) * Use Gapic-Showcase v0.35.1 ([#3018](googleapis/sdk-platform-java#3018)) ([82094b3](googleapis/sdk-platform-java@82094b3)) ### Documentation * add support option to 'new issue' choices ([#3055](googleapis/sdk-platform-java#3055)) ([674401b](googleapis/sdk-platform-java@674401b)) </details> --- 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>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Client libraries no longer always use the default endpoint. The resolved endpoint will be determined by the EndpointContext and used by the client.