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

Enable execute() to run prepared_statements for all connectors #601

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

margaretselzer
Copy link
Contributor

I am absolutely not sure about what applies to PostgreSQL and MySQL in terms of limitation to one query at a time or return value, but I hope that this is in any case a good baseline.

I have taken the liberty to modify the return type of execute() for MySql from void to size_t so that it is in sync with other connectors.

@rbock
Copy link
Owner

rbock commented Sep 18, 2024

Thanks!

FYI: It might take a few days for me to take a closer look. I am a bit busy with something else right now :-)

Feel free to ping me if it feels like it takes too long.

@margaretselzer
Copy link
Contributor Author

No worries. Do it when you can.

The only thing depending on this for me right now is query() (and some documentation updates). But lacking that does not block anything for me. Just doin' it for the thrill of a bit of intellectual satisfaction. ;-)

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