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 2020/12/14 12:46:48 UTC

[GitHub] [airflow] ashb commented on issue #12480: Support deserialize_yaml in variable get

ashb commented on issue #12480:
URL: https://github.com/apache/airflow/issues/12480#issuecomment-744415753


   Can you explain why we need this, and why json isn't enough?
   
   Mostly I'm not sure I like the the precedent of supporting multiple encoding types for variables.
   
   Accepting YAML for input only _might_ be acceptable, but I think it should then be converted to JSON for storage (see the discussin in #13022) -- we could then also _maybe_ convert it back to YAML for display. But it should be stored as json so there is less code paths to deal with.


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