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

Make the server http compliant #33

Merged
merged 2 commits into from
Jul 1, 2021
Merged

Make the server http compliant #33

merged 2 commits into from
Jul 1, 2021

Conversation

sansyrox
Copy link
Member

@sansyrox sansyrox commented Jun 30, 2021

Fixes #31

This PR will fix (#31)

@sansyrox sansyrox changed the title Make the server http compliance Make the server http compliant Jul 1, 2021
@sansyrox
Copy link
Member Author

sansyrox commented Jul 1, 2021

The dropping of request was due to poor error handling.

It can now process 100,000 reqeuests without dropping.

➜  ~ oha -n 100000 http://localhost:5000
Summary:
  Success rate:	1.0000
  Total:	35.7292 secs
  Slowest:	0.0700 secs
  Fastest:	0.0010 secs
  Average:	0.0179 secs
  Requests/sec:	2798.8274

  Total data:	1.72 MiB
  Size/request:	18 B
  Size/sec:	49.20 KiB

Response time histogram:
  0.005 [7091]  |■■■■■■■
  0.010 [29564] |■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■
  0.014 [9949]  |■■■■■■■■■■
  0.019 [4959]  |■■■■■
  0.024 [18979] |■■■■■■■■■■■■■■■■■■■■
  0.029 [19959] |■■■■■■■■■■■■■■■■■■■■■
  0.033 [6878]  |■■■■■■■
  0.038 [945]   |■
  0.043 [840]   |
  0.048 [638]   |
  0.053 [198]   |

Latency distribution:
  10% in 0.0064 secs
  25% in 0.0087 secs
  50% in 0.0194 secs
  75% in 0.0258 secs
  90% in 0.0295 secs
  95% in 0.0318 secs
  99% in 0.0433 secs

Details (average, fastest, slowest):
  DNS+dialup:	0.0026 secs, 0.0016 secs, 0.0030 secs
  DNS-lookup:	0.0000 secs, 0.0000 secs, 0.0001 secs

Status code distribution:
  [200] 100000 responses

@sansyrox sansyrox merged commit 6f81f30 into main Jul 1, 2021
@sansyrox sansyrox deleted the http-compliance branch July 1, 2021 08:24
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.

QPS drops drastically after processing many requests
1 participant