Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

Chadha93
Copy link
Collaborator

@Chadha93 Chadha93 commented Sep 23, 2025

tested ok:
image


Summary by cubic

Adds configurable concurrency controls for self-hosted crawling to tune throughput and resource usage. Defaults are provided and mapped to the Playwright service with sensible Docker resource limits.

  • New Features

    • New env vars: NUM_WORKERS_PER_QUEUE, CRAWL_CONCURRENT_REQUESTS, MAX_CONCURRENT_JOBS, BROWSER_POOL_SIZE.
    • Playwright uses MAX_CONCURRENT_PAGES derived from CRAWL_CONCURRENT_REQUESTS.
    • Docker deploy limits/reservations added for api and playwright services for more predictable performance.
  • Migration

    • Set desired values in .env (or use defaults) and restart with docker-compose up -d.
    • Optionally adjust CPU/RAM limits to match your host capacity.

Signed-off-by: Chadha93 <gauravchadha1676@gmail.com>
@Chadha93 Chadha93 requested a review from mogery as a code owner September 23, 2025 07:56
@mogery
Copy link
Member

mogery commented Sep 23, 2025

MAX_CONCURRENT_PAGES

Where is this env respected in the playwright worker?

Signed-off-by: Chadha93 <gauravchadha1676@gmail.com>
@Chadha93
Copy link
Collaborator Author

Chadha93 commented Sep 23, 2025

MAX_CONCURRENT_PAGES

Where is this env respected in the playwright worker?

I only tested for API level only, I've now pushed a new commit for MAX_CONCURRENT_PAGES env with respect to playwright worker as well.

lmk if I should update the fallback values, though I've kept it to 10 as general.

@Chadha93
Copy link
Collaborator Author

Hi @mogery, I've updated the PR, ready for review again.

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.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.