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

fix(sessds): Optimize the parameters for lower latency #12841

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ieQu1
Copy link
Member

@ieQu1 ieQu1 commented Apr 7, 2024

Fixes EMQX-12051

Release version: v/e5.7

Summary

Optimize storage and session polling parameters to trade throughput for the lower latency.

PR Checklist

Please convert it to a draft if any of the following conditions are not met. Reviewers may skip over until all the items are checked:

  • Added tests for the changes
  • Added property-based tests for code which performs user input validation
  • Changed lines covered in coverage report
  • Change log has been added to changes/(ce|ee)/(feat|perf|fix|breaking)-<PR-id>.en.md files
  • For internal contributor: there is a jira ticket to track this change
  • Created PR to emqx-docs if documentation update is required, or link to a follow-up jira ticket
  • Schema changes are backward compatible

Checklist for CI (.github/workflows) changes

  • If changed package build workflow, pass this action (manual trigger)
  • Change log has been added to changes/ dir for user-facing artifacts update

@ieQu1 ieQu1 requested review from lafirest and a team as code owners April 7, 2024 00:10
@ieQu1 ieQu1 force-pushed the dev/EMQX-12051-reduce-latency branch 3 times, most recently from 3792338 to e6ec206 Compare April 7, 2024 14:09
@ieQu1 ieQu1 force-pushed the dev/EMQX-12051-reduce-latency branch from e578ca9 to 00f4d5a Compare April 10, 2024 11:20
@keynslug
Copy link
Contributor

I'll work on taming test flakiness shortly in the meantime.

@ieQu1 ieQu1 force-pushed the dev/EMQX-12051-reduce-latency branch from 00f4d5a to fd4b9dc Compare April 19, 2024 19:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants