ashb commented on a change in pull request #12745:
URL: https://github.com/apache/airflow/pull/12745#discussion_r533705611



##########
File path: airflow/models/baseoperator.py
##########
@@ -1371,6 +1371,11 @@ def is_smart_sensor_compatible(self):
         """Return if this operator can use smart service. Default False."""
         return False
 
+    @property
+    def inherits_from_dummy_operator(self):
+        """Used to determine if an Operator is inherited from DummyOperator"""

Review comment:
       ```suggestion
           """Used to determine if an Operator is inherited from 
DummyOperator"""
           # This looks like `isinstance(self, DummyOperator) would work, but 
this also needs to cope when `self` is a Serialized instance of a DummyOperator 
or one of its sub-classes (which don't inherit from anything but BaseOperator).
   ```
   
   But with line wrapping




----------------------------------------------------------------
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:
us...@infra.apache.org


Reply via email to