Skip to content

feat: add POST_RUN outcome telemetry - #1648

Draft
saishreeeee wants to merge 7 commits into
databricks:1.13.latestfrom
saishreeeee:telemetry-post-run
Draft

feat: add POST_RUN outcome telemetry#1648
saishreeeee wants to merge 7 commits into
databricks:1.13.latestfrom
saishreeeee:telemetry-post-run

Conversation

@saishreeeee

@saishreeeee saishreeeee commented Aug 22, 2026

Copy link
Copy Markdown
Collaborator

Description

  • Adds opt-in POST_RUN outcome telemetry on top of feat: add opt-in POST_PARSE adapter telemetry #1647.
  • Captures invocation status, termination reason, duration, and node/hook counts.
  • Handles fail-fast and ephemeral-node result reconciliation.
  • Flushes pending telemetry safely during cleanup.

Testing

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

Ship the default-off send path and privacy-reduced parse event so reviewers can inspect transport, flag handling, and payload shape without run-outcome aggregation.
Layer run-outcome aggregation, event listening, and flush/close onto the opt-in POST_PARSE send path.
@github-actions

github-actions Bot commented Aug 22, 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 530
  handle.py
  impl.py 910-911, 914-915
  dbt/adapters/databricks/telemetry
  __init__.py
  builder.py 107, 153-162, 166-171, 175-177, 217-218, 227-234, 243-252, 258-259, 302, 322, 354-356
  client.py 19, 30, 36, 56, 59-60, 62-63
  config.py 11, 17, 19, 30-31, 41
  coordinator.py 106, 130-132, 147-148, 157, 167, 173-177, 180-184, 190, 203, 253-258, 299, 349, 416
  encoder.py
  hooks.py 19-25, 38-45, 55, 60-69, 88, 127-128, 138, 141
  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 draft August 22, 2026 12:28
Do not join the daemon send at cleanup, coerce workspace identity before the HTTP header, and classify compute paths with or without a leading slash.
…d sends

Repeated dbtRunner invokes reset the process-global invocation ID before leftover adapter cleanup, and close() after set_post_run can drop an already-scheduled event.
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