pierrejeambrun commented on code in PR #73322:
URL: https://github.com/apache/airflow/pull/73322#discussion_r4104973101
##########
airflow-core/src/airflow/api_fastapi/core_api/routes/public/dag_versions.py:
##########
@@ -132,3 +138,76 @@ def get_dag_versions(
dag_versions=dag_versions,
total_entries=total_entries,
)
+
+
+@dag_versions_router.get(
+ "/{base_version_number}/diff/{target_version_number}",
+ responses=create_openapi_http_exception_doc(
+ [
+ status.HTTP_400_BAD_REQUEST,
+ status.HTTP_404_NOT_FOUND,
+ ]
+ ),
+ dependencies=[Depends(requires_access_dag(method="GET",
access_entity=DagAccessEntity.VERSION))],
+ # Serializing unset fields as null would erase the absent-vs-null
distinction the value
+ # fields document.
+ response_model_exclude_unset=True,
+)
+def get_dag_version_diff(
Review Comment:
Should this be public API or UI only at this point. I would put this in the
UI for now so we can still update / break it while we stabilize the Rest API
interface in the couple of next iteration. Then we can move it to public when
we feel it's stable.
--
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]
