Skip to content

adds cancellation of the source future support#3146

Merged
OlegDokuka merged 3 commits into3.4.xfrom
enhancement/3.4.x-3138
Aug 15, 2022
Merged

adds cancellation of the source future support#3146
OlegDokuka merged 3 commits into3.4.xfrom
enhancement/3.4.x-3138

Conversation

@OlegDokuka
Copy link
Copy Markdown
Contributor

closes #3138

Signed-off-by: Oleh Dokuka odokuka@vmware.com

Signed-off-by: Oleh Dokuka <odokuka@vmware.com>
Signed-off-by: OlegDokuka <odokuka@vmware.com>
@OlegDokuka OlegDokuka requested a review from a team as a code owner August 10, 2022 10:17
@OlegDokuka OlegDokuka changed the base branch from main to 3.4.x August 10, 2022 10:17
@OlegDokuka OlegDokuka added the type/enhancement A general enhancement label Aug 10, 2022
@OlegDokuka OlegDokuka added this to the 3.4.23 milestone Aug 10, 2022
Oleh Dokuka added 2 commits August 10, 2022 13:34
Signed-off-by: Oleh Dokuka <odokuka@vmware.com>
Signed-off-by: OlegDokuka <odokuka@vmware.com>
Signed-off-by: Oleh Dokuka <odokuka@vmware.com>
Signed-off-by: OlegDokuka <odokuka@vmware.com>
Copy link
Copy Markdown
Contributor

@simonbasle simonbasle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this will likely cause some conflict when merging into main, but not too hard to solve. feel free to merge this @OlegDokuka 👍

@OlegDokuka OlegDokuka merged commit 18ce835 into 3.4.x Aug 15, 2022
@reactorbot
Copy link
Copy Markdown

@OlegDokuka this PR seems to have been merged on a maintenance branch, please ensure the change is merge-forwarded to intermediate maintenance branches and up to main 🙇

@simonbasle
Copy link
Copy Markdown
Contributor

merged into 3.5.0-M6 via commit ebfc60a

@simonbasle simonbasle deleted the enhancement/3.4.x-3138 branch August 16, 2022 08:33
@steven-sheehy
Copy link
Copy Markdown

I believe the Mono.fromFuture() docs are now incorrect after this change:

Note that the future is not cancelled when that Mono is cancelled, but that behavior can be obtained by using a doFinally(Consumer) that checks for a SignalType.CANCEL and calls CompletableFuture.cancel(boolean).

@simonbasle
Copy link
Copy Markdown
Contributor

I believe the Mono.fromFuture() docs are now incorrect after this change:

Note that the future is not cancelled when that Mono is cancelled, but that behavior can be obtained by using a doFinally(Consumer) that checks for a SignalType.CANCEL and calls CompletableFuture.cancel(boolean).

that's a good point 👍
do you want to submit a doc PR to 3.4.x branch @steven-sheehy ?

OlegDokuka added a commit that referenced this pull request Oct 28, 2022
this commits fixes misleading documentation introduced after #3146. Also, this commit adds extra methods overloads which allow suppression of the cancellation propagation to the observed `Future`
chemicL pushed a commit that referenced this pull request Mar 7, 2023
this commits fixes misleading documentation introduced after #3146. Also, this commit adds extra methods overloads which allow suppression of the cancellation propagation to the observed `Future`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type/enhancement A general enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Propagate cancel signal on Mono to wrapped CompletableFuture

5 participants