Giorgi / DuckDB.NET / 41643517054

94%

Build Type

push

github

Committed by Giorgi

Commit Message

Finish the TIME_NS read follow-up: one conversion, TIME path as before

- DuckDBTimeNs.ToDuckDBTimeOnly() is the one TIME_NS to DuckDBTimeOnly
  conversion; DuckDBValue and both reader paths use it.
- The TIME_NS branch of GetValidValue moves into the final switch, so a
  TIME read no longer checks for TIME_NS first.
- Remove the unused NanosecondsPerMicrosecond constant.
- DESIGN.md: rewrap the TIME/TIME_NS/TIMETZ paragraph, say how TIME_NS
  24:00 reads, and that DuckDBTimeOnly drops sub-microsecond digits as
  DuckDBTimestamp does for TIMESTAMP_NS.
- Tests: merge the two overlapping TIME_NS precision tests, read TIME_NS
  in a struct field and a map key and value (with NULLs), as DateTime
  through the object path, and 24:00 through DuckDBValue.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01LUiK1srtXX6GfTBufhTdvN

Coverage Stats

2633 of 2907 branches covered (90.57%)

Branch coverage included in aggregate %.

5 of 6 new or added lines in 4 files covered. (83.33%)

1 existing line in 1 file now uncovered.

4233 of 4494 relevant lines covered (94.19%)

1054005.68 hits per line