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

perf: remove `async` from dispatch; will be 16% faster #550

Merged

yusukebe merged 1 commit into main from perf/remove-async-dispatch

Sep 26, 2022

Member

yusukebe commented Sep 24, 2022 •

edited

Remove async from dispatch function in hono.ts. A single handler that does not use async will be about 16% faster on Bun:

v2.2.1:

Reqs/sec    158951.18   43864.54  202188.36
Latency        1.26ms     1.39ms    41.26ms
HTTP codes:
1xx - 0, 2xx - 1589211, 3xx - 0, 4xx - 0, 5xx - 0
others - 0
Throughput:    21.67MB/s

This PR:

Reqs/sec    188225.08   24307.40  218436.45
Latency        1.06ms   109.58us    12.83ms
HTTP codes:
1xx - 0, 2xx - 1880915, 3xx - 0, 4xx - 0, 5xx - 0
others - 0
Throughput:    25.65MB/s

This benchmark using: https://github.com/SaltyAom/bun-http-framework-benchmark


          perf: remove async from dispatch; will be 16% faster

294a12a

yusukebe merged commit e4c8a6f into main

yusukebe deleted the perf/remove-async-dispatch branch

September 26, 2022 12:12

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment