Skip to content

docs: compare impit against other impersonation clients - #523

Merged
barjin merged 2 commits into
masterfrom
docs/comparison-matrix
Aug 27, 2026
Merged

docs: compare impit against other impersonation clients#523
barjin merged 2 commits into
masterfrom
docs/comparison-matrix

Conversation

@barjin

@barjin barjin commented Aug 26, 2026

Copy link
Copy Markdown
Member

Throughput, package size, and profile counts measured locally against a Node.js HTTP/2 server; numbers are indicative and worth rerunning on CI hardware.

@github-actions github-actions Bot added this to the 148th sprint - Tooling team milestone Aug 26, 2026
@github-actions github-actions Bot added the t-tooling Issues with this label are in the ownership of the tooling team. label Aug 26, 2026
@barjin
barjin requested a review from Pijukatel August 26, 2026 09:16

@Pijukatel Pijukatel left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was this measured before #524?

I am not sure about such a benchmark comparison in readme. It does not have references to the versions used and it does not show the exact benchmarking suite used.

Maybe it would be better to have a CI job that will report these results together with having the benchmark code available for anyone to run it.

(And update the benchmark section in readme for each new release)

@Pijukatel Pijukatel left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, as a first step, but I think my comments should be addressed before closing #469

@barjin
barjin merged commit 863ddd0 into master Aug 27, 2026
8 checks passed
@barjin
barjin deleted the docs/comparison-matrix branch August 27, 2026 12:30
barjin added a commit that referenced this pull request Sep 4, 2026
Closes #469 — replaces the hand-measured table from #523 with a
benchmark harness that measures the latest published clients against a
local server in CI, monthly or on demand, and opens a PR when the
numbers move.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t-tooling Issues with this label are in the ownership of the tooling team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants