ashb commented on code in PR #67506:
URL: https://github.com/apache/airflow/pull/67506#discussion_r3353493492


##########
task-sdk/src/airflow/sdk/api/client.py:
##########
@@ -1136,6 +1136,13 @@ def __init__(self, *, base_url: str | None, dry_run: 
bool = False, token: str, *
         # Set timeout if not explicitly provided
         kwargs.setdefault("timeout", API_TIMEOUT)
 
+        # Cap the httpx connection pool. Without an explicit value httpx 
defaults to 100 max
+        # connections, which is far higher than a single task subprocess ever 
needs (the
+        # supervisor sets ``max_connections=10`` in ``_ensure_client``). A 
bounded default

Review Comment:
   This code is only run in the supervisor, so this comment makes no sense



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to