Skip to content

add a maximum connection parameter and set the default to 1000 #198

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

Merged
merged 3 commits into from
Jul 2, 2013

Conversation

mhellmic
Copy link
Contributor

configure the max connections parameter.

@mhellmic
Copy link
Contributor Author

the second commit changes the mysql-config test so that the build should pass. Is it run automatically?

@hunner
Copy link
Contributor

hunner commented Jun 26, 2013

Looks like it doesn't currently merge with master (due to the tmpdir changes) so the Travis tests would run automatically, but since they can't merge cleanly they don't run at all. Could you rebase this onto the current master? Thanks!

basedir = <%= @basedir %>
datadir = <%= @datadir %>
tmpdir = /tmp
max_connections = <%= @max_connections %>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does mysql have a default max_connections that we could use? It seems like adding this may be considered backwards incompatible otherwise, since it previously was unaccounted for.

@mhellmic
Copy link
Contributor Author

Hi,

I fixed the merge issue, I think. I fetched and rebased with your master to incorporate the changes.

For the defaults: The mysql 5.5 says the following:
"The default value is 151."
(....)
"Linux or Solaris should be able to support at 500 to 1000 simultaneous connections routinely"

I changed the default to 151, then.

source:
https://dev.mysql.com/doc/refman/5.5/en/too-many-connections.html

apenney added a commit that referenced this pull request Jul 2, 2013
add a maximum connection parameter and set the default to 1000
@apenney apenney merged commit aa5d1f0 into puppetlabs:master Jul 2, 2013
@mhellmic mhellmic deleted the maxconnections branch July 2, 2013 18:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants