diff options
| author |  kim <89579420+NyaaaWhatsUpDoc@users.noreply.github.com> | 2023-03-03 23:02:23 +0000 |
|---|---|---|
| committer |  GitHub <noreply@github.com> | 2023-03-03 23:02:23 +0000 |
| commit | a8e6bdfa33f3232ebc8f241b9c90e4da9191a627 (patch) | |
| tree | 087eca372fb13093f39837683682ca9e11b96188 /internal/processing/media | |
| parent | [bugfix] Federate status delete using just the URI (#1584) (diff) | |
| download | gotosocial-a8e6bdfa33f3232ebc8f241b9c90e4da9191a627.tar.xz | |
[performance] cache media attachments (#1525)
* replace concurrency worker pools with base models in State.Workers, update code and tests accordingly
* add media attachment caching, slightly tweak default cache config
* further tweak default cache config values
* replace other media attachment db calls to go through cache
* update envparsing test
* fix delete media attachment sql
* fix media sql query
* invalidate cached media entries during status create / update
* fix envparsing test
* fix typo in panic log message...
* add 'updated_at' column during UpdateAttachment
* remove unused func
---------
Signed-off-by: kim <grufwub@gmail.com>
Diffstat (limited to 'internal/processing/media')
| -rw-r--r-- | internal/processing/media/delete.go | 2 | ||||
| -rw-r--r-- | internal/processing/media/unattach.go | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/internal/processing/media/delete.go b/internal/processing/media/delete.go index 02bd6cd0d..c5a398094 100644 --- a/internal/processing/media/delete.go +++ b/internal/processing/media/delete.go @@ -40,7 +40,7 @@ func (p *Processor) Delete(ctx context.Context, mediaAttachmentID string) gtserr } // delete the attachment - if err := p.state.DB.DeleteByID(ctx, mediaAttachmentID, attachment); err != nil && !errors.Is(err, db.ErrNoEntries) { + if err := p.state.DB.DeleteAttachment(ctx, mediaAttachmentID); err != nil && !errors.Is(err, db.ErrNoEntries) { errs = append(errs, fmt.Sprintf("remove attachment: %s", err)) } diff --git a/internal/processing/media/unattach.go b/internal/processing/media/unattach.go index 7c6f7dbac..8db602239 100644 --- a/internal/processing/media/unattach.go +++ b/internal/processing/media/unattach.go @@ -49,7 +49,7 @@ func (p *Processor) Unattach(ctx context.Context, account *gtsmodel.Account, med attachment.UpdatedAt = time.Now() attachment.StatusID = "" - if err := p.state.DB.UpdateByID(ctx, attachment, attachment.ID, updatingColumns...); err != nil { + if err := p.state.DB.UpdateAttachment(ctx, attachment, updatingColumns...); err != nil { return nil, gtserror.NewErrorNotFound(fmt.Errorf("db error updating attachment: %s", err)) } |