diff options
| author |  tobi <31960611+tsmethurst@users.noreply.github.com> | 2022-05-02 12:53:46 +0200 |
|---|---|---|
| committer |  GitHub <noreply@github.com> | 2022-05-02 11:53:46 +0100 |
| commit | a5852fd7e43bf97ea7def9de20cd6d02d954094d (patch) | |
| tree | d72b7fe9bf603786a5533e6b566a8412b2b8f634 /internal/db/bundb/account.go | |
| parent | Add logging to the new generic worker package (#516) (diff) | |
| download | gotosocial-a5852fd7e43bf97ea7def9de20cd6d02d954094d.tar.xz | |
[performance] Speed up some of the slower db queries (#523)
* remove unnecessary LOWER() db calls
* warn during slow db queries
* use bundb built-in exists function
* add db block test
* update account block query
* add domain block db test
* optimize domain block query
* fix implementing wrong test
* exclude most columns when checking block
* go fmt
* remote more unnecessary use of LOWER()
Diffstat (limited to 'internal/db/bundb/account.go')
| -rw-r--r-- | internal/db/bundb/account.go | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/internal/db/bundb/account.go b/internal/db/bundb/account.go index 876fb5186..59292055e 100644 --- a/internal/db/bundb/account.go +++ b/internal/db/bundb/account.go @@ -22,6 +22,7 @@ import ( "context" "errors" "fmt" + "strings" "time" "github.com/spf13/viper" @@ -199,7 +200,7 @@ func (a *accountDB) GetLocalAccountByUsername(ctx context.Context, username stri account := new(gtsmodel.Account) q := a.newAccountQ(account). - Where("LOWER(?) = LOWER(?)", bun.Ident("username"), username). // ignore casing + Where("username = ?", strings.ToLower(username)). // usernames on our instance will always be lowercase WhereGroup(" AND ", whereEmptyOrNull("domain")) if err := q.Scan(ctx); err != nil { |