Max Gabrielsson

Results 162 comments of Max Gabrielsson

It passed my local CI now after merging with main and getting Marks concurrency patches, lets see if it passes here too.

On main this no longer throws an internal error, but instead a proper exception: ``` v0.10.2-dev722 07352ec6cf Enter ".help" for usage hints. Connected to a transient in-memory database. Use ".open...

https://github.com/duckdb/duckdb/pull/11868 has been superseded by https://github.com/duckdb/duckdb/pull/11998 so this should be unblocked now!

Sorry, I said I was going to look into this but then went on vacation and been distracted with other things to get in before the release since I got...

This occurs in the rust client as well, even with latest `arrow`.

Actually, I think we might be in the wrong here. Arrow stores the interval in a 128bit integer, interpreted as multiple fields; ``` ┌──────────────────────────────┬─────────────┬──────────────┐ │ Nanos │ Days │ Months...

@Mause The diagram is from here: https://github.com/apache/arrow-rs/blob/36a6e515f99866eae5332dfc887c6cb5f8135064/arrow-array/src/types.rs#L265-L269 I agree seems like the rust def is wrong. So what now?

Thanks for driving this @david542542!

@cholmes I can't think of any reason why the ordering would matter for us right now, so feel free to go ahead! ( I also kind of like: xmin, xmax,...

Alright, well if thats already standardized then maybe it makes more sense to follow the existing convention.