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

Bug Report - High Memory Usage when Providing Float to Concurrency #2619

Closed
abdeljalil09 opened this issue Jun 30, 2023 · 2 comments
Closed

Comments

@abdeljalil09
Copy link
Contributor

Description:
I encountered an issue while using the Bull.js module in Nest.js. When I provided a float value (5 / 2, which is 2.5) to the concurrency property in the processors configuration, I noticed a significant increase in memory usage.

Expected Behavior:
The concurrency value should be rounded to the nearest integer, as is the common practice in similar scenarios.

Steps to reproduce
Actual Results:
Upon providing 5 / 2 as the value for concurrency, the memory usage increases significantly, causing potential performance and stability issues.

Expected behavior
I would expect the concurrency value to be rounded to the nearest integer, resulting in a value of 3 in this case (Math.round(5 / 2)).

@manast
Copy link
Member

manast commented Jun 30, 2023

I think it is better to throw an exception in this case actually.

@stale
Copy link

stale bot commented Sep 4, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Sep 4, 2023
@stale stale bot closed this as completed Sep 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants