Skip to content

Conversation

6matt
Copy link

@6matt 6matt commented Oct 7, 2025

Title

Remove TCP connector limit

Pre-Submission checklist

  • I have Added testing in the tests/litellm/ directory, Adding at least 1 test is a hard requirement - see details
  • I have added a screenshot of my new test passing locally
  • My PR passes all unit tests on make test-unit
  • My PR's scope is as isolated as possible, it only solves 1 specific problem
Screenshot 2025-10-06 at 8 26 54 PM

Type

🐛 Bug Fix

Changes

  • Remove limit on concurrent TCP connections
    • aiohttp.TCPConnector has a default limit of 100 - this could become a bottleneck in case of high concurrency or long-running requests

Copy link

vercel bot commented Oct 7, 2025

@6matt is attempting to deploy a commit to the CLERKIEAI Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Contributor

@ishaan-jaff ishaan-jaff left a comment

Choose a reason for hiding this comment

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

@AlexsanderHamir are there any implications for this perf wise ?

@ishaan-jaff
Copy link
Contributor

I think we can also tune the connector for low latency / high perf while we're at it @AlexsanderHamir

@AlexsanderHamir
Copy link
Collaborator

think we can also tune the connector for low latency / high perf while we're at it @AlexsanderHamir

Definitely, it's been an ongoing issue.

@6matt
Copy link
Author

6matt commented Oct 7, 2025

@ishaan-jaff are you requesting any specific changes from me, or just blocking this PR?

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