You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "pierrejeambrun (via GitHub)" <gi...@apache.org> on 2025/04/01 14:47:00 UTC
[PR] Add AIP-84 newsfragment [airflow]
pierrejeambrun opened a new pull request, #48631:
URL: https://github.com/apache/airflow/pull/48631
closing: https://github.com/apache/airflow/issues/43378
Add AIP-84 newsfragment based on https://github.com/apache/airflow/issues/43378.
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add AIP-84 newsfragment [airflow]
Posted by "pierrejeambrun (via GitHub)" <gi...@apache.org>.
pierrejeambrun merged PR #48631:
URL: https://github.com/apache/airflow/pull/48631
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add AIP-84 newsfragment [airflow]
Posted by "rawwar (via GitHub)" <gi...@apache.org>.
rawwar commented on code in PR #48631:
URL: https://github.com/apache/airflow/pull/48631#discussion_r2023105696
##########
airflow-core/newsfragments/aip-84.significant.rst:
##########
@@ -0,0 +1,50 @@
+Modernize and improve the REST API.
+
+As part of this change the following breaking changes have occurred:
+
+- The API returns 422 status code instead of 400 for validation errors.
+
+ For instance when the request payload, path params, or query params are invalid.
+
+- When listing a resource for instance on GET ``/dags``, ``fields`` parameter is not supported anymore to obtain a partial response.
+
+ The full objects will be returned by the endpoint. This feature might be added back in upcoming 3.x versions.
+
+- Passing list in query parameters switched from ``form, non exploded`` to ``form, exploded``
+ i.e before ``?my_list=item1,item2`` now ``?my_list=item1&my_list=item2``
+
+- ``execution_date`` was deprecated and has been removed. Any payload or parameter mentioning this field has been removed.
+
+- Datetime format are RFC3339-compliant in FastAPI, more permissive than ISO8601,
+ meaning that the API returns zulu datetime for responses, more info here https://github.com/fastapi/fastapi/discussions/7693#discussioncomment-5143311.
+ Both ``Z`` and ``00+xx`` are supported for payload and params.
+
+ This is due FastAPI and pydantic v2 default behavior.
+
+- PATCH on ``DagRun`` and ``TaskInstance`` are more generic and allow in addition to update the resource state to update the note content.
Review Comment:
```suggestion
- PATCH on ``DagRun`` and ``TaskInstance`` are more generic and allow in addition to update the resource state and the note content.
```
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org