-
Notifications
You must be signed in to change notification settings - Fork 394
Add JRuby 10.0 support #4928
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
base: master
Are you sure you want to change the base?
Add JRuby 10.0 support #4928
Conversation
1ff5503
to
8f670ba
Compare
BenchmarksBenchmark execution time: 2025-10-09 21:53:01 Comparing candidate commit 765d7e9 in PR branch Found 1 performance improvements and 0 performance regressions! Performance is the same for 43 metrics, 2 unstable metrics. scenario:profiling - Allocations ()
|
27ec659
to
4438796
Compare
23bcef4
to
da1db77
Compare
11cbb99
to
bf88571
Compare
bf88571
to
cacdda7
Compare
Typing analysisIgnored filesThere are 539 ignored files in the Steepfile out of 836. Ignored files
Note: Ignored files are excluded from the next sections.
|
3e675aa
to
37e62ed
Compare
37e62ed
to
765d7e9
Compare
# MongoDB does not support JRuby 10.0, which fails on an error related the bson_ruby NativeService for latest versions. | ||
# https://github.com/mongodb/mongo-ruby-driver#mongodb-ruby-driver | ||
'mongo-latest' => '✅ 2.5 / ✅ 2.6 / ✅ 2.7 / ✅ 3.0 / ✅ 3.1 / ✅ 3.2 / ✅ 3.3 / ✅ 3.4 / ✅ 3.5 / ✅ jruby / ❌ jruby 10.0', | ||
'mongo-min' => '✅ 2.5 / ✅ 2.6 / ✅ 2.7 / ✅ 3.0 / ✅ 3.1 / ✅ 3.2 / ✅ 3.3 / ✅ 3.4 / ✅ 3.5 / ✅ jruby / ❌ jruby 10.0', |
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.
I'm stuck on how to properly skip JRuby 10.0 for mongodb
tests (see also the tasks/runtime_matcher.rb
file), or if that's even the right way to go. JRuby 10.0 fails with the latest version 2.21.3 with an error related to:
Failure/Error: Unable to find org.bson_ruby.NativeService.basicLoad(NativeService.java to read failed line
Java::JavaLang::NoSuchMethodError:
'org.jruby.RubyModule org.jruby.Ruby.fastGetModule(java.lang.String)'...
Given the lack of support, I propose to skip these tests, but I'm not sure how best to skip them. What am I missing?
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 sounds like a bug in the jruby version of the bson gem; did you spot which version we're using?
(We could adjust the dsl to support skipping jruby versions, but it may be easier to skip on all jruby versions temporarily rather than doing that...?)
What does this PR do?
Adds JRuby 10.0 support to CI.
Motivation:
JRuby 10.0 is out!
Change log entry
No change log entry.
Additional Notes:
Tests that are skipped and need further investigation:
Datadog::Core::Remote::Component::Barrier
Datadog::Tracing::Contrib::Shoryuken::Tracer
These all lead with
TODO: JRuby 10.0 -
...I also added a temporary
jruby_100?
method as recommended by Marco that includes a comment to update Compatibility documentation after all tests are addressed.The skipped tests are documented on the Ruby Guild board: https://github.com/orgs/DataDog/projects/47/views/10?query=sort%3Aupdated-desc+is%3Aopen&sliceBy%5Bvalue%5D=lang+support
How to test the change?
Green CI.