Skip to content

Fix run config table integration - #1564

Merged
ghanse merged 2 commits into
mainfrom
fix-run-config-table
Oct 8, 2026
Merged

ghanse merged 2 commits into
mainfrom
fix-run-config-table

Conversation

@ghanse

@ghanse ghanse commented Oct 1, 2026

Copy link
Copy Markdown
Collaborator

Changes

This PR fixes an issue with #1547 to remove runner dependency on psycopg.

Linked issues

Relates to #1547

Tests

  • manually tested
  • added unit tests
  • added integration tests
  • added end-to-end tests
  • added performance tests

Documentation and Demos

  • added/updated demos
  • added/updated docs
  • added/updated agent skills

@ghanse
ghanse requested a review from a team as a code owner October 1, 2026 23:31
@ghanse
ghanse requested review from alexott and removed request for a team October 1, 2026 23:31
@codecov

codecov Bot commented Oct 1, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.09%. Comparing base (b645540) to head (543490b).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1564       +/-   ##
===========================================
+ Coverage   79.39%   93.09%   +13.70%     
===========================================
  Files         142      142               
  Lines       14210    14210               
  Branches      151      151               
===========================================
+ Hits        11282    13229     +1947     
+ Misses       2859      912     -1947     
  Partials       69       69               
Flag Coverage Δ
anomaly 50.91% <ø> (ø)
anomaly-serverless 50.92% <ø> (ø)
integration 48.63% <ø> (?)
integration-serverless 47.65% <ø> (?)
mcp 80.34% <ø> (ø)
unit 66.85% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions

github-actions Bot commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

✅ 1097/1097 passed, 52 skipped, 4h43m8s total

Running from acceptance #6126

@github-actions

github-actions Bot commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

✅ 1/1 passed, 28m59s total

Running from mcp #875

@github-actions

github-actions Bot commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

✅ 195/195 passed, 2 flaky, 1 skipped, 7h50m13s total

Flaky tests:

  • 🤪 test_apply_anomaly_check_info_column_structure (2m30.833s)
  • 🤪 test_driver_only_contributions_smoke (3m11.362s)

Running from anomaly #2240

@OGordon100

Copy link
Copy Markdown
Contributor

Have verified, jobs run fine for me.

@ghanse
ghanse requested a review from mwojtyczka October 2, 2026 13:27

@mwojtyczka mwojtyczka left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mwojtyczka mwojtyczka added Approved to Merge When PR is reviewed and approved. To be merged once all tests pass DQX App Feature/Bug related to the DQX App labels Oct 2, 2026
ghanse and others added 2 commits October 8, 2026 10:54
The task runner reads its staged run config over Spark JDBC (serverless
ships the JDBC driver but not psycopg), with a bounded socketTimeout so a
stalled Lakebase endpoint can't hang the run; psycopg is dropped from the
task-runner dependencies. Staged-config cleanup moves to the scheduler's
staged-config sweep (runner no longer self-deletes), and the stale
dq_run_configs table comment is corrected. Includes tests.

Co-authored-by: Marcin Wojtyczka <marcin.wojtyczka@databricks.com>
Co-authored-by: Isaac <no-reply@databricks.com>
@ghanse
ghanse force-pushed the fix-run-config-table branch from a0a1de6 to 543490b Compare October 8, 2026 14:54
@ghanse
ghanse merged commit 4c6c017 into main Oct 8, 2026
31 of 32 checks passed

This branch was successfully deployed

1 active deployment
tool — 543490b1 Deployed Oct 8, 2026 by ghanse via integration_serverless #6126
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Approved to Merge When PR is reviewed and approved. To be merged once all tests pass DQX App Feature/Bug related to the DQX App

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants