-
Notifications
You must be signed in to change notification settings - Fork 430
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
feat(interactive): Add Rate Limiter for Control Flow #3368
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #3368 +/- ##
=======================================
Coverage 41.04% 41.04%
=======================================
Files 109 109
Lines 11298 11298
=======================================
Hits 4637 4637
Misses 6661 6661 Continue to review full report in Codecov by Sentry.
|
@@ -45,4 +45,7 @@ public class FrontendConfig { | |||
|
|||
public static final Config<Integer> QUERY_CACHE_SIZE = | |||
Config.intConfig("query.cache.size", 100); | |||
|
|||
public static final Config<Integer> QUERY_PER_SECOND_LIMIT = | |||
Config.intConfig("query.per.second.limit", 100); |
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.
By default there should be no limit, or set it to a value that large enough.
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.
done
docs/storage_engine/groot.md
Outdated
@@ -74,6 +75,7 @@ helm status demo | |||
| ingestor.replicaCount | Number of Ingestor Pod | 2 | | |||
| frontend.service.type | Kubernetes Service type of frontend | NodePort | | |||
| dataset.modern | Load [modern graph](https://tinkerpop.apache.org/docs/current/tutorials/getting-started/) dataset at the start | false | | |||
| query.per.second.limit | the maximum qps can be handled by frontend service | 2147483647 (without limitation) | |
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.
这个加frontend 前缀吧。frontend.query.per.second.limit , 然后把frontend前缀的几个项目都整合到一起吧。
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.
done
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.
LGTM
What do these changes do?
Related issue number
Fixes