amoghrajesh commented on PR #51930:
URL: https://github.com/apache/airflow/pull/51930#issuecomment-2990513000

   > Instead of as_uri can we put things as json?
   
   The reason i didnt choose `json` was because the extra column in Connections 
is stored as `Text` (which as per postgres docs: `In addition, PostgreSQL 
provides the text type, which stores strings of any length)` so it potentially 
could overload the os environment variables size wise (i believe linux has a  
limit of 128KiB), so i decided against using json as its longer.
   
   Do you think it would benefit from using json here?


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

To unsubscribe, e-mail: commits-unsubscr...@airflow.apache.org

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

Reply via email to