Skip to content

Support ADO.NET schema data types and metadata probing - #364

Draft
alaricd wants to merge 2 commits into
Giorgi:developfrom
pengdows:draft/ado-schema-metadata
Draft

alaricd wants to merge 2 commits into
Giorgi:developfrom
pengdows:draft/ado-schema-metadata

Conversation

@alaricd

@alaricd alaricd commented Oct 14, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add standard DataSourceInformation metadata needed by ADO.NET schema probing.
  • Add DataTypes metadata for DuckDB native types.
  • Align metadata column ordering, types, normalized versions, DBNull values, and restriction validation with the common ADO.NET contract.
  • Correct index restriction metadata.
  • Add focused schema coverage.

Validation

  • dotnet test DuckDB.NET.Test/Test.csproj --no-restore --filter FullyQualifiedName~SchemaTests -m:1
  • 23 tests passed.

Giorgi and others added 2 commits October 6, 2026 01:23
Add ducklabs-logo.png to the sponsors folder and point the README's
sponsor image at it instead of duckdb-labs-logo.png.

The image URL reads from main, so the logo appears only after this
change is merged there.

(cherry picked from commit e5a8102)
@codecov

codecov Bot commented Oct 14, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 99.09091% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 87.17%. Comparing base (79857ba) to head (513a2ef).
⚠️ Report is 222 commits behind head on main.

Files with missing lines Patch % Lines
DuckDB.NET.Data/DuckDBSchema.cs 99.09% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #364      +/-   ##
==========================================
- Coverage   88.09%   87.17%   -0.92%     
==========================================
  Files          65       82      +17     
  Lines        2310     3665    +1355     
  Branches      322      557     +235     
==========================================
+ Hits         2035     3195    +1160     
- Misses        202      330     +128     
- Partials       73      140      +67     

☔ 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.

@coveralls

Copy link
Copy Markdown

Coverage Report for CI Build 38652328885

Warning

No base build found for commit 05ef6f0 on main.
Coverage changes can't be calculated without a base build.
If a base build is processing, this comment will update automatically when it completes.

Coverage: 88.732%

Details

  • Patch coverage: 110 of 110 lines across 1 file are fully covered (100%).

Uncovered Changes

No uncovered changes found.

Coverage Regressions

Requires a base build to compare against. How to fix this →


Coverage Stats

Coverage Status
Relevant Lines: 3665
Covered Lines: 3335
Line Coverage: 91.0%
Relevant Branches: 1935
Covered Branches: 1634
Branch Coverage: 84.44%
Branches in Coverage %: Yes
Coverage Strength: 407270.02 hits per line

💛 - Coveralls

@Giorgi
Giorgi changed the base branch from main to develop October 15, 2026 07:42
@Giorgi

Giorgi commented Oct 15, 2026

Copy link
Copy Markdown
Owner

Is it ready for review?

This branch is waiting to be deployed

1 waiting deployment
external — 513a2ef2 Waiting Oct 14, 2026 by alaricd via Authorize #646
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.

3 participants