Skip to content
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

Use platform defaults from defaults file rather than eager load. #3972

Merged
merged 1 commit into from Jun 28, 2016

Conversation

@headius
Copy link
Member

headius commented Jun 16, 2016

This trims off a bit of startup time by not loading more of RG
than is necessary. Pending release of rubygems/rubygems#1644 (and install of that release into JRuby).

Before:

[] ~/projects/jruby $ time jruby -e 1

real    0m2.397s
user    0m6.476s
sys 0m0.310s

[] ~/projects/jruby $ time jruby --dev -e 1

real    0m1.884s
user    0m2.717s
sys 0m0.223s

After:

[] ~/projects/jruby $ time jruby -e 1

real    0m2.181s
user    0m5.681s
sys 0m0.295s

[] ~/projects/jruby $ time jruby --dev -e 1

real    0m1.733s
user    0m2.493s
sys 0m0.214s
This trims off a bit of startup time by not loading more of RG
than is necessary. Pending release of rubygems/rubygems#1644.
@headius headius added this to the JRuby 9.1.3.0 milestone Jun 16, 2016
headius added a commit that referenced this pull request Jun 28, 2016
This pulls in PR rubygems/rubygems#1644 so we can merge #3972.
@headius headius merged commit 4f0fdb8 into jruby:master Jun 28, 2016
0 of 2 checks passed
0 of 2 checks passed
continuous-integration/appveyor/pr AppVeyor build failed
Details
continuous-integration/travis-ci/pr The Travis CI build failed
Details
@headius headius deleted the headius:platform_defaults branch Jun 28, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant
You can’t perform that action at this time.