Skip to content

Use offset with length for statement query - #573

Closed
smvv wants to merge 1 commit into
duckdb:mainfrom
smvv:fix-query-statement-length
Closed

Use offset with length for statement query#573
smvv wants to merge 1 commit into
duckdb:mainfrom
smvv:fix-query-statement-length

Conversation

@smvv

@smvv smvv commented Jul 29, 2026

Copy link
Copy Markdown
Member

CI job release fails in Install sdist on main with:

In member function ‘std::string duckdb::DuckDBPyStatement::Query() const’: src/pystatement.cpp:32:35:
error: ‘class duckdb::SQLStatement’ has no member named ‘stmt_length’
     32 |         auto &length = statement->stmt_length;
        |                                   ^~~~~~~~~~~

@evertlammerts

Copy link
Copy Markdown
Member

Thanks for the fix! There's another breaking change in core's main, so I fixed both in #575. Feel free to commandeer :-)

@smvv
smvv deleted the fix-query-statement-length branch July 30, 2026 09:23
evertlammerts added a commit that referenced this pull request Jul 30, 2026
Fixes latest build issues introduced by duckdb/duckdb#24159 and
duckdb/duckdb#24273.

Note that duckdb core's debug build is broken again due to missing
imports.

Supersedes #573
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

Successfully merging this pull request may close these issues.

2 participants