Skip to content

Conversation

Stebalien
Copy link
Member

Previously, we were limited to 1 message at a time.

@Stebalien Stebalien requested a review from raulk March 14, 2023 20:05
Copy link
Contributor

@arajasek arajasek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, we could consider sanity checking the value (non-zero, have some max, etc.)

@Stebalien
Copy link
Member Author

LGTM, we could consider sanity checking the value (non-zero, have some max, etc.)

Good point.

@Stebalien Stebalien force-pushed the steb/configurable-concurrency branch 3 times, most recently from 0d2664e to 1d50961 Compare March 14, 2023 20:29
Previously, we were limited to 1 message at a time.
@Stebalien Stebalien force-pushed the steb/configurable-concurrency branch from 1d50961 to e922a4b Compare March 14, 2023 20:31
@Stebalien Stebalien merged commit 28e3cd4 into master Mar 14, 2023
Stebalien added a commit that referenced this pull request Mar 14, 2023
…388)

Previously, we were limited to 1 message at a time.
@Stebalien Stebalien deleted the steb/configurable-concurrency branch March 14, 2023 20:35
@scotthconner
Copy link

It doesn't look like it actually defaults to 4? as we changed the lazy static to new_env() versus new()?

Stebalien added a commit that referenced this pull request Apr 7, 2023
…388)

Previously, we were limited to 1 message at a time.
Stebalien added a commit that referenced this pull request Apr 7, 2023
…388)

Previously, we were limited to 1 message at a time.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants