go-duckdb icon indicating copy to clipboard operation
go-duckdb copied to clipboard

Row#SetRowValue is not considering projection

Open unrealhoang opened this issue 1 year ago • 1 comments

https://github.com/marcboeker/go-duckdb/blob/51311da74864898764a6ccbbec2dc104486e46df/row.go#L32-L37

Also I see that this function is not being used anywhere, maybe remove it so people not use it mistakenly instead of SetRowValue[T].

unrealhoang avatar Dec 09 '24 02:12 unrealhoang

also currently there's no way to set row value to null, SetRowValue[*int64](row, colIdx, nil) will raise a cast error: cannot cast *int64 to int64, and returning error to FillRow callback will segfault.

unrealhoang avatar Dec 09 '24 06:12 unrealhoang

👋 Thanks for opening this issue!

This issue has been moved to https://github.com/duckdb/duckdb-go/issues/30, and I'm closing it here.

Please see https://github.com/marcboeker/go-duckdb/issues/565 for more details.

mlafeldt avatar Oct 20 '25 08:10 mlafeldt