shubham22 commented on issue #35490: URL: https://github.com/apache/airflow/issues/35490#issuecomment-1899437696
@syedahsn - confused about which approach you're calling "best" here. The main difference between your config and my suggestion is the merge and replace of _list_ configs that you're doing. My suggestion is that we should only merge configurations when they're not defined in `executor_config` and we should never merge the values of the configurations (including when they are of type _lists_. In your suggestion, I can never get rid of RUN_TASK_KWARGS tags from my task, even if I want to. -- 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
