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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Performance: evmserver start-up: Improve ChargeableField.seed #15236

Merged
merged 3 commits into from May 26, 2017

Conversation

isimluk
Copy link
Member

@isimluk isimluk commented May 26, 2017

1000.times { ChargeableField.seed }

ms bytes objects queries query (ms) comments
59,197.1 49,858,421* 44,085,570 59,000 19,059.5 before
4,636.3 16,794,676* 5,632,636 2,000 763.2 after
94% 66% 87% 97% 96% improvement

馃惛

@miq-bot
Copy link
Member

miq-bot commented May 26, 2017

Checked commits isimluk/manageiq@5dc513d~...00c0d55 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
1 file checked, 0 offenses detected
Everything looks fine. 馃弳

@martinpovolny
Copy link

馃悩

@chessbyte chessbyte self-assigned this May 26, 2017
@chessbyte chessbyte merged commit b181ffc into ManageIQ:master May 26, 2017
@chessbyte chessbyte added this to the Sprint 62 Ending Jun 5, 2017 milestone May 26, 2017
@chessbyte
Copy link
Member

@isimluk Nice find!

@isimluk isimluk deleted the perf-chargeable-field-seed branch May 31, 2017 10:37
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

4 participants