Page MenuHomePhabricator

[keyserver] add index to target_message column of messages table
AbandonedPublic

Authored by ginsu on Nov 17 2022, 9:23 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 8, 5:15 PM
Unknown Object (File)
Fri, Nov 8, 5:15 PM
Unknown Object (File)
Fri, Nov 8, 4:51 PM
Unknown Object (File)
Sep 28 2024, 11:30 AM
Unknown Object (File)
Sep 28 2024, 11:29 AM
Unknown Object (File)
Sep 28 2024, 11:23 AM
Unknown Object (File)
Sep 15 2024, 12:20 PM
Unknown Object (File)
Sep 6 2024, 10:48 PM

Details

Reviewers
atul
rohan
Summary

Add index to target_message column of messages table. This is to better optimize how we will be fetching reaction messages for a certain message


Depends on D5658
Linear Task: ENG-2265

Test Plan

Ran the migration, and tested the stack locally in my evironment to make sure everything still runs as expected/nothing crashes.

Diff Detail

Repository
rCOMM Comm
Branch
eng-2265 (branched from master)
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

ginsu requested review of this revision.Nov 17 2022, 9:37 AM
This revision is now accepted and ready to land.Nov 17 2022, 9:42 AM

What diff introduces the target_message column? I don't see a stack here

@ashoat D5658, just updated the summary to make it a stack

Seems like it would be more simple to have a single migration instead of two

Seems like it would be more simple to have a single migration instead of two

Okay that's a good idea, I will combine the two in the first diff

abandoning because D5658 now handles this migration