dstandish commented on code in PR #58289:
URL: https://github.com/apache/airflow/pull/58289#discussion_r2577478846
##########
airflow-core/src/airflow/example_dags/example_outlet_event_extra.py:
##########
@@ -68,6 +68,9 @@ def asset_with_extra_by_context(*, outlet_events=None):
def _asset_with_extra_from_classic_operator_post_execute(context, result):
context["outlet_events"][asset].extra = {"hi": "bye"}
+ # TODO: AIP-76 probably we want to make it so this could be
Review Comment:
my point is to give the task the ability to truthfully record what it does.
so it should be able to add or remove events i'd say. or alter the event. or
suppress the event (i.e. don't make the event) and yeah i guess one of the
bigger differences i'm suggesting is, instead of one and always one event, then
it could potentially be more (e.g. multiple partitions affected in one task) or
none (nothing done)
--
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]
