Skip to content

Make licensee's max degree of parallism configurable #641

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

Merged
merged 7 commits into from
May 13, 2025

Conversation

jkbschmid
Copy link
Contributor

via CRAWLER_LICENSEE_PARALLELISM.
The default of 10 stays as is.

Fixes #640

via `CRAWLER_LICENSEE_PARALLELISM`.
The default of 10 stays as is.
Copy link
Collaborator

@elrayle elrayle left a comment

Choose a reason for hiding this comment

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

Always good to provide more control to be able to tweak how the system runs and the resources it uses. Thanks for submitting this!

I have a suggestion to ease long term maintenance.

jkbschmid and others added 2 commits May 9, 2025 17:46
Co-authored-by: E. Lynette Rayle <[email protected]>
Co-authored-by: E. Lynette Rayle <[email protected]>
Copy link
Collaborator

@elrayle elrayle left a comment

Choose a reason for hiding this comment

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

Thanks again for adding this config. Looks good.

@elrayle elrayle merged commit 07d267f into clearlydefined:master May 13, 2025
2 checks passed
@qtomlinson
Copy link
Collaborator

@jkbschmid @elrayle Thank you for the contribution!

Could you document this new addition in a manner similar to this commit?

@jkbschmid
Copy link
Contributor Author

@qtomlinson Thanks for the reminder, I opened clearlydefined/operations#126.

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.

Make licensee's degree of parallelism configurable
3 participants