-
-
Notifications
You must be signed in to change notification settings - Fork 925
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
Sequel + Celluloid + jtds error in 9.1.x #3945
Comments
@jeremyevans Can you give me a hint why we might be seeing this, before I dig deep into Sequel? |
@headius wasn't there a loading issue recently? ... think it might happen due concurrent (double) loading. |
@headius The only thing I can think of is In terms of what's happening, there is a hash of setup procs ( |
FYI, I ran into this same issue when running the repro script in issue #3398 today. It has 4 threads that all call |
@jeremyevans I think your theory was correct. I fixed #4091 in 93707ca, which reverted a bad change made some months ago that prevented the require lock from blocking. Given the similarity between that issue and this one, I'm going to optimistically call it solved. |
Environment
jruby-9.1.0.0
jruby-9.1.1.0
jruby-9.1.2.0
celluloid 0.17.3
sequel 4.34.0
jdbc-jtds 1.3.1
Issue:
When using jdbc-jtds inside of an actor in a celluloid pool i get the following sequel error thrown from multiple actors:
Sequel::AdapterNotFound: Could not load jdbc/jtds adapter: adapter class not registered in ADAPTER_MAP
Same gem versions + same ruby code work without error in jruby-1.7.x and jruby-9.0 (up to and including 9.0.5.0). No issues if i run the code serially (without celluloid).
Simplified example:
The text was updated successfully, but these errors were encountered: