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
Allow multi-map async queries to be unbuffered #597
base: main
Are you sure you want to change the base?
Conversation
See DapperLib#596 for more detail. Originally, the code would dispose the dataReader that it opened before enumeration would begin. Now when you dispose the enumerator, it will also close the command, which is line with other behavior. Buffered behavior remains consistent. Has no impact on sync version.
@mgravell , I'll fix the merge conflict but is there any object to merging this? |
Needs a test/usage-illustration, but other than that: looks good |
Any update on this? |
@mburbea Any update on example usage/test for this? |
@NickCraver, hey sorry it's been a while since I looked at this issue I'm not currently working on a project that uses the dapper library (that was back in september) so I have not had time to look further into this. If you rather I close the issue. I can do that. |
Would be great to see this fixed in the current version of Dapper rather than 2,0. That seems to be a way off still. |
Is there a reason this can't get merged (or otherwise fixed) until V2? I'm working on something at the moment that uses only async calls and could really benefit from disabling buffering in several places. |
See #596 for more detail. Originally, the code would dispose the
dataReader that it opened before enumeration would begin. Now when you
dispose the enumerator, it will also close the command, which is line
with other behavior.
Buffered behavior remains consistent. Has no impact on sync version.