You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I'm investigating the defaults of uv wrt to large projects. From https://github.com/search?q=UV_HTTP_TIMEOUT&type=code&p=1, it seems airflow is the main OSS user of setting UV_HTTP_TIMEOUT to a non-default value. I've seen that in #38467, airflow added UV_HTTP_TIMEOUT (or UV_REQUEST_TIMEOUT, as it was initially called). At the point the PR was made, the uv timeout was a full request timeout. After that PR was merged, uv switched to a read timeout, i.e. a timeout only when no data was transferred: UV_REQUEST_TIMEOUT.
I was wondering, with the timeout now being a read timeout, does airflow still need to set UV_HTTP_TIMEOUT, is the default in uv too low?
reacted with thumbs up emoji reacted with thumbs down emoji reacted with laugh emoji reacted with hooray emoji reacted with confused emoji reacted with heart emoji reacted with rocket emoji reacted with eyes emoji
Uh oh!
There was an error while loading. Please reload this page.
-
I'm investigating the defaults of uv wrt to large projects. From https://github.com/search?q=UV_HTTP_TIMEOUT&type=code&p=1, it seems airflow is the main OSS user of setting
UV_HTTP_TIMEOUTto a non-default value. I've seen that in #38467, airflow addedUV_HTTP_TIMEOUT(orUV_REQUEST_TIMEOUT, as it was initially called). At the point the PR was made, the uv timeout was a full request timeout. After that PR was merged, uv switched to a read timeout, i.e. a timeout only when no data was transferred: UV_REQUEST_TIMEOUT.I was wondering, with the timeout now being a read timeout, does airflow still need to set
UV_HTTP_TIMEOUT, is the default in uv too low?CC @potiuk
Beta Was this translation helpful? Give feedback.
All reactions