potiuk commented on PR #30113: URL: https://github.com/apache/airflow/pull/30113#issuecomment-1478383101
> > Yes. With the exception of backwards compatibility. If removal of the state would cause previous requests to fail (Even wit t state "queued", then we should not remove it just make it optional and raise deprecation warning in case someone passes it.. > > Agreed, but in this case requests which set the `state` field would previously fail with a 400, see example curl in the initial issue (#30075), so I think all we need to do here is fix up the spec + documents as per #30149 Ah. I see. yeah. Then it makes sense. -- 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]
