em-synchrony 1.0.3 and 1.0.4 do not work with ActiveRecord 5.0 about em-synchrony

[Xanders]

Hello. If you're (or someone) still ineresting in, I've created a branch with ActiveRecord 5 fix:
https://github.com/Xanders/em-synchrony/tree/active_record_5
Use it in Gemfile like:

gem 'em-synchrony', github: 'Xanders/em-synchrony', branch: 'active_record_5', require: %w[em-synchrony em-synchrony/mysql2 em-synchrony/activerecord em-synchrony/em-http em-synchrony/fiber_iterator] # just a sample, fill the require list by yourself

Also it looks like tests was fixed two months ago: 1a232d2
So project is not totally dead. Hope to see EM-Synchrony support in ActiveRecord-Import in the future again...

[dgutov]

@Xanders Hey, is there a reason you've branched your fix not from master, but from a proposed pull request branch?

[Xanders]

@dgutov Yes, it's because my fix is based on changes from that PR, so we should wait for #190 to be merged until it will be possible to merge my fixes.

[dgutov]

my fix is based on changes from that PR

Any particular reason for that?

#190 would have to be rebased first.

[Xanders]

@dgutov Is there any other way to create PR with code using code from other PR except of copy-paste all of that code? I don't understand what the problem now, sorry.

[dgutov]

I'm asking why you've used the approach from that PR and not followed the approach that we currently have in master.

[Xanders]

@dgutov I've tried current master code (both active_record.rb and active_record_4.rb) with AR5 and had a lot of strange errors, didn't know how to fix them. But when i tried fork from #190 it was almost worked except of some moments, that i fixed. I assumed fork is more recent version of synchrony/ar then current master, so I'm forked it, not master.
Was it wrong?

[dgutov]

It's not more recent, no. But it's possible that it uses a better approach, considering you've had to fix fewer places.

[Xanders]

@dgutov So I was wrong, sorry. But I think code from that fork is much simpler and more stable, so I still prefer it.
But note, I didn't write any heavy/complex tests yet, so some bugs may be there.

[dgutov]

Do the existing tests pass?

[Xanders]

@dgutov Only my app's tests. I didn't run synchrony's tests, I don't have much time. It was just a quick fix to force my app working.

[dgutov]

Please let us know the result when you get around to it.

[dgutov]

Hey folks, could you please try the current em-synchrony master?

See if it works okay with your AR5 based applications.