Skip to content

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

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

alaricd wants to merge 4 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 (e28f086) to head (513a2ef).
⚠️ Report is 122 commits behind head on develop.

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             @@
##           develop     #364      +/-   ##
===========================================
+ Coverage    86.79%   87.17%   +0.37%     
===========================================
  Files           82       82              
  Lines         3560     3665     +105     
  Branches       553      557       +4     
===========================================
+ Hits          3090     3195     +105     
+ Misses         331      330       -1     
- Partials       139      140       +1     

☔ 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

coveralls commented Oct 14, 2026 •

Copy link
Copy Markdown

Coverage Report for CI Build 39342681652

Warning

Build has drifted: This PR's base is out of sync with its target branch, so coverage data may include unrelated changes.
Quick fix: rebase this PR. Learn more →

Coverage decreased (-0.2%) to 92.737%

Details

  • Coverage decreased (-0.2%) from the base build.
  • Patch coverage: 4 uncovered changes across 1 file (160 of 164 lines covered, 97.56%).
  • 18 coverage regressions across 6 files.

Uncovered Changes

File Changed Covered %
DuckDB.NET.Data/DuckDBSchema.cs 164 160 97.56%

Coverage Regressions

18 previously-covered lines in 6 files lost coverage.

File Lines Losing Coverage Coverage
DuckDB.NET.Data/DataChunk/Writer/VectorDataWriterBase.cs 7 75.12%
DuckDB.NET.Data/Arrow/DuckDBArrowArrayStream.cs 3 88.46%
DuckDB.NET.Data/DuckDBAppender.cs 2 97.55%
DuckDB.NET.Data/DuckDBAppenderRow.cs 2 95.96%
DuckDB.NET.Data/DuckDBConnection.ScalarFunction.Extensions.cs 2 95.49%
DuckDB.NET.Data/DuckDBConnection.TableFunction.Extensions.cs 2 91.22%

Coverage Stats

Coverage Status
Relevant Lines: 4171
Covered Lines: 3932
Line Coverage: 94.27%
Relevant Branches: 2452
Covered Branches: 2210
Branch Coverage: 90.13%
Branches in Coverage %: Yes
Coverage Strength: 965880.58 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?

@alaricd
alaricd marked this pull request as ready for review October 19, 2026 12:07

This branch is waiting to be deployed

1 waiting deployment
external — a18c32ff Waiting Oct 19, 2026 by alaricd via Authorize #674
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