Skip to content

Conversation

@mlafeldt
Copy link
Member

The old extension API is superseded by duckdb_entrypoint_c_api, which uses DuckDB's modern C extension API with proper version negotiation and error handling.

The old extension API is superseded by duckdb_entrypoint_c_api which
uses DuckDB's modern C extension API with proper version negotiation
and error handling.
@mlafeldt mlafeldt force-pushed the remove_duckdb_entrypoint branch from ee96b1b to ea6b2fd Compare December 18, 2025 10:46
@mlafeldt mlafeldt requested a review from samansmink December 18, 2025 10:54
@mlafeldt
Copy link
Member Author

@samansmink I think you worked on the macros last, so it's your honor to tell me if this is ok. 😄

Copy link
Collaborator

@samansmink samansmink left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me! the legacy entrypoint macro was never used in our rust extension template so let's rip it out!

@samansmink samansmink merged commit 20cf791 into main Dec 18, 2025
3 checks passed
@samansmink samansmink deleted the remove_duckdb_entrypoint branch December 18, 2025 11:36
@mlafeldt mlafeldt self-assigned this Dec 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants