daniel_walker
04/27/2021, 9:34 AMdaniel_walker
04/27/2021, 2:12 PMdouwe_maan
04/27/2021, 3:21 PMmeltano variant of target-postgres in mind, and when we deprioritized everything related to the UI/Analysis and made the datamill-co variant the default instead, we didn't go back to update the transforms and models. While not a priority for us right now, it's definitely worth tracking in issues, so if you could create those that'd be appreciated. I wonder if it would be possible to have the transforms work with all variants of the loader, by branching based on the actual loader used, which I think should be available through environment variables Meltano inserts into dbt's execution environment.
As for The dialect 'postgres' is not yet supported by meltano analyze , that sounds like an actual bug we should also track in an issue and resolve. I think the error originates here: https://gitlab.com/meltano/meltano/-/blob/master/src/meltano/core/connection_service.py#L63, since the template "postgresql://{user}:{password}@{host}:{port}/{dbname}" expects a user key, while the config for datamill-co uses postgres_user instead. We'd need to make that not dependent on the actual setting names used.