o-nikolas commented on PR #40778: URL: https://github.com/apache/airflow/pull/40778#issuecomment-2229221767
> > For Airflow 3 (when we can easily make breaking changes) we'll update the metrics to include the executor name in all cases always. > > What about introducing a setting that will control over which behavior is used and for Airflow 3 we will eliminate the setting and make the new way (with executor name) the default one? This way we can implement everything now and not have a tech debt later. > > WDYT? I think Airflow has too many configurations as it is 😅 Given the short period of time between now and Airflow 3 I don't think it's worth the complexity. Either way we'll have to circle back and make a code change (either remove that configuration logic or make the small code change to modify the behaviour) so I don't think it buys us too much, personally. -- 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
