dimberman commented on a change in pull request #14761:
URL: https://github.com/apache/airflow/pull/14761#discussion_r603537945
##########
File path: airflow/decorators/base.py
##########
@@ -29,7 +29,60 @@
from airflow.utils.task_group import TaskGroup, TaskGroupContext
-class BaseDecoratedOperator(BaseOperator):
+def validate_python_callable(python_callable):
+ """
+ Validate that python callable can be wrapped by operator.
+ Raises exception if invalid.
+
+ :param python_callable: Python object to be validated
+ :raises: TypeError, AirflowException
+ """
+ if not callable(python_callable):
+ raise TypeError('`python_callable` param must be callable')
+ if 'self' in signature(python_callable).parameters.keys():
+ raise AirflowException('@task does not support methods')
Review comment:
@turbaszek what do you mean? I imagine we want to keep parity with how
the task decorator currently works
--
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]
