Skip to content

feat: expose Statement sqlite3_stmt_status values#162

Merged
DjDeveloperr merged 1 commit intodenodrivers:mainfrom
kigiri:expose-stmt-status-counters
Apr 10, 2026
Merged

feat: expose Statement sqlite3_stmt_status values#162
DjDeveloperr merged 1 commit intodenodrivers:mainfrom
kigiri:expose-stmt-status-counters

Conversation

@kigiri
Copy link
Copy Markdown
Contributor

@kigiri kigiri commented Apr 2, 2026

Exposed those values to help having more details to optimize queries:
https://sqlite.org/c3ref/c_stmtstatus_counter.html

Doc is copy / pasted from SQLite page.

I considered exposing them as getters but the we would not be able to reset them, I felt this API was closest to the original SQLite interface

@kigiri kigiri force-pushed the expose-stmt-status-counters branch from 1ad0516 to 55d1cb5 Compare April 2, 2026 13:38
@DjDeveloperr
Copy link
Copy Markdown
Member

some formatting issues, otherwise LGTM! thanks

@kigiri kigiri force-pushed the expose-stmt-status-counters branch from 55d1cb5 to b3abca3 Compare April 7, 2026 12:16
@kigiri
Copy link
Copy Markdown
Contributor Author

kigiri commented Apr 7, 2026

@DjDeveloperr fixed the format, sorry for that !

Copy link
Copy Markdown
Member

@DjDeveloperr DjDeveloperr left a comment

Choose a reason for hiding this comment

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

LGTM, thank you!

@DjDeveloperr DjDeveloperr merged commit f4b316f into denodrivers:main Apr 10, 2026
5 checks passed
@kigiri kigiri deleted the expose-stmt-status-counters branch April 14, 2026 10:03
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