pierrejeambrun commented on PR #65839:
URL: https://github.com/apache/airflow/pull/65839#issuecomment-4343284061
> Forcing it to do GET /dags/{dag_id} purely to extract a token that the
server then decodes back to (bundle_name, relative_fileloc) is a round-trip
that exists only because we expose the parser's internal addressing scheme.
Unfortunately this is how things were implemented in AF2. That's why we kept
it in AF3 to not disrupt / break more the API contract.
https://airflow.apache.org/docs/apache-airflow/2.11.0/stable-rest-api-ref.html#operation/reparse_dag_file
Now it's public API so we can't modify it, that endpoint has to stay.
Adding a duplicated endpoint just to avoid 1 request is not worth it IMO,
both from a use case and maintainability perspective. Also this endpoint is
confusing from a semantic standpoint because you do not request reparse of a
dag, but of a file...
I'm -1 for this change.
--
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]
