Skip to content

Conversation

@ajcvickers
Copy link
Contributor

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 force-pushed the NuthinButAGThang0618 branch from 6d48cc4 to 8140cfb Compare June 21, 2019 00:12
@ajcvickers ajcvickers requested a review from AndriySvyryd June 21, 2019 00:15
@ajcvickers ajcvickers force-pushed the NuthinButAGThang0618 branch from 8140cfb to b62fea7 Compare June 21, 2019 00:17
@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

4 participants