dstandish edited a comment on pull request #13423: URL: https://github.com/apache/airflow/pull/13423#issuecomment-774741417
hey @turbaszek @ashb @kaxil rebased after merge of [BashOperator to raise AirflowSkipException on exit code 127](https://github.com/apache/airflow/pull/13421) that change has implications for this one what i have done for now is keep the skip logic in the `run_command` function, along with the fail logic. this means that the skip logic would stay with the hook i feel a little iffy about this. i think best if the skip logic is handled in the operator, that the hook should not have an opinion about this. so what i'm thinking is add `namedtuple('SubprocessResult', ['exit_code', 'output')` and then let the caller decide what to do. so in `run_command` it would not raise _any_ exception based on exit code -- it would just provide code along with output please let me know your thoughts. small note: i think will be nice to parameterize returning all output vs last line but this can be added in later pr. ---------------------------------------------------------------- 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]
