Skip to content

Remove BufferResultSets connection string setting#378

Merged
bgrainger merged 1 commit intomysql-net:masterfrom
bgrainger:remove-buffer-result-sets
Oct 31, 2017
Merged

Remove BufferResultSets connection string setting#378
bgrainger merged 1 commit intomysql-net:masterfrom
bgrainger:remove-buffer-result-sets

Conversation

@bgrainger
Copy link
Copy Markdown
Member

@bgrainger bgrainger requested a review from caleblloyd October 25, 2017 22:56
Copy link
Copy Markdown
Contributor

@caleblloyd caleblloyd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It was a great feature when it was needed for EF 1.x! Does uncomplicated the code and tests now. Thanks!

@bgrainger
Copy link
Copy Markdown
Member Author

@caleblloyd I'll hold off on merging and publishing this until Pomelo.EFCore is published. (Otherwise, updating MySqlConnector in a client project would break Pomelo.)

@yukozh
Copy link
Copy Markdown

yukozh commented Oct 29, 2017

Pomelo 2.0.0.1 has been published

@bgrainger bgrainger merged commit c1ec8bf into mysql-net:master Oct 31, 2017
bgrainger added a commit that referenced this pull request Oct 31, 2017
Remove BufferResultSets connection string setting.

Conflicts:
	.travis.yml
	src/MySqlConnector/MySqlClient/Results/Row.cs
bgrainger added a commit that referenced this pull request Oct 31, 2017
Remove BufferResultSets connection string setting.

Conflicts:
	.travis.yml
	src/MySqlConnector/MySqlClient/Results/Row.cs
@bgrainger bgrainger deleted the remove-buffer-result-sets branch October 31, 2017 14:19
@bgrainger
Copy link
Copy Markdown
Member Author

Removed in 0.30.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants