Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Eventually create a dev version for the latest duckdb-wasm dev release #10

Closed
tobilg opened this issue Jan 11, 2024 · 9 comments
Closed

Comments

@tobilg
Copy link
Contributor

tobilg commented Jan 11, 2024

It'd be really great to have a (dev) version that supports the dynamic loading of extension with DuckDB-WASM.

@holdenmatt
Copy link
Owner

I tried upgrading to the latest duckdb-wasm version on npm, but it seems JSDelivr hosting is broken because the package is now too big:
duckdb/duckdb-wasm#1561

I don't think it makes sense for me to host the wasm files, or make it a requirement for users of this library to host them. So I'll need to wait until the JsDelivr issue is fixed before upgrading (hopefully soon).

@tobilg
Copy link
Contributor Author

tobilg commented Jan 17, 2024

Thanks for the feedback, I agree. There seems to be some progress now: duckdb/duckdb-wasm#1589

Thus, https://www.npmjs.com/package/@duckdb/duckdb-wasm/v/1.28.1-dev89.0 was release a few hours ago which is 185MB unpacked now, compared to 1.63GB in the last version...

@holdenmatt
Copy link
Owner

Nice.

It looks like the JsDelivr links have a limit of 150MB: https://cdn.jsdelivr.net/npm/@duckdb/duckdb-wasm@1.28.1-dev89.0/dist/duckdb-browser-eh.worker.js

I'll try to keep an eye on progress (or feel free to ping here if it gets solved, and I'll do the upgrade). Thanks!

@tobilg
Copy link
Contributor Author

tobilg commented Jan 18, 2024

@carlopi
Copy link

carlopi commented Jan 18, 2024

@holdenmatt / @tobilg: this has indeed solved with the latest build, sorry for the inconvenience, I will shortly add a CI check that sizes have to be compatible with jsdelivr limits so to avoid regressions

@holdenmatt
Copy link
Owner

Ah great, thanks @carlopi!

Confirmed the JsDelivr links are working for me on the latest version. Will upgrade versions later today.

@tobilg
Copy link
Contributor Author

tobilg commented Jan 19, 2024

Looking forward to the new release, thanks!

@holdenmatt
Copy link
Owner

Ok, this has now been upgraded to the latest duckdb-wasm version.
I haven't seen any issues in my testing but let me know if anything isn't working for you :)

image

@tobilg
Copy link
Contributor Author

tobilg commented Jan 20, 2024

Thank you very much @holdenmatt!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants