amoghrajesh commented on code in PR #70298:
URL: https://github.com/apache/airflow/pull/70298#discussion_r4044086210
##########
dev/registry/extract_parameters.py:
##########
@@ -530,6 +770,7 @@ def make_entry(
"provider_id": provider_id,
"provider_name": provider_name,
"supports_durable_execution": is_durable_capable(cls_or_obj,
resumable_mixin),
+ "supports_deferrable": supports_deferrable(cls_or_obj),
Review Comment:
Fixed. The decorator entry now carries both fields, computed from the
operator the decorator actually builds. Since the registered `class-name`
points at the factory function rather than an operator, there was nothing to
inspect; a new `_resolve_decorated_operator_class` reads the
`decorated_operator_class=` the factory passes to `task_decorator_factory` and
resolves it against the factory's own module. That convention holds for all 26
decorator modules in the tree.
Also gave the fixture a decorator entry, so `test_all_module_fields_present`
now exercises that path, plus a test asserting the entry reports the built
operator's real capability rather than just having the keys.
Fixed the durable tooltip too, since that line is in this diff: it now says
the flag defaults to True on most but not all operators, instead of stating it
as a flat default.
--
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]
