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 2021/12/05 15:13:20 UTC

[GitHub] [airflow] potiuk commented on issue #20042: Lack of parameters for postgresql image.

potiuk commented on issue #20042:
URL: https://github.com/apache/airflow/issues/20042#issuecomment-986247484


   I still think it's not a good idea to add a change like that - having options would encourage users to use this "not recommended" setting. I think we shoudl really treat the postgres support "as is" just for quick start but having options on that is not a good idea (because we'd have to support all different options there). 
   
   If you want to experiment with different images in your local setting, there is no problem with downloading the chart and replacing it directly in the chart, which gives you a nice development-friendly option, so you are not "left alone" - there is already a possibility of modifying the image for dev settings.
   
   I will convert it into a discussion - maybe there is a good reason and someone would really think it's a good idea, but I think we should not implement it.  


-- 
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-unsubscribe@airflow.apache.org

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