search

alitrack / duckdb_fdw

DuckDB Foreign Data Wrapper for PostgreSQL
MIT License
272 stars 18 forks source link

Update files for duckdb 0.10.x #36

Closed ahuarte47 closed 4 months ago

ahuarte47 commented 4 months ago

This PR allows to build duckdb_fdw with duckdb 0.10.0

ahuarte47 commented 4 months ago

Hi @alitrack sorry, I do not understand, do you want I delete duckdb.hpp and duckdb.h files? I see this PR already merged. I copied them from last source code zip file: libduckdb-linux-amd64

ahuarte47 commented 4 months ago

Hi @alitrack I think merging this does duckdb-fdw needs to build with duckdb 0.10.0, I am geting now an error when running the extension with duckdb 0.9.2: duckdb_fdw.so: undefined symbol: _ZN6duckdb9ErrorData7MessageEv

Maybe we should to have a set of duckdb.hpp and duckdb.h files for old versions and other for 0.10.x and to use conditional #includes & #defines depending on a version setting?

What do you think?