Concurrent-ruby support

[rainerborene]

any plans to replace celluloid with concurrent-ruby?

[phstc]

Hi @rainerborene

It seems to be an interesting idea, it was also commented here. Unfortunately I can't work on this at moment, but a pull request would be very appreciated.

[gwincr11]

I am interested in using this gem, but I just removed the .16 version of celluloid that you require to get rid of a memory leak. It is a known leaky gem https://github.com/ASoftCo/leaky-gem.

Can Shoryuken at least be updated to the non-leaky version?

[phstc]

TIL: https://github.com/ASoftCo/leaky-gems

@gwincr11 yay! Changed here.

@rainerborene @gwincr11 can you guys help with testing the change above? 🙏

gem 'shoryuken', github: 'phstc/shoryuken', branch: 'feature/185-bump-celluloid-non-leaky-version'

[gwincr11]

Oh nice, thank you!
I can try to test. I have actually never used this gem before so I have some setup to figure out.

[geun]

+1

[phstc]

Just a heads up... I'm very excited with the changes in #291 so far. I'm thinking on releasing Shoryuken 3.0.0, with that change. It's basically removing Celluloid, in favor of a thread pool and a heartbeat, both from concurrent-ruby. No more actors, and the issues with messages, having to trap died actors etc. Super simple, as it should be.