amoghrajesh commented on code in PR #70291:
URL: https://github.com/apache/airflow/pull/70291#discussion_r3637433761
##########
dev/registry/tests/test_extract_parameters.py:
##########
@@ -225,6 +225,24 @@ def execute(self, context):
return None
+class ManuallyDurableOperator:
+ """Implements durable execution directly (e.g. via task_state_store),
without
+ ResumableJobMixin -- mirrors KubernetesPodOperator/AgentOperator."""
+
+ __supports_durable_execution = True
Review Comment:
There's a genuine reason actually. Two things:
1. We do not want users to change it
2. We do not want the inherited operators to get the property by itself, for
example - KPO has `__supports_durable_execution` set but I wouldnt want
SparkKPO to have it as well just cos it inherits KPO, we do not know if
SparkKPO or any inherited operator _really supports_ durable execution
--
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]
