[performance] cache more database calls, reduce required database calls overall#3290

Merged

NyaaaWhatsUpDoc merged 31 commits intosuperseriousbusiness:mainfrom NyaaaWhatsUpDoc:performance/cache-fiddling

Sep 16, 2024

+1,663-2,115

Commits on Sep 16, 2024

improvements to caching for lists and relationship to accounts / follows
NyaaaWhatsUpDoc
committed
fix nil panic in AddToList()
NyaaaWhatsUpDoc
committed
ensure list related caches are correctly invalidated
NyaaaWhatsUpDoc
committed
ensure returned ID lists are ordered correctly
NyaaaWhatsUpDoc
committed
bump go-structr to v0.8.9 (returns early if zero uncached keys to be loaded)
NyaaaWhatsUpDoc
committed
remove zero checks in uncached key load functions (go-structr now handles this)
NyaaaWhatsUpDoc
committed
fix issues after rebase on upstream/main
NyaaaWhatsUpDoc
committed
update the expected return order of CSV exports (since list entries are now down by entry creation date)
NyaaaWhatsUpDoc
committed
rename some funcs, allow deleting list entries for multiple follow IDs at a time, fix up more tests
NyaaaWhatsUpDoc
committed
use returning statements on delete to get cache invalidation info
NyaaaWhatsUpDoc
committed
fixes to recent database delete changes
NyaaaWhatsUpDoc
committed
fix broken list entries delete sql
NyaaaWhatsUpDoc
committed
remove unused db function
NyaaaWhatsUpDoc
committed
update remainder of delete functions to behave in similar way, some other small tweaks
NyaaaWhatsUpDoc
committed
fix delete user sql, allow returning on err no entries
NyaaaWhatsUpDoc
committed
uncomment + fix list database tests
NyaaaWhatsUpDoc
committed
update remaining list tests
NyaaaWhatsUpDoc
committed
update envparsing test
NyaaaWhatsUpDoc
committed
add comments to each specific key being invalidated
NyaaaWhatsUpDoc
committed
add more cache invalidation explanatory comments
NyaaaWhatsUpDoc
committed
whoops; actually delete poll votes from database in the DeletePollByID() func
NyaaaWhatsUpDoc
committed
remove added but-commented-out field
NyaaaWhatsUpDoc
committed
improved comment regarding paging being disabled
NyaaaWhatsUpDoc
committed
make cache invalidation comments match what's actually happening
NyaaaWhatsUpDoc
committed
fix up delete query comments to match what is happening
NyaaaWhatsUpDoc
committed
rename function to read a bit better
NyaaaWhatsUpDoc
committed
don't use ErrNoEntries on delete when not needed (it's only needed for a RETURNING call)
NyaaaWhatsUpDoc
committed
update function name in test
NyaaaWhatsUpDoc
committed
move list exclusivity check to AFTER eligibility check. use log.Panic() instead of panic()
NyaaaWhatsUpDoc
committed
use the poll_id column in poll_votes for selecting votes in poll ID
NyaaaWhatsUpDoc
committed
fix function name
NyaaaWhatsUpDoc
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[performance] cache more database calls, reduce required database calls overall#3290

Commits on Sep 16, 2024

improvements to caching for lists and relationship to accounts / follows

fix nil panic in AddToList()

ensure list related caches are correctly invalidated

ensure returned ID lists are ordered correctly

bump go-structr to v0.8.9 (returns early if zero uncached keys to be loaded)

remove zero checks in uncached key load functions (go-structr now handles this)

fix issues after rebase on upstream/main

update the expected return order of CSV exports (since list entries are now down by entry creation date)

rename some funcs, allow deleting list entries for multiple follow IDs at a time, fix up more tests

use returning statements on delete to get cache invalidation info

fixes to recent database delete changes

fix broken list entries delete sql

remove unused db function

update remainder of delete functions to behave in similar way, some other small tweaks

fix delete user sql, allow returning on err no entries

uncomment + fix list database tests

update remaining list tests

update envparsing test

add comments to each specific key being invalidated

add more cache invalidation explanatory comments

whoops; actually delete poll votes from database in the DeletePollByID() func

remove added but-commented-out field

improved comment regarding paging being disabled

make cache invalidation comments match what's actually happening

fix up delete query comments to match what is happening

rename function to read a bit better

don't use ErrNoEntries on delete when not needed (it's only needed for a RETURNING call)

update function name in test

move list exclusivity check to AFTER eligibility check. use log.Panic() instead of panic()

use the poll_id column in poll_votes for selecting votes in poll ID

fix function name