Skip to content

Remove the on demand database backup and database.data_store - #1041

Open
epompeii wants to merge 1 commit into
u/ep/s3-r2/disaster-recoveryfrom
u/ep/s3-r2/server-backup
Open

epompeii wants to merge 1 commit into
u/ep/s3-r2/disaster-recoveryfrom
u/ep/s3-r2/server-backup

Conversation

@epompeii

Copy link
Copy Markdown
Member

The on demand database backup predates continuous replication and is unused. Its config section, database.data_store, took an AWS S3 access point ARN and was the only reason bencher_schema depended on the AWS SDK. Continuous replication under plus.disaster_recovery is the supported path.

Changes

  • POST /v0/server/backup and its OPTIONS handler are removed, and the OpenAPI spec is regenerated.
  • bencher server backup is removed from the CLI.
  • database.data_store and its types are removed from the server config; bencher_schema drops aws-sdk-s3 and aws-credential-types, and the root manifest drops async-compression and rusqlite, which had no user left.
  • The manual backup workflow is deleted.
  • Docs in all 9 languages lose the Backups section and its CLI chunks; the database config reference and the shared example lose data_store; the disaster recovery pages no longer refer to backups.

Breaking changes

  • POST /v0/server/backup and bencher server backup are gone.
  • A config that still carries database.data_store is ignored. Remove the section.

Second of a stack of three, on top of the plus.disaster_recovery rename. The last PR moves the container registry storage onto object_store, at which point the AWS SDK leaves the workspace.

The POST /v0/server/backup endpoint, the bencher server backup CLI
subcommand, and the database.data_store server config section are gone.
Continuous replication under plus.disaster_recovery is the supported
path, so bencher_schema no longer depends on the AWS SDK.
A config that still carries database.data_store is ignored.
@epompeii
epompeii added this pull request to stack #1042 September 18, 2026 06:48
@github-actions

Copy link
Copy Markdown
Contributor

🐰 Bencher Report

ProjectBencher
Branchu/ep/s3-r2/server-backup
Testbedintel-v1
Click to view all benchmark results
BenchmarkLatencyBenchmark Result
microseconds (µs)
(Result Δ%)
Upper Boundary
microseconds (µs)
(Limit %)
Adapter::Json📈 view plot
🚷 view threshold
5.24 µs
(+8.35%)Baseline: 4.83 µs
5.86 µs
(89.44%)
Adapter::Magic (JSON)📈 view plot
🚷 view threshold
5.03 µs
(+7.50%)Baseline: 4.68 µs
5.57 µs
(90.32%)
Adapter::Magic (Rust)📈 view plot
🚷 view threshold
28.33 µs
(+7.46%)Baseline: 26.37 µs
29.69 µs
(95.42%)
Adapter::Rust📈 view plot
🚷 view threshold
4.68 µs
(+21.52%)Baseline: 3.85 µs
5.95 µs
(78.70%)
Adapter::RustBench📈 view plot
🚷 view threshold
4.68 µs
(+21.60%)Baseline: 3.85 µs
5.93 µs
(78.93%)
🐰 View full continuous benchmarking report in Bencher

@epompeii
epompeii marked this pull request as ready for review September 18, 2026 07:47

This branch has not been deployed

No deployments
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