search

isaacbrodsky / h3-duckdb

Bindings for H3 to DuckDB
Apache License 2.0
159 stars 8 forks source link

v0.10.0 update #81

Closed akhenakh closed 7 months ago

akhenakh commented 7 months ago

Tested a few queries, solves https://github.com/isaacbrodsky/h3-duckdb/issues/76

isaacbrodsky commented 7 months ago

It looks like there's a combination of test failures due to needing to specify an error message, and a build failure on Linux which probably needs to have some code copied from the extension template in order to fix. Happy for you to update this PR; otherwise I will take a look at upgrading the extension Soon(tm).

akhenakh commented 7 months ago

Interesting cause I've tested on Linux only and it worked great.

sure you can take over, the PR was mainly to remind you the changes for the Exception, you can close it.

isaacbrodsky commented 7 months ago

going to close this for the time being, I am working on upgrading in #82. Thanks!