Re: [PR] Added `output_processor` parameter to `BashProcessor` [airflow]

["shahar1 (via GitHub)" <gi...@apache.org>]

shahar1 commented on code in PR #40843:
URL: https://github.com/apache/airflow/pull/40843#discussion_r1702231827


##########
airflow/operators/bash.py:
##########
@@ -63,6 +63,8 @@ class BashOperator(BaseOperator):
         If None (default), the command is run in a temporary directory.
         To use current DAG folder as the working directory,
         you might set template ``{{ dag_run.dag.folder }}``.
+    :param result_processor: Function to further process the result of the bash script

Review Comment:
   I'm a bit confused - do you agree with the renaming? (it is not implemented in practice)



-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org