This repository was archived by the owner on Sep 8, 2026. It is now read-only.
Archive FTWDB as an energy-storage test lab - #37
Merged
Merged
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_761a9d89-254d-476d-9534-db4daf85c3e2) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
FTW now embeds DuckDB in Core for history and retains SQLite for configuration. Retire FTWDB as a product and preserve the repository as a frozen energy-storage test lab.
The README points to the adopted Core integration and an archive record covering energy semantics, retries, durable acknowledgements, recovery, retention, format compatibility, memory and migration lessons. Historical architecture, roadmap and release pages now carry retirement notices. The archive includes the September engine comparison, 74 runner/result files with provenance hashes, and the scoped large Core import result. Workstation paths and process dumps are removed; recorded timings and validation stay unchanged.
The original twelve local changed files are preserved byte-for-byte on
archive/final-lab-snapshot-2026-09-08, commit3c701a89fc1c70eba1bb4e39ce2b4d87aec82dc2. That snapshot is separate from the later fixes already on main; this PR does not replace the current storage implementation with the older checkout.Validation: original file hashes match the preserved snapshot on GitHub; all 74 archived artifact hashes, JSON and Python syntax pass; reported medians reproduce from the raw repeats; local document links and fixture identity pass; git diff --check is clean. The preserved database workloads have not been rerun. Existing portable CI runs on this PR before the repository is archived.
Note
Low Risk
Documentation and archived benchmark artifacts only; no changes to the FTW runtime or supported storage path.
Overview
Retires FTWDB as an active product and reframes the repo as a frozen energy-storage test lab. The root and bench READMEs now state the 8 September 2026 archive date, redirect new storage work to FTW Core (DuckDB history + SQLite config), and link the archive record, engine comparison report, and
archive/final-lab-snapshot-2026-09-08branch. Install/release language shifts to “last published” and historical commands are explicitly research-only.Adds
bench/engine-evaluation-2026-09-07/as a portable snapshot of the 7 September 2026 local engine shootout: sanitized JSON results (timings/validation unchanged; workstation paths and cgroup/process dumps removed; memory peaks kept),provenance.jsonwith per-file SHA transforms, and reproducible harness code (Docker-pinnedrun.py, SQLite/ DuckDB/ClickHouse adapters, parquet/restore/semantics probes). The superseded report generator is omitted so the archive does not resurrect an outdated product recommendation; runners useFTWDB_EVAL_FIXTUREinstead of a machine-local path.Reviewed by Cursor Bugbot for commit bea0a38. Bugbot is set up for automated code reviews on this repo. Configure here.