feat: duckdb support TimeUnit::Us as Timestamp (#2780)

joseph-isaacs · web-flow · commit 6cdaba224181 · 2025-03-25T10:24:20.000Z
diff --git a/vortex-duckdb/src/convert/types/to.rs b/vortex-duckdb/src/convert/types/to.rs
@@ -77,6 +77,7 @@ pub fn ext_to_duckdb(ext_dtype: &ExtDType) -> LogicalTypeHandle {
             }
             match time_unit {
                 TimeUnit::Ns => LogicalTypeHandle::from(LogicalTypeId::TimestampNs),
+                TimeUnit::Us => LogicalTypeHandle::from(LogicalTypeId::Timestamp),
                 TimeUnit::Ms => LogicalTypeHandle::from(LogicalTypeId::TimestampMs),
                 TimeUnit::S => LogicalTypeHandle::from(LogicalTypeId::TimestampS),
                 _ => {

Original file line number	Diff line number	Diff line change
`@@ -77,6 +77,7 @@ pub fn ext_to_duckdb(ext_dtype: &ExtDType) -> LogicalTypeHandle {`
`77`	`77`	`}`
`78`	`78`	`match time_unit {`
`79`	`79`	`TimeUnit::Ns => LogicalTypeHandle::from(LogicalTypeId::TimestampNs),`
	`80`	`+ TimeUnit::Us => LogicalTypeHandle::from(LogicalTypeId::Timestamp),`
`80`	`81`	`TimeUnit::Ms => LogicalTypeHandle::from(LogicalTypeId::TimestampMs),`
`81`	`82`	`TimeUnit::S => LogicalTypeHandle::from(LogicalTypeId::TimestampS),`
`82`	`83`	`_ => {`