diff options
| author | Ellie Huxtable <ellie@atuin.sh> | 2026-01-23 12:27:43 -0800 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2026-01-23 12:27:43 -0800 |
| commit | 126070068abda34fda0b880a36cc604b4ac8be2a (patch) | |
| tree | 1b5529df0adca1a228fa107bfe2ee9bf240e7f61 /crates/atuin-server-database/src | |
| parent | chore(deps)!: update tls deps, remove built-in tls server support (#3091) (diff) | |
| download | atuin-126070068abda34fda0b880a36cc604b4ac8be2a.zip | |
chore!: remove total_history from api index response (#3094)
Remove the expensive and inaccurate `total_history` field from the API
index endpoint. The query `select sum(total) from
total_history_count_user` ran on every request but is no longer
relevant. The underlying table remains for per-user cached counts used
by `/sync/count`.
## Checks
- [x] I am happy for maintainers to push small adjustments to this PR,
to speed up the review cycle
- [x] I have checked that there are no existing pull requests for the
same thing
Diffstat (limited to 'crates/atuin-server-database/src')
| -rw-r--r-- | crates/atuin-server-database/src/lib.rs | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/crates/atuin-server-database/src/lib.rs b/crates/atuin-server-database/src/lib.rs index db170b50..a4ddf23c 100644 --- a/crates/atuin-server-database/src/lib.rs +++ b/crates/atuin-server-database/src/lib.rs @@ -113,7 +113,6 @@ pub trait Database: Sized + Clone + Send + Sync + 'static { async fn update_user_password(&self, u: &User) -> DbResult<()>; - async fn total_history(&self) -> DbResult<i64>; async fn count_history(&self, user: &User) -> DbResult<i64>; async fn count_history_cached(&self, user: &User) -> DbResult<i64>; |