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.

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

Uni.{join,combine} shall limit concurrency #699

Closed
jponge opened this issue Oct 4, 2021 · 6 comments · Fixed by #829
Closed

Uni.{join,combine} shall limit concurrency #699

jponge opened this issue Oct 4, 2021 · 6 comments · Fixed by #829
Labels
enhancement New feature or request good first issue Good for newcomers
Milestone

Comments

@jponge
Copy link
Member

jponge commented Oct 4, 2021

Arbitrarily large sets of Uni can result in too many concurrent connections in certain contexts.

We shall provide a limit, just like for async operations and flatmap-isms.

@jponge jponge added enhancement New feature or request good first issue Good for newcomers labels Oct 4, 2021
@jponge jponge added this to the 1.3.0 milestone Nov 18, 2021
@jponge jponge modified the milestones: 1.3.0, 1.4.0 Jan 3, 2022
@ashwinair
Copy link

Hi @jponge, is this issue still open, can I work on this?

@jponge
Copy link
Member Author

jponge commented Jan 18, 2022 via email

@ashwinair
Copy link

ashwinair commented Jan 19, 2022

You can certainly have a look.

sorry @jponge for replying you late, well i'm new here that's why i was learning about reactive programming and Mutiny before i start to contribute.

Can you please point me to the right direction, so that i get a rough idea about from where i can get started for this issue.

@jponge
Copy link
Member Author

jponge commented Jan 19, 2022 via email

@ashwinair
Copy link

ashwinair commented Jan 19, 2022

If you are only learning about reactive programming then implementing / making changes to an operator in Mutiny might be a too big step.

I’d suggest you become familiar with Mutiny, for instance by using it in Quarkus apps, then eventually once you have a solid understanding it might be more approachable to make changes in Mutiny itself.

Sure as you suggested, i will first create some quarkus app using mutiny to get more familiar with it.

Thank you 😊

@jponge
Copy link
Member Author

jponge commented Jan 25, 2022

Uni.join() #825

@jponge jponge linked a pull request Jan 28, 2022 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants