rumbin commented on pull request #18098:
URL: https://github.com/apache/superset/pull/18098#issuecomment-1019082927


   Beto, are you aware of this project?
   https://github.com/slidoapp/dbt-superset-lineage
   I haven't tried it, though...
   
   Regarding your approach, I have some doubts on automatically updating the DB 
connection.
   In many cases people may want to use a different user here, or different 
connection strings for the sake of things like user impersonation, proxy 
settings, roles, warehouse settings (Snowflake), etc.
   I know that you are currently only covering Postgres, but for other 
databases it may become troublesome.
   Maybe this step could be made optional?


-- 
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: notifications-unsubscr...@superset.apache.org

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



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org
For additional commands, e-mail: notifications-h...@superset.apache.org

Reply via email to