Skip to content

fix: async channel cancellation and handler exceptions - #83

Open
andreaTP wants to merge 1 commit into
mainfrom
fix/async-channel-futures
Open

andreaTP wants to merge 1 commit into
mainfrom
fix/async-channel-futures

Conversation

@andreaTP

Copy link
Copy Markdown

Replaces the CompletableFuture wrapper with a FutureTask: cancel now stops or interrupts the task, callers can't complete the future, and exceptions thrown by completion handlers reach the executor thread again (as with Guava).

Futures returned by the async channel ignored cancel, could be completed by
callers, and exceptions thrown by completion handlers were silently lost.
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