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.

Sign up for GitHub

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

Jump to bottom

[QE]optimization of rate-limit #2814

Closed
Sophie-Xie opened this issue Sep 8, 2021 · 1 comment · Fixed by #2860
Closed

[QE]optimization of rate-limit #2814

Sophie-Xie opened this issue Sep 8, 2021 · 1 comment · Fixed by #2860
Assignees
@critical27
Labels
type/enhancement Type: make the code neat or more efficient
Milestone
v2.6.0

Comments

@Sophie-Xie
Copy link
Contributor

Now the unit of rate limit is byte, we need to optimization it for more easier to understand. Ex. percent.
@Sophie-Xie Sophie-Xie added the type/enhancement Type: make the code neat or more efficient label Sep 8, 2021
@jamieliu1023 jamieliu1023 mentioned this issue Sep 11, 2021
Closed
@critical27 critical27 mentioned this issue Sep 22, 2021
Closed
@critical27 critical27 changed the title [QE]optimization the unit of rate-limit [QE]optimization of rate-limit Sep 22, 2021
@critical27
Copy link
Contributor

Will make it more dynamically configurable in #2860

@critical27 critical27 linked a pull request Sep 23, 2021 that will close this issue
Enable prefix bloom filter by default #2860
Merged
@Sophie-Xie Sophie-Xie added this to the v2.6.0 milestone Sep 26, 2021
@jamieliu1023 jamieliu1023 mentioned this issue Oct 2, 2021
Closed
This was referenced Oct 20, 2021
Closed
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@critical27 critical27

Labels
type/enhancement Type: make the code neat or more efficient
Projects
None yet
Milestone
v2.6.0
Development

Successfully merging a pull request may close this issue.

Enable prefix bloom filter by default critical27/nebula
2 participants
@critical27 @Sophie-Xie