ashb commented on issue #6643: [AIRFLOW-6040] Fix KubernetesJobWatcher Read 
time out error
URL: https://github.com/apache/airflow/pull/6643#issuecomment-565378569
 
 
   > I can implement it like this but do we really want the timeout to always 
be `1` for any integer >0?
   
   🤦‍♂ No not at all. Broken logic. `max` not `min` was what I wanted. (I.e. to 
ensuere that the timeout never goes negative, and `0` probably means no 
timeout. Maybe we want to allow that but I suspect we don't.)

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to