myriation/atuin
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Fix SQL cache query (#318)

Browse source 
I just deployed the older version and it was falling back on the full
count. Turns out this is because it won't upcast from INT4 to INT8
automatically, and it has to be manual

At some point the underlying total should be changed to int8, but also I
highly doubt anyone will have enough shell history to fill an int4 lol
This commit is contained in:
Ellie Huxtable 2022-04-21 09:19:54 +01:00 committed by GitHub
parent fe05d86bfa
commit 24e2971787
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
atuin-server/src/database.rs
View file

@ -120,7 +120,7 @@ impl Database for Postgres {
} }
async fn count_history_cached(&self, user: &User) -> Result<i64> { async fn count_history_cached(&self, user: &User) -> Result<i64> {
let res: (i64,) = sqlx::query_as( let res: (i32,) = sqlx::query_as(
"select total from total_history_count_user "select total from total_history_count_user
where user_id = $1", where user_id = $1",
) )
@ -128,7 +128,7 @@ impl Database for Postgres {
.fetch_one(&self.pool) .fetch_one(&self.pool)
.await?; .await?;
Ok(res.0) Ok(res.0 as i64)
} }
async fn count_history_range( async fn count_history_range(