duckdb / duckdb_azure

Azure extension for DuckDB
MIT License
51 stars 17 forks source link

Remove erroneous createifnotexist and update DuckDB #11

Closed samansmink closed 1 year ago

samansmink commented 1 year ago

This PR removes 2 createifnotexists statements that were erroneous and caused permission issues. This PR fixes #9, and possibly other auth issues? I tested this using the reproduction from #9 using a container SAS token. Since tokens are not validated in azurite we cant really test this in CI atm unfortunately. We should probably setup a test container in the future with a small test suite that runs on main

Also I updated to DuckDB v0.9.0, adding the reusable binary distribution workflow to the CI so we can produce binaries for duckdb 0.9.0