29167982032
90%

Ran 11 Jul 2026 09:02PM UTC

Jobs 0

Files 0

Run time –

Badge

Embed ▾

pending completion

Build # 29167982032

Build Type

push

github

Committed by

Giorgi

Commit Message

Skip Convert.ChangeType on the same-type object read path

NumericVectorDataReader.GetValue(offset, targetType) backs the non-generic read
surface (GetValue(ordinal), this[ordinal], GetValues). It always ran the boxed
value through Convert.ChangeType, even when targetType already matched the
column's natural type — the common case, since those APIs read using the
column's ClrType.

Convert.ChangeType does not early-out on same-type for IConvertible values: it
dispatches through ToXxx() and re-boxes the result, so every value was boxed
twice. Add a value.GetType() == targetType fast path that returns the already
boxed value directly.

Benchmark (read 1,000,000 rows x 3 numeric columns via GetValue, .NET 10, M4 Pro):

  Before: 54.50 ms, 137.3 MB allocated
  After:  22.73 ms,  68.7 MB allocated

-58% time and -50% allocations on the object-returning path (the remaining
allocation is the unavoidable single box of the object API). The typed
GetFieldValue<T> path is unaffected. Behavior is unchanged: ChangeType returned
the same value for same-type, and BigInteger (not IConvertible) already used
this same early-out inside ChangeType.

Coverage Stats

Source Files on build 29167982032

Detailed source file information is not available for this build.

Giorgi / DuckDB.NET / 29167982032
90%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on build 29167982032

Giorgi / DuckDB.NET / 29167982032 90%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on build 29167982032

Giorgi / DuckDB.NET / 29167982032
90%

README BADGES
x