shubham22 commented on issue #35490: URL: https://github.com/apache/airflow/issues/35490#issuecomment-1857074044
I'd vote for simplicity and keeping things consistent irrespective of the type of the field, which means `executor_config` should _always_ override the `run_task_kwargs` for any field specified in both. If the user really cares about passing the tags that they've already defined in `run_task_kwargs`, then they can mention them again. If we go with additive approach, it is another documentation that a user _must_ read and they wouldn't have a way to remove certain tags. Yes, you can design the system with ~ or something, but that's unnecessary logic on top of the logic (additive) which is already inconsistent. -- 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
