From c47481da126e45e695dc6d63c8dc057c4fec6cb9 Mon Sep 17 00:00:00 2001 From: Diego Imbert <70353967+diegoimbert@users.noreply.github.com> Date: Sat, 15 Nov 2025 19:10:05 +0100 Subject: [PATCH] fix: temporary fix for duckdb type_aliases causing issues (#7148) --- .../windmill-duckdb-ffi-internal/src/lib.rs | 25 ++++++++++--------- 1 file changed, 13 insertions(+), 12 deletions(-) diff --git a/backend/windmill-duckdb-ffi-internal/src/lib.rs b/backend/windmill-duckdb-ffi-internal/src/lib.rs index 501b5c5dee..25f8e5aba8 100644 --- a/backend/windmill-duckdb-ffi-internal/src/lib.rs +++ b/backend/windmill-duckdb-ffi-internal/src/lib.rs @@ -245,7 +245,7 @@ fn do_duckdb_inner( } // Statement needs to be stepped at least once or stmt.column_names() will panic let mut column_names = None; - let mut type_aliases = None; + // let mut type_aliases = None; loop { let row = rows.next(); match row { @@ -260,17 +260,18 @@ fn do_duckdb_inner( column_names.as_ref().unwrap() } }; - let type_aliases = match type_aliases.as_ref() { - Some(type_aliases) => type_aliases, - None => { - type_aliases = Some( - (0..stmt.column_count()) - .map(|i| stmt.column_logical_type(i).get_alias()) - .collect::>(), - ); - type_aliases.as_ref().unwrap() - } - }; + // let type_aliases = match type_aliases.as_ref() { + // Some(type_aliases) => type_aliases, + // None => { + // type_aliases = Some( + // (0..stmt.column_count()) + // .map(|i| stmt.column_logical_type(i).get_alias()) + // .collect::>(), + // ); + // type_aliases.as_ref().unwrap() + // } + // }; + let type_aliases = (0..stmt.column_count()).map(|_| None).collect::>(); let row = row_to_value(row, &column_names.as_slice(), &type_aliases.as_slice()) .map_err(|e| e.to_string())?;