[GitHub] ashb commented on issue #4325: [AIRFLOW-3528] handle socket exception with SFTPOperator

2018-12-18 Thread GitBox
ashb commented on issue #4325: [AIRFLOW-3528] handle socket exception with SFTPOperator URL: https://github.com/apache/incubator-airflow/pull/4325#issuecomment-448247831 Okay, thanks for trying though! This is an automated

[GitHub] ashb commented on issue #4325: [AIRFLOW-3528] handle socket exception with SFTPOperator

2018-12-18 Thread GitBox
ashb commented on issue #4325: [AIRFLOW-3528] handle socket exception with SFTPOperator URL: https://github.com/apache/incubator-airflow/pull/4325#issuecomment-448241004 I guessed at what error it might report. The main thing is you have try-inside-try, so this won't do what you expect.

[GitHub] ashb commented on issue #4325: [AIRFLOW-3528] handle socket exception with SFTPOperator

2018-12-18 Thread GitBox
ashb commented on issue #4325: [AIRFLOW-3528] handle socket exception with SFTPOperator URL: https://github.com/apache/incubator-airflow/pull/4325#issuecomment-448236255 This fix I don't think will do what we want, as I think now this will end up ultimately logging this as a message: