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

Make DuckPGQ loadable in DuckDB WASM #170

Open
2 tasks done
Dtenwolde opened this issue Nov 25, 2024 · 2 comments
Open
2 tasks done

Make DuckPGQ loadable in DuckDB WASM #170

Dtenwolde opened this issue Nov 25, 2024 · 2 comments

Comments

@Dtenwolde
Copy link
Contributor

What happens?

Currently the DuckPGQ extension is not easily loadable in DuckDB WASM. For instance, go to https://duckdb.org/docs/api/wasm/overview.html and type:

install duckpgq from community; 
load duckpgq; 

And you'll receive the following error:

IO Error: Extension "https://community-extensions.duckdb.org/v1.1.3/wasm_eh/duckpgq.duckdb_extension.wasm" could not be loaded: Could not load dynamic lib: duckpgq
Error: bad export type for '_ZTIN6duckdb23CreatePropertyGraphInfoE': undefined

To Reproduce

Go to https://duckdb.org/docs/api/wasm/overview.html and type:

install duckpgq from community; 
load duckpgq; 

And you'll receive the following error:

IO Error: Extension "https://community-extensions.duckdb.org/v1.1.3/wasm_eh/duckpgq.duckdb_extension.wasm" could not be loaded: Could not load dynamic lib: duckpgq
Error: bad export type for '_ZTIN6duckdb23CreatePropertyGraphInfoE': undefined

OS:

macOs 13 - Apple M1 Pro

DuckDB Version:

v1.1.3

DuckDB Client:

WASM

Full Name:

Daniel ten Wolde

Affiliation:

CWI

How did you load the extension?

Community extension version

Did you include all relevant data sets for reproducing the issue?

Yes

Did you include all code required to reproduce the issue?

  • Yes, I have

Did you include all relevant configuration (e.g., CPU architecture, Python version, Linux distribution) to reproduce the issue?

  • Yes, I have
@Dtenwolde
Copy link
Contributor Author

Mentioned here: #166

@carlopi
Copy link
Contributor

carlopi commented Nov 26, 2024

Thanks, this is at the moment expected, problem is that DuckPGQ is somehow different than other extensions in the sense that it bundles DuckDB (and it's parser).

I think this might require wrapping duckdb code in a duckdb_impl namespace, so that it do not collides with duckdb functions (that might or might not be equivalent) that are provided on static linking by duckdb-wasm.

Anther possible way out is enabling DuckDB to be statically linked in, and do the remapping at the Wasm layer, but I am not sure that's viable.

DuckPGQ can work free-standing in the browser, but I would consider more proper to work in having this supported on top of regular duckdb-wasm.

Thanks for opening the tracking issue. Unsure what's the path forward, but can work, and would be super cool to have it working.

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

No branches or pull requests

2 participants