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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Performance] 馃煝 MTTFB 38% improved by - Flux.log removal POST /api/login #297

Open
ghkdqhrbals opened this issue Dec 13, 2023 · 1 comment
Assignees
Labels
feature: performance performance improvement

Comments

@ghkdqhrbals
Copy link
Owner

ghkdqhrbals commented Dec 13, 2023

POST /api/login

  • Implements
    • caching FriendsList DTO into Redis
  • Resource consumption
    • chat-db : 0.23 cpu
    • chat-server : 0.69 cpu
  • Improvements
    • avg MTTFB : 108.5 -> 79.2 馃憤 38% improvement
    • avg TPS : 16.6 -> 15.6 馃憥 7% worse - no big difference

Average of MTTFB(Mean Test Time of First Byte) is 38% improved by removing .log(). And TPS isn't that big difference between before and after. Small changes may be affected by things like thread initialization.

This stress test started with 99 concurrent users.

Test during 3 min

Before

Total Vusers 99
Agent 1
Processes Threads 3 / 33
Sample Ignore 0
TPS 16.6
Peak TPS 24
Mean Test Time 5,845.54
Executed Tests 2,891
Successful Tests 2,891
Errors 0
  • AVG MTTFB : 108ms
image image

After

Total Vusers 99
Agent 1
ProcessesThreads 3 / 33
Sample Ignore 0
TPS 15.6
Peak TPS 21
Mean Test Time 6,221.57
Executed Tests 2,714
Successful Tests 2,714
Errors 0
  • AVG MTTFB : 79ms
image image
@ghkdqhrbals ghkdqhrbals added the feature: performance performance improvement label Dec 13, 2023
@ghkdqhrbals ghkdqhrbals self-assigned this Dec 13, 2023
@ghkdqhrbals ghkdqhrbals changed the title [Performance] removing .log() in Flux : User-Server [Performance] MTTFB 38% improvement Dec 13, 2023
@ghkdqhrbals
Copy link
Owner Author

ghkdqhrbals commented Dec 13, 2023

After

This stress test started with 1000 concurrent users.

But server dead 馃槀

Test during 3 min

Total Vusers 1,000
Agent 1
Processes Threads 10 / 100
Sample Ignore 0
TPS 10.9
Peak TPS 22
Mean Test Time 40,830.90聽ms
Executed Tests 2,227
Success Tests 1,888
Errors 339 ( This means that server cannot handle with 1,000 concurrent users )

image

@ghkdqhrbals ghkdqhrbals changed the title [Performance] MTTFB 38% improvement [Performance] MTTFB 38% improvement - Flux.log removal Dec 13, 2023
@ghkdqhrbals ghkdqhrbals changed the title [Performance] MTTFB 38% improvement - Flux.log removal [Performance] MTTFB 38% improvement - Flux.log removal POST /api/login Dec 14, 2023
@ghkdqhrbals ghkdqhrbals changed the title [Performance] MTTFB 38% improvement - Flux.log removal POST /api/login [Performance] 馃煝 MTTFB 38% improvement - Flux.log removal POST /api/login Jan 3, 2024
@ghkdqhrbals ghkdqhrbals changed the title [Performance] 馃煝 MTTFB 38% improvement - Flux.log removal POST /api/login [Performance] 馃煝 MTTFB 38% improved by - Flux.log removal POST /api/login Jan 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature: performance performance improvement
Projects
Development

No branches or pull requests

1 participant