KEMBAR78
Add default contextView() implem to [Flux|Mono|Synchronous]Sink by simonbasle · Pull Request #3021 · reactor/reactor-core · GitHub
Skip to content

Conversation

@simonbasle
Copy link
Contributor

This commit adds a default implementation to the contextView() method added
in 3.5.0-M1 to the FluxSink, MonoSink and SynchronousSink classes.

@simonbasle simonbasle requested a review from a team as a code owner April 19, 2022 15:12
@simonbasle simonbasle added this to the 3.5.0-M2 milestone Apr 19, 2022
@simonbasle simonbasle added the type/enhancement A general enhancement label Apr 19, 2022
@simonbasle simonbasle self-assigned this Apr 19, 2022
@simonbasle simonbasle requested a review from a team April 19, 2022 15:12
@simonbasle simonbasle changed the title Add a default implementation to the [Flux|Mono|Synchronous]Sink#contextView Add default contextView() implem to [Flux|Mono|Synchronous]Sink Apr 20, 2022
@simonbasle simonbasle merged commit 6eecf85 into main Apr 20, 2022
@simonbasle simonbasle deleted the fluxSinkDefaultContextView branch April 20, 2022 07:44
simonbasle added a commit that referenced this pull request Apr 20, 2022
This commit backports #3021, a default implementation of the
`contextView()` method of the FluxSink, MonoSink and SynchronousSink
"old sinks" classes.

The backport is necessary because the new method was not added only in
3.5.0-M1 as initially stated, but in 3.4.17 as well.

Backport of commit 6eecf85.
See #2971, #2974 and #3021.
simonbasle added a commit that referenced this pull request Apr 20, 2022
This commit backports #3021, a default implementation of the
`contextView()` method of the FluxSink, MonoSink and SynchronousSink
"old sinks" classes.

The backport is necessary because the new method was not added only in
3.5.0-M1 as initially stated, but in 3.4.17 as well.

Backport of commit 6eecf85.
See #2971, #2974 and #3021.
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.

2 participants