You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/01/20 15:18:01 UTC

[GitHub] [airflow] vemikhaylov commented on pull request #13790: Add params to the DAG details endpoint

vemikhaylov commented on pull request #13790:
URL: https://github.com/apache/airflow/pull/13790#issuecomment-763702992


   I decided to use the list of objects with "name" and "value" keys instead of the simple dictionary dumping for the following reasons:
   
   1) It's more extensible. For instance, we might want to add description for parameters in the future anyhow, this structure allows us to do so.
   
   2) It's more consistent with some other entities in the API like `Config`, where `ConfigOption` is represented in the same fashion.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org