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

Timeout worker pool size #611

Closed
wants to merge 2 commits into from

Conversation

Projects
None yet
2 participants
@fooblahblah
Copy link
Contributor

commented Mar 27, 2013

Adds a new config property, jruby.timeout.thread.pool.max, to control the number of Timeout worker threads. Defaults to # of cores as before.

Jeff Simpson added some commits Mar 27, 2013

Jeff Simpson
Added call to get the RubyInstanceConfig.POOL_MAX setting, which uses…
… Options.THREADPOOL_MAX. By default use # of cores, otherwise the setting value
@@ -106,10 +106,10 @@ public RubyInstanceConfig() {
String[] elements = COMPILE_EXCLUDE.split(",");
excludedMethods.addAll(Arrays.asList(elements));
}

This comment has been minimized.

Copy link
@BanzaiMan

BanzaiMan Mar 29, 2013

Member

Please remove white space changes in order to reduce noise. Thank you.

@fooblahblah

This comment has been minimized.

Copy link
Contributor Author

commented Mar 29, 2013

Sure, sorry about that! Would you prefer a new pull request, update to this one, a rebased force push?

@BanzaiMan

This comment has been minimized.

Copy link
Member

commented Mar 29, 2013

Updating this is fine.

@fooblahblah

This comment has been minimized.

Copy link
Contributor Author

commented Apr 1, 2013

I'm going to close this one and re-submit since the upstream also changed. It was easier to re-apply against master.

@fooblahblah fooblahblah closed this Apr 1, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.