dstaple commented on PR #15667: URL: https://github.com/apache/airflow/pull/15667#issuecomment-1505517928
I noticed that @apply_defaults is deprecated but it was left with the following comment: "Make it still be a wrapper to keep the previous behaviour of an extra stack frame" https://github.com/apache/airflow/blob/2.5.3/airflow/utils/decorators.py#L52-L54 It seems to me the number of scenarios where this will make a difference is relatively limited, but considering you chose to do it this way, I'm wondering if there are situations where a user cannot expect to simply remove the `@apply_defaults` decorator to resolve this DeprecationWarning? -- 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]
