Skip to content

Remove Beam SQL Data Catalog provider - #40663

Merged
Abacn merged 2 commits into
apache:masterfrom
aIbrahiim:remove-sql-datacatalog
Oct 19, 2026
Merged

Abacn merged 2 commits into
apache:masterfrom
aIbrahiim:remove-sql-datacatalog

Conversation

@aIbrahiim

Copy link
Copy Markdown
Contributor

GCP Data Catalog is gone, so the Beam SQL datacatalog module no longer works and was failing PostCommit SQL. This removes that module (and its Pre/PostCommit wiring)

Fixes: #35514

@aIbrahiim aIbrahiim changed the title remove Beam SQL Data Catalog provider Remove Beam SQL Data Catalog provider Oct 19, 2026
@codecov

codecov Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 65.25%. Comparing base (76f6e6d) to head (4829ac5).
⚠️ Report is 239 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master   #40663      +/-   ##
============================================
+ Coverage     56.16%   65.25%   +9.08%     
- Complexity     2288    15804   +13516     
============================================
  Files          1124     1954     +830     
  Lines        178336   118225   -60111     
  Branches       1489    12632   +11143     
============================================
- Hits         100170    77148   -23022     
+ Misses        75645    36383   -39262     
- Partials       2521     4694    +2173     
Flag Coverage Δ
go ?
java 65.25% <ø> (-10.54%) ⬇️
python ?

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.

@aIbrahiim

Copy link
Copy Markdown
Contributor Author

cc @Abacn

Comment thread CHANGES.md Outdated
@github-actions

Copy link
Copy Markdown
Contributor

Checks are failing. Will not request review until checks are succeeding. If you'd like to override that behavior, comment assign set of reviewers

@Abacn
Abacn merged commit 0c37058 into apache:master Oct 19, 2026
31 of 33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The PostCommit SQL job is flaky

2 participants