rawwar commented on code in PR #37165:
URL: https://github.com/apache/airflow/pull/37165#discussion_r1477281155
##########
airflow/operators/python.py:
##########
@@ -403,6 +399,25 @@ def get_python_source(self):
"""Return the source of self.python_callable."""
return textwrap.dedent(inspect.getsource(self.python_callable))
+ def _validate_python_callable(self, python_callable):
Review Comment:
I initially had an idea of suggesting a common validation method that is
invoked for any kind of operator. This can help us to place all the
pre-requisite checks w.r.t any operator. Probably enforce it to be implemented
in the inherited classes by making it an abstract method. But, for now, I've
wrote this as I don't understand the codebase enough.
--
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]
