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

Add methods to get affected rows instead of their count #123

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

Conversation

PhenX
Copy link

@PhenX PhenX commented Mar 11, 2022

May be an alternative to #29 but could not find an way to make it work on MySQL.

@timaiv
Copy link

timaiv commented Sep 21, 2022

I think we need CancellationToken token = default parameter for RunAndReturnAsync method.

@vaclavholusa
Copy link

could not find an way to make it work on MySQL.

Will this work? https://stackoverflow.com/a/16592867/2284302

@artiomchi Do you think it's possible to somehow merge this PR even with the MySQL exlusion? This is indeed higly needed feature.

Or maybe release some forked version with output support for mssql?

@larsmaehlum
Copy link

could not find an way to make it work on MySQL.

Will this work? https://stackoverflow.com/a/16592867/2284302

@artiomchi Do you think it's possible to somehow merge this PR even with the MySQL exlusion? This is indeed higly needed feature.

Or maybe release some forked version with output support for mssql?

Would love for this to either be merged in without MySQL support or released as an addon library if feasible.
Throwing an exception if used on MySQL until a way to implement support would still be better than not having this feature for any database.

@PhenX
Copy link
Author

PhenX commented Aug 28, 2024

Hello, any news on this @artiomchi ?

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.

4 participants