turbaszek commented on a change in pull request #9784:
URL: https://github.com/apache/airflow/pull/9784#discussion_r453371305
##########
File path: airflow/utils/decorators.py
##########
@@ -21,13 +21,16 @@
import os
from copy import copy
from functools import wraps
+from typing import Callable, TypeVar, cast
from airflow.exceptions import AirflowException
signature = inspect.signature
+T = TypeVar('T', bound=Callable) # pylint: disable=invalid-name
Review comment:
Not sure I understand it. I wondering if removing the `@apply_defulats`
will solve the same issue? I think now we have `BaseOperatorMeta` we can
consider removing this decorator.
##########
File path: airflow/utils/decorators.py
##########
@@ -21,13 +21,16 @@
import os
from copy import copy
from functools import wraps
+from typing import Callable, TypeVar, cast
from airflow.exceptions import AirflowException
signature = inspect.signature
+T = TypeVar('T', bound=Callable) # pylint: disable=invalid-name
Review comment:
Not sure I understand it. I'm wondering if removing the
`@apply_defulats` will solve the same issue? I think now we have
`BaseOperatorMeta` we can consider removing this decorator.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
