-
Notifications
You must be signed in to change notification settings - Fork 159
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
limit community building concurrency #142
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
❌ Changes requested. Reviewed everything up to 4b3a0d9 in 17 seconds
More details
- Looked at
32
lines of code in1
files - Skipped
0
files when reviewing. - Skipped posting
1
drafted comments based on config settings.
1. graphiti_core/utils/maintenance/community_operations.py:14
- Draft comment:
Typo in constant name:MAX_COMMMUNITY_BUILD_CONCURRENCY
should beMAX_COMMUNITY_BUILD_CONCURRENCY
. - Reason this comment was not posted:
Marked as duplicate.
Workflow ID: wflow_MaeAPe3eyvZstYMy
Want Ellipsis to fix these issues? Tag @ellipsis-dev
in a comment. You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet
mode, and more.
…ing concurrency);
I have read the CLA Document and I hereby sign the CLA You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot. |
CLA Assistant needs a kick. Will investigate further when I get a moment. |
Important
Limits concurrency in
build_communities()
to 10 usingasyncio.Semaphore
incommunity_operations.py
.MAX_COMMMUNITY_BUILD_CONCURRENCY
constant set to 10 incommunity_operations.py
.asyncio.Semaphore
tobuild_communities()
to limit concurrency to 10.build_community()
calls inlimited_build_community()
to enforce concurrency limit.This description was created by
for 4b3a0d9. It will automatically update as commits are pushed.