Add global variable max_nonsuper_connections
#389
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
@tianx has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator. |
tianx
force-pushed
the
max_total_user_connections
branch
from
ed282b2 to
3c4dc52
Compare
|
@tianx updated the pull request - view changes - changes since last import |
tianx
force-pushed
the
max_total_user_connections
branch
from
3c4dc52 to
2f07838
Compare
|
@tianx updated the pull request - view changes - changes since last import |
tianx
force-pushed
the
max_total_user_connections
branch
from
2f07838 to
21b4b8d
Compare
|
@tianx updated the pull request - view changes - changes since last import |
tianx
force-pushed
the
max_total_user_connections
branch
from
21b4b8d to
a0a68cc
Compare
|
@tianx updated the pull request - view changes - changes since last import |
|
This PR is ready for review. |
tianx
force-pushed
the
max_total_user_connections
branch
from
a0a68cc to
1b9c6fc
Compare
|
@tianx updated the pull request - view changes - changes since last import |
tianx
force-pushed
the
max_total_user_connections
branch
from
1b9c6fc to
ea9cc6c
Compare
|
@tianx updated the pull request - view changes - changes since last import |
tianx
changed the title
max_total_user_connectionsmax_total_nonsuper_user_connections
|
Make it explicit that the limit is enforced on nonsuper users. |
tianx
force-pushed
the
max_total_user_connections
branch
from
ea9cc6c to
7646653
Compare
|
@tianx updated the pull request - view changes - changes since last import |
tianx
force-pushed
the
max_total_user_connections
branch
from
7646653 to
be566db
Compare
|
@tianx updated the pull request - view changes - changes since last import |
tianx
changed the title
max_total_nonsuper_user_connectionsmax_nonsuper_connections
Summary: The new global variable `max_nonsuper_connections` will enforce the limit for the sum of all non-admin connections. This will be useful to limit regular user connections while still allow super users to connect to server. Test Plan: mtr max_nonsuper_connections.test and sys_vars tests.
tianx
force-pushed
the
max_total_user_connections
branch
from
be566db to
3dc7050
Compare
|
@tianx updated the pull request - view changes - changes since last import |
abhinav04sharma
pushed a commit
to abhinav04sharma/mysql-5.6
that referenced
this pull request
Dec 6, 2016
Summary: The new global variable `max_nonsuper_connections` will enforce the limit for the sum of all non-admin connections. This will be useful to limit regular user connections while still allow super users to connect to server. Closes facebook#389 Github Author: Tian Xia <[email protected]> Github PR Sync: {sync, type="child", parent="github", parentrepo="facebook/mysql-5.6", parentprnum="389", parentprfbid="333592887017230"} Test Plan: mtr max_nonsuper_connections.test and sys_vars tests. Reviewers: arahut, anca, svcscm Reviewed By: anca, svcscm Subscribers: svcscm, ebergen, webscalesql-eng@ Differential Revision: https://phabricator.intern.facebook.com/D4123732 Tasks: 14204602 Signature: t1:4123732:1478568243:49c33f511fdbf72c5badccef652e9c8852f82842
hermanlee
pushed a commit
that referenced
this pull request
Jan 31, 2017
Summary: The new global variable `max_nonsuper_connections` will enforce the limit for the sum of all non-admin connections. This will be useful to limit regular user connections while still allow super users to connect to server. Closes #389 Reviewed By: anca-agape Differential Revision: D4123732 Pulled By: tianx fbshipit-source-id: 7ffb140
VitaliyLi
pushed a commit
to VitaliyLi/mysql-5.6
that referenced
this pull request
Feb 9, 2017
Summary: The new global variable `max_nonsuper_connections` will enforce the limit for the sum of all non-admin connections. This will be useful to limit regular user connections while still allow super users to connect to server. Closes facebook#389 Reviewed By: anca-agape Differential Revision: D4123732 Pulled By: tianx fbshipit-source-id: 7ffb140
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary:
The new global variable
max_nonsuper_connectionswill enforce the limit forthe sum of all non-admin connections. This will be useful to limit regular user
connections while still allow super users to connect to server.
Test Plan:
mtr max_nonsuper_connections.test and sys_vars tests.