gopidesupavan commented on PR #42555: URL: https://github.com/apache/airflow/pull/42555#issuecomment-2380880904
> I'm not entirely sure that removing these parameters is the right way to deal with it (then you'll break compatibility with older Airflow versions where it works). @uranusjr / @raphaelauv - I would be happy for your feedback. Thanks for the review. Yes i completely agree. but IMHO keeping these parameters also cause problems here. https://github.com/apache/airflow/blob/main/airflow/models/abstractoperator.py#L768 here the callable value args has only context and jinja_env, if the function does accepts more arguments this would fail ?. Happy for any other suggestions :) -- 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]
