etsy / boundary-layer

Builds Airflow DAGs from configuration files. Powers all DAGs on the Etsy Data Platform
Apache License 2.0
262 stars 58 forks source link

Support TriggerDagRunOperator #43

Open y2k-shubham opened 4 years ago

y2k-shubham commented 4 years ago
mchalek commented 4 years ago

hi @y2k-shubham , and thanks for the question. It is a good point about the difficulty of verifying the cross-dag dependencies that are introduced by these operators. At Etsy we have had a longstanding plan to add cross-dag dependency checking to our deployment infrastructure, but it's still not done (and we have had our fair share of errors due to incorrect ExternalTaskSensor specifications as a consequence).

However, given that we do support ExternalTaskSensor, it seems reasonable to me that we could also support TriggerDagRunOperator as well. We would only have to add a configuration for this operator here (and if you would like to do this, we would be happy for the contribution!).

Thanks for the reminder about the need for the cross-dag checking. With the holiday season coming up, we should have more down-time than usual; perhaps opportunities to add this will present themselves.