Skip to content

Make licensee's max degree of parallism configurable#641

Merged
elrayle merged 7 commits intoclearlydefined:masterfrom
jkbschmid:jkbschmid-licensee-parallel
May 13, 2025
Merged

Make licensee's max degree of parallism configurable#641
elrayle merged 7 commits intoclearlydefined:masterfrom
jkbschmid:jkbschmid-licensee-parallel

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 <elrayle@users.noreply.github.com>
Co-authored-by: E. Lynette Rayle <elrayle@users.noreply.github.com>
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