Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

include boilerplate for logging adapter version to database #41

Open
dataders opened this issue Jun 28, 2022 · 0 comments
Open

include boilerplate for logging adapter version to database #41

dataders opened this issue Jun 28, 2022 · 0 comments
Assignees

Comments

@dataders
Copy link
Contributor

There's a common pattern in adapters wherein the version is sent to a user_agent_string or application_name field when opening a connection that enables the underlying database's product team to have telemetry on the volume of dbt invocations that they're getting. This is normally found in the AdapterConnectionManager.open() method (e.g. DatabricksConnectionManager.open() )

If possible, it would be great to provide at least a template for adapter developers know that it's a good idea to do. Perhaps it can be mentioned in our how to guides in conjunction with unique_field(), which is used for dbt Labs's telemetry

# handle = myadapter_library.connect(
# host=credentials.host,
# port=credentials.port,
# username=credentials.username,
# password=credentials.password,
# catalog=credentials.database
# )

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants