rDrama
/
rDrama
9
10
Fork 6
Code Issues Pull Requests Packages Releases Wiki Activity

WIP: Comment Query Optimization (Eager Loading) #1

Closed
Snakes wants to merge 2 commits from <deleted>:comment-query-optimization into master
pull from: <deleted>:comment-query-optimization
merge into: rDrama:master
Conversation 1 Commits 2 Files Changed 3
Snakes commented 2022-11-23 19:16:59 +00:00

Using eager loading for comment listings, esp. for post viewing.

Has been tested on production data and works; however, it greatly increased database CPU usage, which is almost the opposite of what should happen. Needs more investigation.

Note to Self: Tell the Mottizens when this is done too.

Using eager loading for comment listings, esp. for post viewing. Has been tested on production data and works; however, it greatly increased database CPU usage, which is almost the opposite of what should happen. Needs more investigation. Note to Self: Tell the Mottizens when this is done too.
justcool393 commented 2022-11-23 19:18:08 +00:00

this is a good idea <3

this is a good idea <3
❤️ 1
justcool393 added the
improvement
 label 2022-11-29 23:22:19 +00:00
Snakes force-pushed comment-query-optimization from 040ec838fe to 86274a01b7 2022-11-30 01:42:59 +00:00 Compare
Aevann closed this pull request 2023-01-22 23:32:40 +00:00
Aevann referenced this issue from a commit 2023-06-25 13:54:45 +00:00
RFV test #1
Aevann referenced this issue from a commit 2023-06-25 14:25:10 +00:00
RFV test #1

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug

Something is not working   
duplicate

This issue or pull request already exists   
feature

New feature   
good first issue

New and want to help?   
help wanted

Need some help   
improvement

Code-focused: maintenance, refactoring   
invalid

Something is wrong   
question

More information is needed   
wontfix

This won't be fixed
No Label
bug
duplicate
feature
good first issue
help wanted
improvement
invalid
question
wontfix
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: rDrama/rDrama#1
There is no content yet.
Delete Branch "<deleted>:comment-query-optimization"

Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?