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

Document ForwardingExecutorService behavior for default methods #6299

Conversation

findepi
Copy link
Contributor

@findepi findepi commented Jan 17, 2023

Currently, Guava is built with Java 9 and ExecutorService has no default methods. This changes in Java 19, where ExecutorService.close is added. Ensure users have right expectations about future evolution of the ForwardingExecutorService and its compatibility with newer Java versions.

Fixes #6296

Currently, Guava is built with Java 9 and `ExecutorService` has no
default methods. This changes in Java 19, where `ExecutorService.close`
is added. Ensure users have right expectations about future evolution of
the `ForwardingExecutorService` and its compatibility with newer Java
versions.
@cpovirk cpovirk self-assigned this Jan 18, 2023
@cpovirk cpovirk added type=api-docs Change/add API documentation package=concurrent P2 labels Jan 18, 2023
copybara-service bot pushed a commit that referenced this pull request Jan 18, 2023
Currently, Guava is built with Java 9 and `ExecutorService` has no
default methods. This changes in Java 19, where `ExecutorService.close`
is added. Ensure users have right expectations about future evolution of
the `ForwardingExecutorService` and its compatibility with newer Java
versions.

Fixes #6296
Fixes #6299

RELNOTES=n/a
PiperOrigin-RevId: 502827344
@findepi findepi deleted the findepi/document-forwardingexecutorservice-behavior-for-default-methods-3b0318 branch January 18, 2023 16:38
@findepi
Copy link
Contributor Author

findepi commented Jan 18, 2023

Thanks @cpovirk for adding the note to other classes, and for the merge!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P2 package=concurrent type=api-docs Change/add API documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support ExecutorService.close in ForwardingExecutorService
2 participants