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/13 23:24:06 UTC

[GitHub] [airflow] dstandish edited a comment on pull request #13050: airflow should use same JSONEncoder as flask

dstandish edited a comment on pull request #13050:
URL: https://github.com/apache/airflow/pull/13050#issuecomment-744086704


   maybe it makes sense to leave dag serialization always with `json`, assuming it doesn't have any interactions with flask.
   
   however, given its location and naming (`from airflow.settings import json`), which makes it seem "global", it perhaps could lead to confusion.  e.g. could be named as `dag_serde_json` or something... but this is above my pay grade :) 


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