Reduced memory consumption at server start #942

Merged
merged 1 commit into from Jan 22, 2013

Conversation

Projects
None yet
3 participants
Contributor

jlecomte commented Jan 20, 2013

This meta-data copy operation seems unnecessary to me, although I would like someone else to confirm before merging this PR. This change reduces the memory consumption at server start by almost 40% in our case (from ~160MB to <100 MB) because we have so many language resource bundles.

Contributor

drewfish commented Jan 22, 2013

Yeah, that copy seems unnecessary to me as well. Travis builds passed.

+1

Contributor

isao commented Jan 22, 2013

+1

drewfish added a commit that referenced this pull request Jan 22, 2013

Merge pull request #942 from jlecomte/rsperf
Reduced memory consumption at server start

@drewfish drewfish merged commit 1bbf16a into YahooArchive:develop Jan 22, 2013

1 check was pending

default The Travis build is in progress
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment