mik-laj edited a comment on pull request #15013:
URL: https://github.com/apache/airflow/pull/15013#issuecomment-810669033


   > Is there a good reason why extra must be encoded JSON? Seems like a nice 
solution would be to allow for any string there.
   
   We expect the extra field to be a  dictionary in Web UI. The use of 
primitive types also causes a problem with extending a given connection ie. it 
is not future-proff. We also have a design misunderstood here. This field 
should contain extra **fields**, not additional data. 


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