Closed igitur closed 1 year ago
Comparing https://github.com/AlexR2D2/metabase_duckdb_driver/blob/main/src/metabase/driver/duckdb.clj to https://github.com/metabase/crate-driver/blob/master/src/metabase/driver/crate.clj , it seems there are still many methods/conversions/whatever missing in the DuckDB driver. The date-specific parts start at https://github.com/metabase/crate-driver/blob/master/src/metabase/driver/crate.clj#L142 .
I'd love to submit a PR, but I have no experience with Clojure and wouldn't even know where to begin to debug this. If you don't have the capacity, I can try, but it will take a while.
Maybe the Sqlite driver is a better base to work from, since the syntax is so similar to DuckDB.
Please, check the new version of driver. Added the date processing functions as you proposed. Thanks for issue researching.
Sorry, currently on holiday. Will test and give feedback mid January.
Thanks, this is much better. Appreciate the effort. I think there are a few more issues, but I will investigate more that it's a driver problem and if so, will log new issues.
Using v0.6.1 of the driver and connecting successfully to a DuckDB file. I can see the tables and browse around a bit, but it seems any query that uses a date in the query generates this error:
I found a similar issue for the Crate driver at https://github.com/metabase/crate-driver/issues/1 , and it seems it required a driver fix.