eladkal commented on code in PR #48006:
URL: https://github.com/apache/airflow/pull/48006#discussion_r2006139220
##########
airflow/models/mappedoperator.py:
##########
@@ -57,6 +60,13 @@ def expand_start_from_trigger(self, *, context: Context,
session: Session) -> bo
:meta private:
"""
+ if self.partial_kwargs.get("start_from_trigger",
self.start_from_trigger):
+ log.warning(
+ "Starting a mapped task from triggerer is currently
unsupported",
+ task_id=self.task_id,
+ dag_id=self.dag_id,
+ )
+ return False
Review Comment:
Should this be warning or exception?
If user is doing `TimeSensorAsync.partial(...,
start_from_trigger=True).expand(...)` wouldn't it be better to show broken dag
message with noting that start_from_trigger must be false in this case
--
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]
