Repository navigation
Conversation
|
Benchmark coverage corrected (commit
Fresh Linux qualification is running before the retention comparison is retried. |
|
Review/update plan after checking current upstream: This remains useful as the independent measurement tool for #2153 and #2145. It also carries an older copy of #2160 ( The next useful gate is a fresh native SQLite smoke qualification with baseline, candidate and harness revisions pinned separately. Retain complete logical-data parity, equivalent cache and endpoint coverage, zero-drop/error checks, resource accounting and durability checks. Diagnostic/profile runs should stay separate from timing results, and benchmark-only workflows must retain their no-publish/no-deploy guards. The comparison markers currently in the body identify historical revisions. They should be updated only when a new comparison is actually requested and run. The successful qualification and comparisons remain valid for their recorded revisions; the excluded failed benchmark run is not evidence for a speed claim. I would land this tool-only change separately, then use it to measure the focused retention change and a narrower storage candidate against contemporary upstream. No new benchmark or rebase was performed during this discussion review. |
|
I’m rebasing the harness onto current upstream I’m preserving the old |
4de4194 to
6a924cc
Compare
|
Review feedback addressed (commit
Normal CI has also passed on this head. Post-push Chromium validation passed 148 checks with three explicit fixture-dependent skips against the unchanged server/frontend. No primary/diagnostic benchmark was requested or run, and no new speed claim is made. |
|
Review feedback addressed (commit
This is controller qualification with identical production code, not a new speed comparison. Historical measurements remain labelled with their original revisions. |
Fixes #2150; related to #2146.
The standalone SQLite harness now qualifies upstream v3.15.0 (
7fa86fe1) with separately pinned baseline, candidate and harness revisions. Upstream was merged normally, preserving review history; production code matches that revision.The manifest covers all 22 application tables and 211 selected columns, including observer rowid and
client_node_names. Added tests cover the legacy 18/21-table layouts and shifting receipt timestamps while preserving repeater clocks. Existing resource, data-parity, durability, endpoint-coverage and no-publish/no-deploy gates remain intact.Local validation on
28f91906:163341aaand passes at28f91906.Fresh Linux S qualification and normal CI passed on this head. Post-push Chromium passed 148 checks with three explicit fixture-dependent skips. The small A/A run validates the controller; it is not a speed comparison. No operator/customizer changes or larger measurements are included.
Historical evidence retains its original revisions: October 20 qualification, its CI, original qualification, retention comparison, and optional-backend comparison. These are not relabelled as v3.15.0 measurements.