Skip to content

Conversation

sgrif
Copy link
Contributor

@sgrif sgrif commented Mar 12, 2019

50 is probably way too high. @jtgeibel and I are interested in seeing
how this is affecting our memory usage, it'd be great to configure this
without a redeploy.

50 is probably way too high. @jtgeibel and I are interested in seeing
how this is affecting our memory usage, it'd be great to configure this
without a redeploy.
@sgrif sgrif requested a review from jtgeibel March 12, 2019 21:18
@jtgeibel
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Mar 12, 2019

📌 Commit 42be196 has been approved by jtgeibel

@bors
Copy link
Contributor

bors commented Mar 12, 2019

⌛ Testing commit 42be196 with merge 793d3ed...

bors added a commit that referenced this pull request Mar 12, 2019
Make the number of threads on the server configurable

50 is probably way too high. @jtgeibel and I are interested in seeing
how this is affecting our memory usage, it'd be great to configure this
without a redeploy.
@bors
Copy link
Contributor

bors commented Mar 12, 2019

☀️ Test successful - checks-travis
Approved by: jtgeibel
Pushing 793d3ed to master...

@bors bors merged commit 42be196 into rust-lang:master Mar 12, 2019
@sgrif sgrif deleted the sg-configurable-server-threads branch March 18, 2019 18:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants