quinn_batten
01/30/2023, 6:02 PMedgar_ramirez_mondragon
01/30/2023, 6:06 PMchristoph
01/30/2023, 8:46 PMHenning Holgersen
01/30/2023, 9:04 PMchristoph
01/30/2023, 9:37 PMsqlalchemy_url
and make sure that all taps and targets (including future unwritten ones) have first class support of all parameters that the dialect supports. This potentially leads to somewhat exponential growth of the combination of tap, target and dialect.
Option B: Keep sqlalchemy_url
as a supported setting and leave it up to the users to know where to look for parameter names that would be applicable for their chosen dialect.
Option C: Agree on a minimum number of 'sane', common parameters that are needed to configure each dialect - i.e. these https://docs.sqlalchemy.org/en/20/core/engines.html#database-urls
NOTE: Not everything is configurable via a URL parameter in SQL Alchemy. E.g. to pass additional parameters to the DB-API driver's connect()
function, SQL Alchemy Engines need to be passed a connect_args
parameter on creation. Concrete example is Azure Synapse Dedicated SQL Pool (i.e. Microsoft Parallel Data Warehouse in the cloud) https://github.com/BuzzCutNorman/tap-mssql/pull/20
I think that a combination of Option B + C would be a good outcome.