Usiel commented on PR #32709:
URL: https://github.com/apache/airflow/pull/32709#issuecomment-1651096924
It's all green again on my side. Now, the fixes in the last commit do have
some effects on the public API (`/api/experimental/pools` and `/api/v1/pools`):
`include_deferred` is a required field according to the model, so API calls
without it will fail.
I could use some guidance on this, before I introduce more changes. Do we
prefer to follow the model's constraints here or do we want to try to keep
backwards compatibility?
I'd go for keeping backwards compatibility for `/api/v1/pools` and spit out
a warning if the newly required `include_deferred` is not set, since the stable
API docs say:
```
- APIs are designed to be backward compatible.
```
Do we want to do the same for the experimental (deprecated) API? Or should
we not do any changes to it at all - except setting a default
`include_deferred` to make it work?
--
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]
