-
Notifications
You must be signed in to change notification settings - Fork 200
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
Initial Ractor support #365
base: main
Are you sure you want to change the base?
Conversation
10e89a1
to
fea5c83
Compare
fea5c83
to
e116ba5
Compare
Is there any outstanding feedback here? Thanks |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems fine to me. I haven't investigated everything we need to do to make C extensions Ractor safe but I think this is fine to start with!
Great, I'll rebase and merge it today or tomorrow. |
e116ba5
to
54959e6
Compare
So it seems like in Ruby 3.3, trying to pass a I'm not sure if this is intentional behavior or not. @sandlerr or @tenderlove do you know? Edit: to be explicit, the RemoteError is wrapping |
OK, that exception started to get raised in ruby/ruby@ce47ee00 as a fix, so I'm just going to adapt the test to the different behaviors before and after 3.3.0. |
0f4db3d
to
104ea23
Compare
The stress test hangs under Ruby 3.3 with as few as 3 ractor writers, and it's not clear to me why. |
104ea23
to
6ccb27b
Compare
Allows users to create / open databases, send queries, and process ResultSets inside a Ractor. However, does not allow passing the gem's domain objects through the Ractor communication channels.
See #299 .