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 the FluentFuture type, its factory methods, and addCallback. #5821

Closed
wants to merge 1 commit into from

Conversation

copybara-service[bot]
Copy link
Contributor

Remove @Beta from the FluentFuture type, its factory methods, and addCallback.

This leaves @Beta only for the methods that remain @Beta in Futures itself, which I'm hoping to make some effort toward later this week.

This CL is an admission that we're never going to get around to reworking FluentFuture to be more of a "builder," as discussed in #3419. Maybe there's still a place for something more builder-like, but there's no chance that we'll try to change FluentFuture in place. And probably we'll never do anything.

RELNOTES=util.concurrent: Removed @Beta from the FluentFuture type, its factory methods, and addCallback.

… `addCallback`.

This leaves `@Beta` only for the methods that remain `@Beta` in `Futures` itself, which I'm hoping to make some effort toward later this week.

This CL is an admission that we're never going to get around to reworking `FluentFuture` to be more of a "builder," as discussed in #3419. Maybe there's still a place for something more builder-like, but there's no chance that we'll try to change `FluentFuture` in place. And probably we'll never do anything.

RELNOTES=`util.concurrent`: Removed `@Beta` from the `FluentFuture` type, its factory methods, and `addCallback`.
PiperOrigin-RevId: 416116755
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant