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

Adds DbContext to CommandEventData #16162

Merged
merged 2 commits into from
Jun 21, 2019
Merged

Adds DbContext to CommandEventData #16162

merged 2 commits into from
Jun 21, 2019

Conversation

ajcvickers
Copy link
Member

Fixes #16159

Also introduced a parameter object for the RelationalCommand execution methods. This will make it easier to pass new things here without making a breaking change.

Did a little refactoring to simplify the code in RelationalCommand and avoid a couple of places where we were allocating.

@nphmuller
Copy link

Found a small typo in xml docs for the DbContext parameter, while looking through the changes:
currently being used, ot null if not known.

Fixes #16159

Also introduced a parameter object for the `RelationalCommand` execution methods. This will make it easier to pass new things here without making a breaking change.

Did a little refactoring to simplify the code in `RelationalCommand` and avoid a couple of places where we were allocating.
@ajcvickers ajcvickers merged commit d48ceef into master Jun 21, 2019
@ghost ghost deleted the NuthinButAGThang0618 branch June 21, 2019 00:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make DbContext available from CommandEventData for interception
3 participants