potiuk commented on PR #30176:
URL: https://github.com/apache/airflow/pull/30176#issuecomment-1845330672

   > I would argue if you need logic it is better to create a shell script file 
and use templating to pass in the values instead. Trying to generate a shell 
command with Python code—or more generally, ad-hoc code generation by hand—is a 
very good way to shoot yourself in the foot.
   
   I do not think about much logic - I think it's good to give our users a 
versaitle set of tools. I find ot really easy to have some patterns usable. 
Simple `if x return "this script" else return  "other script"` is much better 
if the logic is done in Python for example.


-- 
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-unsubscr...@airflow.apache.org

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

Reply via email to