dstandish opened a new pull request #15610:
URL: https://github.com/apache/airflow/pull/15610


   Here I add support for storing connections serialized as JSON instead of 
airflow connection URI.
   
   @mik-laj what do you think of this approach?
   
   I use subclass instead of configuration.
   
   And what I do about the extra field, which has that duality re str vs dict, 
is if it is dict I dump to str, and if str i leave it.
   
   This is an alternative to your approach where user can store under node 
`extra_dejson` if they want to store as dict.
   
   Thanks


-- 
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:
[email protected]


Reply via email to