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

Remove @Beta from Futures.getChecked (or deprecate and remove) #3424

Closed
cpovirk opened this issue Mar 25, 2019 · 3 comments
Closed

Remove @Beta from Futures.getChecked (or deprecate and remove) #3424

cpovirk opened this issue Mar 25, 2019 · 3 comments
Labels
P3 no SLO package=concurrent status=triaged type=debeta Request to remove something from @Beta

Comments

@cpovirk
Copy link
Member

cpovirk commented Mar 25, 2019

I admit that I included this method in part because I felt bad having only Futures.getUnchecked, but I feel a little sad about it, especially its use of reflection, which is Android/Proguard-hostile. That said, would we care enough to delete it?

(One crazy possibility is to supply a way to codegen compile-time equivalents to the method.)

@cpovirk cpovirk added package=concurrent status=triaged P2 type=debeta Request to remove something from @Beta labels Mar 25, 2019
@claudivanmoreira
Copy link

Hello @cpovirk , is anyone solving this issue or can I get her to make the adjustment?

@kluever kluever added P3 no SLO and removed P2 labels Aug 13, 2019
@MorrisLaw
Copy link

@ddunig2 check this one out

copybara-service bot pushed a commit that referenced this issue Apr 11, 2023
Fixes #3403 #1519 #1421 #3424

RELNOTES=`concurrent`: Remove `@Beta` from `Futures`.
PiperOrigin-RevId: 523457329
@kluever
Copy link
Member

kluever commented Apr 11, 2023

This is now fixed.

@kluever kluever closed this as completed Apr 11, 2023
copybara-service bot pushed a commit that referenced this issue Apr 11, 2023
Fixes #3403 #1519 #1421 #3424

RELNOTES=`concurrent`: Remove `@Beta` from `Futures`.
PiperOrigin-RevId: 523465435
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P3 no SLO package=concurrent status=triaged type=debeta Request to remove something from @Beta
Projects
None yet
Development

No branches or pull requests

4 participants