Skip to content

Add expresive pipe error#678

Closed
adrianmjim wants to merge 5 commits intomainfrom
feat/add-expresive-pipe-error
Closed

Add expresive pipe error#678
adrianmjim wants to merge 5 commits intomainfrom
feat/add-expresive-pipe-error

Conversation

@adrianmjim
Copy link
Member

No description provided.

@changeset-bot
Copy link

changeset-bot bot commented Apr 29, 2025

⚠️ No Changeset found

Latest commit: b0e0b8c

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions
Copy link
Contributor

Benchmarks results:

> root@ benchmarks:http:cjs /home/runner/work/monorepo/monorepo
> pnpm run --filter "@inversifyjs/http-benchmarks" benchmarks:run:cjs


> @inversifyjs/http-benchmarks@ benchmarks:run:cjs /home/runner/work/monorepo/monorepo/packages/http/tools/http-benchmarks
> BUILD_TARGET=cjs ./bin/run-cjs.cjs

[CJS] Running benchmarks...

Express Basic Get Request
┌─────────┬───────────────────────────┬──────────────────┬──────────────────┬────────────────────────┬─────────┐
│ (index) │ Task name                 │ Latency avg (ms) │ Latency med (ms) │ Throughput avg (req/s) │ Samples │
├─────────┼───────────────────────────┼──────────────────┼──────────────────┼────────────────────────┼─────────┤
│ 0       │ 'currentInversifyExpress''108.491''43.830''4129.633'             │ 45599   │
│ 1       │ 'express''98.266''42.691''4570.434'             │ 50599   │
│ 2       │ 'NestJSExpress''107.321''44.706''4174.738'             │ 46004   │
└─────────┴───────────────────────────┴──────────────────┴──────────────────┴────────────────────────┴─────────┘
currentInversifyExpress vs express Speedup: 0.904x
currentInversifyExpress vs NestJSExpress Speedup: 0.989x

Express v4 Basic Get Request
┌─────────┬────────────────────────────┬──────────────────┬──────────────────┬────────────────────────┬─────────┐
│ (index) │ Task name                  │ Latency avg (ms) │ Latency med (ms) │ Throughput avg (req/s) │ Samples │
├─────────┼────────────────────────────┼──────────────────┼──────────────────┼────────────────────────┼─────────┤
│ 0       │ 'currentInversifyExpress4''96.144''42.353''4687.405'             │ 51814   │
│ 1       │ 'express4''92.692''41.442''4876.616'             │ 53930   │
└─────────┴────────────────────────────┴──────────────────┴──────────────────┴────────────────────────┴─────────┘
currentInversifyExpress4 vs express4 Speedup: 0.961x

Fastify Basic Get Request
┌─────────┬───────────────────────────┬──────────────────┬──────────────────┬────────────────────────┬─────────┐
│ (index) │ Task name                 │ Latency avg (ms) │ Latency med (ms) │ Throughput avg (req/s) │ Samples │
├─────────┼───────────────────────────┼──────────────────┼──────────────────┼────────────────────────┼─────────┤
│ 0       │ 'currentInversifyFastify''40.637''33.023''12327.915'            │ 124612  │
│ 1       │ 'fastify''32.394''27.222''15433.775'            │ 155477  │
│ 2       │ 'NestJSFastify''35.420''29.980''14021.213'            │ 141191  │
└─────────┴───────────────────────────┴──────────────────┴──────────────────┴────────────────────────┴─────────┘
currentInversifyFastify vs fastify Speedup: 0.799x
currentInversifyFastify vs NestJSFastify Speedup: 0.879x

Hono Basic Get Request
┌─────────┬────────────────────────┬──────────────────┬──────────────────┬────────────────────────┬─────────┐
│ (index) │ Task name              │ Latency avg (ms) │ Latency med (ms) │ Throughput avg (req/s) │ Samples │
├─────────┼────────────────────────┼──────────────────┼──────────────────┼────────────────────────┼─────────┤
│ 0       │ 'currentInversifyHono''38.025''31.704''13194.996'            │ 132842  │
│ 1       │ 'hono''34.956''30.121''14190.058'            │ 142889  │
└─────────┴────────────────────────┴──────────────────┴──────────────────┴────────────────────────┴─────────┘
currentInversifyHono vs hono Speedup: 0.930x

@github-actions
Copy link
Contributor

github-actions bot commented May 1, 2025

Benchmarks results:

> root@ benchmarks:http:cjs /home/runner/work/monorepo/monorepo
> pnpm run --filter "@inversifyjs/http-benchmarks" benchmarks:run:cjs


> @inversifyjs/http-benchmarks@ benchmarks:run:cjs /home/runner/work/monorepo/monorepo/packages/http/tools/http-benchmarks
> BUILD_TARGET=cjs ./bin/run-cjs.cjs

[CJS] Running benchmarks...

Express Basic Get Request
┌─────────┬───────────────────────────┬──────────────────┬──────────────────┬────────────────────────┬─────────┐
│ (index) │ Task name                 │ Latency avg (ms) │ Latency med (ms) │ Throughput avg (req/s) │ Samples │
├─────────┼───────────────────────────┼──────────────────┼──────────────────┼────────────────────────┼─────────┤
│ 0       │ 'currentInversifyExpress''109.724''46.592''4058.623'             │ 44951   │
│ 1       │ 'express''100.574''42.800''4472.978'             │ 49517   │
│ 2       │ 'NestJSExpress''110.034''45.583''4063.695'             │ 44840   │
└─────────┴───────────────────────────┴──────────────────┴──────────────────┴────────────────────────┴─────────┘
currentInversifyExpress vs express Speedup: 0.907x
currentInversifyExpress vs NestJSExpress Speedup: 0.999x

Express v4 Basic Get Request
┌─────────┬────────────────────────────┬──────────────────┬──────────────────┬────────────────────────┬─────────┐
│ (index) │ Task name                  │ Latency avg (ms) │ Latency med (ms) │ Throughput avg (req/s) │ Samples │
├─────────┼────────────────────────────┼──────────────────┼──────────────────┼────────────────────────┼─────────┤
│ 0       │ 'currentInversifyExpress4''97.682''42.948''4609.929'             │ 50882   │
│ 1       │ 'express4''93.802''41.455''4802.759'             │ 53123   │
└─────────┴────────────────────────────┴──────────────────┴──────────────────┴────────────────────────┴─────────┘
currentInversifyExpress4 vs express4 Speedup: 0.960x

Fastify Basic Get Request
┌─────────┬───────────────────────────┬──────────────────┬──────────────────┬────────────────────────┬─────────┐
│ (index) │ Task name                 │ Latency avg (ms) │ Latency med (ms) │ Throughput avg (req/s) │ Samples │
├─────────┼───────────────────────────┼──────────────────┼──────────────────┼────────────────────────┼─────────┤
│ 0       │ 'currentInversifyFastify''41.388''35.185''12066.985'            │ 121756  │
│ 1       │ 'fastify''32.275''28.055''15283.542'            │ 153996  │
│ 2       │ 'NestJSFastify''36.672''32.733''13711.503'            │ 137890  │
└─────────┴───────────────────────────┴──────────────────┴──────────────────┴────────────────────────┴─────────┘
currentInversifyFastify vs fastify Speedup: 0.790x
currentInversifyFastify vs NestJSFastify Speedup: 0.880x

Hono Basic Get Request
┌─────────┬────────────────────────┬──────────────────┬──────────────────┬────────────────────────┬─────────┐
│ (index) │ Task name              │ Latency avg (ms) │ Latency med (ms) │ Throughput avg (req/s) │ Samples │
├─────────┼────────────────────────┼──────────────────┼──────────────────┼────────────────────────┼─────────┤
│ 0       │ 'currentInversifyHono''38.024''32.462''13068.031'            │ 131861  │
│ 1       │ 'hono''36.182''32.414''13858.149'            │ 139543  │
└─────────┴────────────────────────┴──────────────────┴──────────────────┴────────────────────────┴─────────┘
currentInversifyHono vs hono Speedup: 0.943x

@github-actions
Copy link
Contributor

github-actions bot commented May 1, 2025

Benchmarks results:

> root@ benchmarks:http:cjs /home/runner/work/monorepo/monorepo
> pnpm run --filter "@inversifyjs/http-benchmarks" benchmarks:run:cjs


> @inversifyjs/http-benchmarks@ benchmarks:run:cjs /home/runner/work/monorepo/monorepo/packages/http/tools/http-benchmarks
> BUILD_TARGET=cjs ./bin/run-cjs.cjs

[CJS] Running benchmarks...

Express Basic Get Request
┌─────────┬───────────────────────────┬──────────────────┬──────────────────┬────────────────────────┬─────────┐
│ (index) │ Task name                 │ Latency avg (ms) │ Latency med (ms) │ Throughput avg (req/s) │ Samples │
├─────────┼───────────────────────────┼──────────────────┼──────────────────┼────────────────────────┼─────────┤
│ 0       │ 'currentInversifyExpress''109.896''43.986''4058.490'             │ 44874   │
│ 1       │ 'express''100.275''43.108''4485.321'             │ 49493   │
│ 2       │ 'NestJSExpress''109.497''45.154''4067.205'             │ 45074   │
└─────────┴───────────────────────────┴──────────────────┴──────────────────┴────────────────────────┴─────────┘
currentInversifyExpress vs express Speedup: 0.905x
currentInversifyExpress vs NestJSExpress Speedup: 0.998x

Express v4 Basic Get Request
┌─────────┬────────────────────────────┬──────────────────┬──────────────────┬────────────────────────┬─────────┐
│ (index) │ Task name                  │ Latency avg (ms) │ Latency med (ms) │ Throughput avg (req/s) │ Samples │
├─────────┼────────────────────────────┼──────────────────┼──────────────────┼────────────────────────┼─────────┤
│ 0       │ 'currentInversifyExpress4''96.494''42.431''4659.138'             │ 51555   │
│ 1       │ 'express4''93.647''41.668''4823.361'             │ 53203   │
└─────────┴────────────────────────────┴──────────────────┴──────────────────┴────────────────────────┴─────────┘
currentInversifyExpress4 vs express4 Speedup: 0.966x

Fastify Basic Get Request
┌─────────┬───────────────────────────┬──────────────────┬──────────────────┬────────────────────────┬─────────┐
│ (index) │ Task name                 │ Latency avg (ms) │ Latency med (ms) │ Throughput avg (req/s) │ Samples │
├─────────┼───────────────────────────┼──────────────────┼──────────────────┼────────────────────────┼─────────┤
│ 0       │ 'currentInversifyFastify''41.562''35.275''12059.771'            │ 121587  │
│ 1       │ 'fastify''32.334''27.733''15333.634'            │ 154362  │
│ 2       │ 'NestJSFastify''36.598''30.886''13639.733'            │ 137437  │
└─────────┴───────────────────────────┴──────────────────┴──────────────────┴────────────────────────┴─────────┘
currentInversifyFastify vs fastify Speedup: 0.786x
currentInversifyFastify vs NestJSFastify Speedup: 0.884x

Hono Basic Get Request
┌─────────┬────────────────────────┬──────────────────┬──────────────────┬────────────────────────┬─────────┐
│ (index) │ Task name              │ Latency avg (ms) │ Latency med (ms) │ Throughput avg (req/s) │ Samples │
├─────────┼────────────────────────┼──────────────────┼──────────────────┼────────────────────────┼─────────┤
│ 0       │ 'currentInversifyHono''38.519''33.219''13009.215'            │ 131348  │
│ 1       │ 'hono''35.443''29.116''14175.732'            │ 142456  │
└─────────┴────────────────────────┴──────────────────┴──────────────────┴────────────────────────┴─────────┘
currentInversifyHono vs hono Speedup: 0.918x

@adrianmjim adrianmjim closed this May 3, 2025
@notaphplover notaphplover deleted the feat/add-expresive-pipe-error branch June 18, 2025 11:25
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.

1 participant