Taragolis commented on PR #24153:
URL: https://github.com/apache/airflow/pull/24153#issuecomment-1217739505

   > It’s better to keep the arguments there since they still need to support 
old Airflow versions for now. A better fix would be to remove filename_template 
from airflow_local_settings where those handlers are used.
   
   Yeah, I also think better to set `filename_template` to None in 
`airflow_local_settings` so it won't affect old version of providers which have 
`filename_template` as mandatory arg.
   
   I would create a PR


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to