Skip to content

docs(readme): ktor and Exposed index figures for the current fork - #522

Merged
bompus merged 1 commit into
fork/consolidatedfrom
docs/readme-kotlin-after-promotion
Oct 18, 2026
Merged

bompus merged 1 commit into
fork/consolidatedfrom
docs/readme-kotlin-after-promotion

Conversation

@bompus

@bompus bompus commented Oct 18, 2026 •

Copy link
Copy Markdown
Owner

A full index of ktor is 26% faster and of JetBrains Exposed 27% faster at fork 5b5ac434 than at d161ef74, with 19% and 17% lower peak memory and identical graphs (codegraph init, Node 24, median of 3 alternating runs per build). The README's mid-size paragraph still quoted #515 alone (22% and 19%, measured before #517 and #518 landed), so this updates that sentence.

Change

One sentence in README.md "Measured results", in the paragraph on the 2026-10-15 mid-size comparison. It now gives the figures for the fork as it is after #515 (Kotlin companion check), #517 (shared compiled-pattern cache) and #518 (no checkpoints inside the resolution commits). It also says that both corpora still take longer than upstream e8ec7fd8 but now peak lower than it. No code changes.

Results

Corpus, measure d161ef7 median (range) 5b5ac43 median (range) Floor Benefit
ktor, wall 13.46 s (13.40-13.76) 9.91 s (9.88-10.16) 0.66 s 26.4% faster
ktor, CPU 52.47 s (52.18-53.09) 36.29 s (36.24-37.29) 1.71 s 30.8% less
ktor, peak memory 2,479 MiB (2,418-2,514) 1,996 MiB (1,957-2,004) 66 MiB 19.5% lower
Exposed, wall 9.55 s (9.48-9.75) 6.95 s (6.92-7.07) 0.16 s 27.2% faster
Exposed, CPU 36.08 s (35.95-36.53) 26.43 s (26.38-26.63) 0.29 s 26.7% less
Exposed, peak memory 2,427 MiB (2,402-2,502) 2,022 MiB (1,992-2,059) 91 MiB 16.7% lower

Upstream e8ec7fd8 in the same runs: ktor 7.62 s, 2,636 MiB; Exposed 5.71 s, 2,133 MiB. Upstream's graphs differ from the fork's, as in the mid-size comparison.

The floor is the spread (max - min) over six runs of two identical copies of the 5b5ac434 build. A change counts only when the medians differ by more than the floor and the two ranges do not overlap; every row above meets that. Graph hashes were identical across d161ef7 and 5b5ac43 on all 18 fork runs (ktor 57e9c5136b5e, Exposed b332758c901b). The run used 24 accepted runs, with none rejected, on a 16-vCPU WSL2 host. Each run was a systemd-run scope capped at 16 GiB, at normal priority. A run was admitted only with load under 2 and no swap.

Checks

README rows checked

  • Mid-size paragraph, ktor and Exposed sentence: moved; updated here.
  • Full-index table, pretix and supabase rows (fork column 11.3 s / 3.35 GiB and 20.8 s / 3.81 GiB): not changed. Same method, d161ef7 against 5b5ac43. pretix wall showed no clear change (9.12 to 8.86 s, -2.9%, floor 0.34 s, ranges overlap). Peak memory also showed no clear change (-2.2%). supabase wall showed no clear change (-0.9%) and neither did peak memory (-0.9%). CPU fell 4.1% on pretix and 3.8% on supabase, but the table does not quote CPU. Graphs were identical.
  • Exposed's 1,020 extension call links: unaffected, since the graphs are identical.
  • Adaptive pool sentence (11-48% faster at 56-132% more memory against its pre-change baseline): historical, not affected.
  • No site page or docs/benchmarks/ file quotes the ktor or Exposed figures.

Not covered: the full-index and sync tables compare fork ba972275 with upstream 3f714bcc (2026-10-14, Node and Bun). Refreshing them for the current fork needs that whole benchmark re-run, with an upstream build. Bun was not measured here.


Summary by cubic

Updates the README's mid-size benchmark paragraph with figures for the current fork. The previous sentence quoted only #515; after #517 and #518, the fork at 5b5ac434 indexes ktor 26% faster and Exposed 27% faster than d161ef74, with 19% and 17% lower peak memory and identical graphs. The paragraph also notes the fork still takes longer than upstream but now peaks below it.

Written for commit 337745d. Summary will update on new commits.

View guided diff Turn on auto-fix

The mid-size paragraph quoted #515 alone (ktor 22%, Exposed 19% faster than
d161ef7). With #517 and #518 also in the fork, a full index at 5b5ac43 is
26% and 27% faster than d161ef7 at 19% and 17% lower peak memory, identical
graphs; it still takes longer than upstream e8ec7fd but peaks lower.
@bompus
bompus force-pushed the docs/readme-kotlin-after-promotion branch from 337745d to a493f39 Compare October 18, 2026 04:04
@coderabbitai

coderabbitai Bot commented Oct 18, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: Repository YAML (base), Organization UI (inherited)
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 5ca9cfbf-e04a-4083-9f08-ce7c982bf0c5
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@greptile-apps

greptile-apps Bot commented Oct 18, 2026 •

Copy link
Copy Markdown

RetriggerView in GreptileConfidence Score: 4/5

[Low impact] Updates benchmark figures in the readme.

The PR appears safe to merge, with a non-blocking fix needed for the benchmark source reference.

Findings

  1. P2 Benchmark source is missing ▶
Fix with agent prompt
### Issue 1
README.md:202
The new measurement source is still `#PRNUM`, so readers cannot find the benchmark results cited by this sentence. Replace it with a link to PR #522, which contains the measurements and method.

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

Summary

Updates the README's ktor and Exposed figures for the fork at 5b5ac434.

  • The README updates full-index results for ktor and Exposed.

Reviews (1) · Last reviewed commit: "docs(readme): ktor and Exposed figures f..." · Reviewed by Greptile

Comment thread README.md Outdated
@bompus
bompus merged commit 0af3e1b into fork/consolidated Oct 18, 2026
2 checks passed
@bompus
bompus deleted the docs/readme-kotlin-after-promotion branch October 18, 2026 04:21
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