punx120 commented on issue #29282: URL: https://github.com/apache/airflow/issues/29282#issuecomment-1473863561
I wrote the PR you refer too. One issue I have now is that the timeout cannot be specified per SSH Operator, but only per Hook. In my dag, I usually create the hook once, and then use it in different operators: ``` hook = SSHHook(...) op1 = SSHOperator(hook, cmd_timeout=20) op2 = SSHOperator(hook, cmd_timeout=60) ``` I think there's value in having cmd_timeout per operator. You also mention `conn_timeout `, not sure if it's used or not, but from the name, it should only be used to establish the connection. -- 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]
