Skip to content

add parameter 'max_threads' to define value in jetty.ini#128

Closed
mklette wants to merge 1 commit intopuppetlabs:masterfrom
mklette:dev-jetty_ini-max_threads
Closed

add parameter 'max_threads' to define value in jetty.ini#128
mklette wants to merge 1 commit intopuppetlabs:masterfrom
mklette:dev-jetty_ini-max_threads

Conversation

@mklette
Copy link
Copy Markdown

@mklette mklette commented May 20, 2014

Dear Puppetlabs Team,

do you mind merging our changes for puppetdb module? It became necessary recently as we updated to puppetdb 2.0 and our 214 hosts were trying to connect simultaneously. If you are interested in more detail, we are happy to send them over.

Thanks
Mathias

@stefanandres
Copy link
Copy Markdown
Contributor

I need that too. I did not see your pull request, so I made a new one a few minutes ago, though I added one more feature: it does not add max_threads if not needed: #130

@kbarber
Copy link
Copy Markdown
Contributor

kbarber commented Jul 1, 2014

Lets go with #130. Not having baked-in defaults is a better behaviour.

@kbarber kbarber closed this Jul 1, 2014
@mklette mklette deleted the dev-jetty_ini-max_threads branch July 1, 2014 12:23
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