This code has been reviewed and submitted internally. Feel free to discuss on
the PR, and we can submit follow-up changes as necessary.
Commits:
Migrate from soon-to-be-deprecated addListener(listener) to equivalent addListener(listener, directExecutor()).
The 1-arg overload is being removed: directExecutor() is often useful, but it should be an explicit choice, as some usages are dangerous:
https://guava.dev/releases/snapshot-jre/api/docs/com/google/common/util/concurrent/ListenableFuture.html#addListener-java.lang.Runnable-java.util.concurrent.Executor-
ef772c0dc891ff994309433e8e90ad01531c4d32
This code has been reviewed and submitted internally. Feel free to discuss on the PR, and we can submit follow-up changes as necessary.
Commits:
Migrate from soon-to-be-deprecated addListener(listener) to equivalent addListener(listener, directExecutor()). The 1-arg overload is being removed: directExecutor() is often useful, but it should be an explicit choice, as some usages are dangerous: https://guava.dev/releases/snapshot-jre/api/docs/com/google/common/util/concurrent/ListenableFuture.html#addListener-java.lang.Runnable-java.util.concurrent.Executor- ef772c0dc891ff994309433e8e90ad01531c4d32