Skip to content

feat: add wiring for telemetry v1 - #1620

Draft
saishreeeee wants to merge 12 commits into
databricks:1.13.latestfrom
saishreeeee:telemetry
Draft

feat: add wiring for telemetry v1#1620
saishreeeee wants to merge 12 commits into
databricks:1.13.latestfrom
saishreeeee:telemetry

Conversation

@saishreeeee

@saishreeeee saishreeeee commented Jul 29, 2026

Copy link
Copy Markdown
Collaborator

Description:
Adds an opt-in, default-off telemetry subsystem for the Databricks adapter. Enable with enable_dbt_telemetry: true in connection_parameters.

  • Two events per eligible invocation (build, run, test, seed, and snapshot only): POST_PARSE contains invocation configuration, manifest resource counts, connection shape, and behavior flags; POST_RUN contains status, termination reason, node and hook result counts, and duration.
  • Privacy: no credentials, hostnames, cluster IDs, or object names. Auth is reduced to an enum family, http_path to compute type, and only a non-PII workspace ID is sent.
  • Best-effort and async: sends run on a background thread with a 10-second request timeout and never block a SQL connection. Run-end waits up to two request timeouts for queued events. Failures are ignored and never affect the dbt command.

Testing:
(will check whether telemetry logs land in the dbt telemetry table)

@saishreeeee
saishreeeee marked this pull request as draft July 29, 2026 10:16
@saishreeeee saishreeeee changed the title feat: add opt-in initial connection telemetry log feat: add initial connection telemetry log Jul 29, 2026
@saishreeeee
saishreeeee marked this pull request as ready for review July 29, 2026 10:55
@sd-db
sd-db marked this pull request as draft July 29, 2026 12:32
@saishreeeee saishreeeee changed the title feat: add initial connection telemetry log feat: add wiring for telemetry v1 Aug 13, 2026
@github-actions

github-actions Bot commented Aug 13, 2026

Copy link
Copy Markdown

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  dbt/adapters/databricks
  connections.py 528
  handle.py
  impl.py 909-910, 913-914
  dbt/adapters/databricks/telemetry
  __init__.py
  builder.py 105, 151-160, 164-169, 173-175, 201-202, 211-218, 227-236, 242-243, 286, 306, 338-340
  client.py 16-19, 28-61
  config.py 11, 17, 19, 30-31, 41
  coordinator.py 106, 125-127, 130-132, 142-143, 152, 162, 168-172, 175-179, 185, 198, 248-253, 297, 306, 309, 341, 395
  encoder.py
  hooks.py 14-20, 28-34, 44, 49-58, 74-82, 91-109, 113-114, 120-130
  listener.py 7-9, 13-18, 34, 37, 49-50, 52, 54, 61-63, 71-79
  models.py
Project Total  

This report was generated by python-coverage-comment-action

@saishreeeee
saishreeeee marked this pull request as ready for review August 20, 2026 12:53
@saishreeeee
saishreeeee marked this pull request as draft August 21, 2026 06:06
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

Successfully merging this pull request may close these issues.

1 participant