-
Notifications
You must be signed in to change notification settings - Fork 236
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
(SERVER-408) backport jruby borrow timeouts #443
(SERVER-408) backport jruby borrow timeouts #443
Conversation
Add :borrow-timeout configuration option, which allows the user to configure the timeout when borrowing instances from the JRuby pool and defaults to 60000 milliseconds. Change all calls to the borrow-from-pool function to calls to borrow-from-pool-with-timeout. Conflicts: test/unit/puppetlabs/services/config/puppet_server_config_service_test.clj
Improve the error handling in the `with-jruby-puppet` macro when borrowing an instance from the pool times out.
This commit simply increases the timeout to 1200000 ms (20 minutes).
@@ -18,12 +18,17 @@ | |||
[puppet-server-config-service jruby-puppet-pooled-service jetty9-service | |||
profiler/puppet-profiler-service]) | |||
|
|||
(def required-config |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two required-config
s now...this one looks like the one to delete
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Man that's the second time I've caused that same problem in the last 3 days. :) nice catch.
Added a commit to fix the duplicate |
👍 assuming Travis passes. |
👍 |
…rt-jruby-borrow-timeouts (SERVER-408) backport jruby borrow timeouts
This PR cherry-picks the following three commits from master:
f72c912
14c498e
8ef2383
This should effectively back-port the jruby borrow timeout work to the stable branch.