-
Notifications
You must be signed in to change notification settings - Fork 568
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
Reimplement the Multi.map() operator + TCK test #1411
Reimplement the Multi.map() operator + TCK test #1411
Conversation
This looks unrelated to this PR, perhaps a flaky test?
|
Yes, test is flaky, retrying. Thanks a lot for your contribution! |
@akarnokd I am a manager responsible for project Helidon. I appreciate your contributions. I would like to talk about future cooperation. If you are interested please email me at |
@tomas-langer Yes I did, yesterday before my PRs, and submitted it via the process described. Can you check it if it landed properly (from akarnokd at gmail.com). @m0mus Sure, sent you an email. |
A commit to master removed the reactor version reference in the meantime.
It uses a straight
Publisher
->Publisher
transformation thus there is no need for all thoseProcessor
bells and whistles.The original trusted the
Mapper
to not crash, this doesn't and cancels the sequence properly.