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

Rails 6: Support lazy transactions #780

Merged
merged 5 commits into from
Apr 29, 2020

Conversation

aidanharan
Copy link
Contributor

@aidanharan aidanharan commented Apr 23, 2020

Lazy transactions changes required by rails/rails#32647

Before
https://travis-ci.org/github/rails-sqlserver/activerecord-sqlserver-adapter/jobs/680126303

6730 runs, 18723 assertions, 24 failures, 1 errors, 26 skips

After
https://travis-ci.org/github/rails-sqlserver/activerecord-sqlserver-adapter/jobs/680176451

6730 runs, 18725 assertions, 8 failures, 1 errors, 26 skips

@YehudaGold
Copy link
Contributor

YehudaGold commented Apr 24, 2020

Hi, i implemented the lazy transactions support in my fork YehudaGold/activerecord-sqlserver-adapter (commit: YehudaGold@6b31617).
by adding the materialize_transactions guard rails/rails#32647 (comment).

I tried to open pull request to your fork:branch from my own but github don`t let my.
is this a permissions issue? can you merge the changes or implement them?

@aidanharan
Copy link
Contributor Author

@ YehudaGold Merged your changes into my branch. Not sure why you couldnt create a PR (might be config of my fork). Thanks for your help.

@YehudaGold YehudaGold mentioned this pull request Apr 26, 2020
@YehudaGold
Copy link
Contributor

I found the problem in BasicsTest#test_0023_an empty transaction does not raise if preventing writes, because tests automatically wrapped in transaction the transaction in the test become SavepointTransaction and sql-server dont have release_savepoint` query so the assert should be corrected to 1.

@aidanharan aidanharan marked this pull request as ready for review April 28, 2020 11:08
@wpolicarpo wpolicarpo merged commit a07552c into rails-sqlserver:6-0-dev Apr 29, 2020
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.

3 participants