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

Added Remix implementation of the TechEmpower Fortunes benchmark #1870

Closed

Conversation

aaronpowell
Copy link
Member

@aaronpowell aaronpowell commented Jun 19, 2023

This builds on top of the pattern from #1866.

This is an attempt to add an implementation of the TechEmpower Fortunes benchmark in Remix. This is interesting to us given Blazor's updates in .NET 8 to support SSR, etc., and also gives another data point for a JavaScript framework that is designed around SSR.

Notes on the implementation:

PS>  .\bombardier-windows-amd64.exe http://localhost:3000/fortunes -d 5s -c 100                                                                           
Bombarding http://localhost:3000/fortunes for 5s using 100 connection(s)
[=============================================================================================================================================================================================================] 5s 
Done!
Statistics        Avg      Stdev        Max
  Reqs/sec       492.30     225.99    1193.12
  Latency      207.76ms    50.49ms   471.49ms
  HTTP codes:
    1xx - 0, 2xx - 2442, 3xx - 0, 4xx - 0, 5xx - 0
    others - 0
  Throughput:     1.88MB/s
  • Included support for running without a DB via NO_DB env var (or npm run start:no-db):
.\bombardier-windows-amd64.exe http://localhost:3000/fortunes -d 5s -c 100                                                                           
Bombarding http://localhost:3000/fortunes for 5s using 100 connection(s)
[=============================================================================================================================================================================================================] 5s 
Done!
Statistics        Avg      Stdev        Max
  Reqs/sec       624.80     348.40    1391.43
  Latency      161.84ms    43.39ms   390.61ms
  HTTP codes:
    1xx - 0, 2xx - 3140, 3xx - 0, 4xx - 0, 5xx - 0
    others - 0
  Throughput:     2.41MB/s

@sebastienros
Copy link
Member

Shouldn't it go in the TechEmpower repository instead?

@DamianEdwards
Copy link
Member

Yeah I think this is better off in TechEmpower itself than in our benchmarks repo. The PR for a Next.js implementation arguably falls into the same bucket but given Next.js' massive popularity it seems like an exception to the rule (until a Next.js implementation is added to TE itself anyway).

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.

3 participants