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

limit the max session when authenticate #4079

Closed
HarrisChu opened this issue Mar 25, 2022 · 1 comment
Closed

limit the max session when authenticate #4079

HarrisChu opened this issue Mar 25, 2022 · 1 comment
Labels
type/enhancement Type: make the code neat or more efficient

Comments

@HarrisChu
Copy link
Contributor

HarrisChu commented Mar 25, 2022

if we have a lot of sessions, it would make meta slowly when

  • update sessions
  • get sessions in heartbeat. MetaClient::loadSessions()

currently, the default value for max_allowed_connections is 1<< 63 -1 per graph, it's too large.

expected:

  1. use a default limit for max_allowed_sessions in meta. e.g. 1000
  2. if exceed, then cannot create new session except user change the configuration or release other sessions.

refer:
https://discuss.nebula-graph.com.cn/t/topic/8042/18

@HarrisChu HarrisChu added the type/enhancement Type: make the code neat or more efficient label Mar 25, 2022
@HarrisChu
Copy link
Contributor Author

#3729

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/enhancement Type: make the code neat or more efficient
Projects
None yet
Development

No branches or pull requests

1 participant