dbt-labs / dbt-adapters

Apache License 2.0
28 stars 38 forks source link

Add adapter telemetry method for core model run snowplow event. #328

Closed VersusFacit closed 3 weeks ago

VersusFacit commented 1 month ago

resolves #301 docs dbt-labs/docs.getdbt.com/#

Problem

We need a way to expose adapter telemetry for our internal product adoption analytics.

Solution

The base adapter provides a simple method which is used in dbt core and its tracking system. This method will be overwritten for each adapter that we wish to collect telemetry for.

Checklist

github-actions[bot] commented 1 month ago

Thank you for your pull request! We could not find a changelog entry for this change. For details on how to document a change, see the contributing guide.

VersusFacit commented 1 month ago

Proof of operation for fallback method (i.e. adapter does not override)

image

Note adapter_details is {}. It'll be non-empty in, say, snowflake (here) which provides an override with snowflake-specific info. This field is optional in our JSON schema.

colin-rogers-dbt commented 1 month ago

Is this unit test-able?