Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Uni.join().first(unis).toTerminate() must not support concurrency limit #830

Closed
jponge opened this issue Jan 28, 2022 · 1 comment · Fixed by #831
Closed

Uni.join().first(unis).toTerminate() must not support concurrency limit #830

jponge opened this issue Jan 28, 2022 · 1 comment · Fixed by #831
Labels
enhancement New feature or request
Milestone
1.4.0

Comments

@jponge
Copy link
Member

jponge commented Jan 28, 2022

It does not make sense to join on the first Uni to signal (item or failure) with a concurrency limit.

The currently unreleased API must throw an exception when a concurrency limit has been set, or be reworked to avoid expressing such an intent.
@jponge
Copy link
Member Author

jponge commented Jan 28, 2022

See #825 #829

@jponge jponge added this to the 1.4.0 milestone Jan 28, 2022
@jponge jponge linked a pull request Jan 28, 2022 that will close this issue
Avoid Uni.join().first(unis).toTerminate() with a concurrency limit #831
Merged
@jponge jponge added the enhancement New feature or request label Jan 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
1.4.0
Development

Successfully merging a pull request may close this issue.

Avoid Uni.join().first(unis).toTerminate() with a concurrency limit jponge/smallrye-mutiny
1 participant
@jponge