dstandish opened a new pull request, #36297: URL: https://github.com/apache/airflow/pull/36297
When param _request_timeout is not provided, the default is no timeout, which means that sometimes a log read through the kubernetes API may hang forever. See more details [here](https://github.com/kubernetes-client/python/blob/2270ff927e8b5e5340aa7f4cc023293fa4d57eb3/examples/watch/timeout-settings.md). We add the timeout so that connections will be periodically dropped, and there is appropriate logic to resume after disconnect. One thing that's maybe not perfect is we increment the "since_time" a few seconds on resume. Generally this should cause a problem because it will be after a read timeout. We might occasionally lose a couple log lines after a connection timeout. But I don't think it's worth the complexity to avoid that. -- 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]
