kalluripradeep commented on PR #61338:
URL: https://github.com/apache/airflow/pull/61338#issuecomment-3833779561

   > Thanks for the PR.
   > 
   > However, AF3 is much more complicated when we want to support a new field 
for `TrigerDagRunOperator` compare to the current PR change.
   > 
   > You can refer to #60810, which is a good example. to
   
   Thanks @jason810496 for the feedback! 
   
   I see that supporting `run_after` requires updates across the entire stack 
similar to #60810. I'll update the PR to include:
   
   1.  `TriggerDagRunOperator` changes (already done)
   2. `DagRunTriggerException` - add `run_after` parameter  
   3. `TriggerDAGRunPayload` - add `run_after` field
   4. API routes and datamodels
   5. task_runner and supervisor updates
   6. Comprehensive tests
   7. Generate OpenAPI spec
   
   I'll model the changes after #60810's implementation pattern. Will update 
shortly!


-- 
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]

Reply via email to