ashb commented on a change in pull request #18724:
URL: https://github.com/apache/airflow/pull/18724#discussion_r722123410



##########
File path: airflow/api_connexion/openapi/v1.yaml
##########
@@ -2863,6 +2863,11 @@ components:
           description: The execution date.
           type: string
           format: datetime
+          deprecated: true

Review comment:
       Given the `include|future` and `include_past` I'm not 100% sure we can 
deprecate this -- does it "make sense" to pass a run_id and then say 
`include_past`, or does that make more sense to stick with execution date?
   
   (I honestly don't have an opinion right now)




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