turbaszek commented on a change in pull request #11113:
URL: https://github.com/apache/airflow/pull/11113#discussion_r502926517



##########
File path: 
airflow/providers/greatexpectations/operators/greatexpectations_base.py
##########
@@ -0,0 +1,30 @@
+from airflow.models import BaseOperator
+
+
+class GreatExpectationsBaseOperator(BaseOperator):

Review comment:
       Then I would say there's no need for such an operator - it gives us 
nothing currently. We should add the operator / hook only if it brings some 
value, for example creates repository of reusable methods. In this case, this 
may only make future refactoring harder. 




----------------------------------------------------------------
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]


Reply via email to