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 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

Verified

This commit was signed with the committer’s verified signature.
makenowjust Hiroya Fujinami
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
@headius headius deleted the platform_defaults branch June 28, 2016 16:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant