Re: [DISCUSS] Remove `max_active_tasks_per_dag`? Or at least the default

2024-10-04 Thread Jarek Potiuk
So, I propose, DAG.max_active_tasks should be evaluated per-dag-run. And we can change the name accordingly if folks on board. Agree. And possibly we name it DAG.max_active_tasks_per_run J. On Fri, Oct 4, 2024 at 3:30 PM Ryan Hatter wrote: > I think I agree with this: > > I feel it should be

Re: [DISCUSS] Remove `max_active_tasks_per_dag`? Or at least the default

2024-10-04 Thread Ryan Hatter
I think I agree with this: I feel it should be applied at the dag *run* scope > and not across all dag runs. > Just a thought: If someone *did* want to run multiple DAG runs at the same time and limit the max active tasks per DAG, they could create a pool for that DAG and pass the pool in default

Re: [DISCUSS] Remove `max_active_tasks_per_dag`? Or at least the default

2024-10-04 Thread Daniel Standish
Ok, sorry, these concurrency settings are confusing. Let me clarify. `max_active_tasks_per_dag` is a core airflow setting and it provides the default for DAG.max_active_tasks. DAG.max_active_tasks I think is a reasonable config to have but the problem in my view is the scope. I feel it should b

[DISCUSS] Remove `max_active_tasks_per_dag`? Or at least the default

2024-10-04 Thread Daniel Standish
The setting max_active_tasks_per_dag seems mostly useless to me / and footgunish. Why? Because you already have a setting for max active dag runs. If you don't want to run more tasks, don't create the extra dag runs. We also already have a mechanism (param on base operator) for limiting indivi