Noticed some places we could be using the cache but aren't.
Depends on D13974
Paths
| Differential D13975 Authored by ashoat on Nov 20 2024, 6:39 AM.
Tags None Referenced Files
Subscribers
Details Summary Noticed some places we could be using the cache but aren't. Depends on D13974 Test Plan Tested in combination with later diffs
Diff Detail
Event TimelineHerald added a subscriber: tomek. · View Herald TranscriptNov 20 2024, 6:39 AM2024-11-20 06:39:50 (UTC-8) Harbormaster completed remote builds in B32725: Diff 45899.Nov 20 2024, 6:57 AM2024-11-20 06:57:00 (UTC-8) This revision is now accepted and ready to land.Nov 20 2024, 6:59 AM2024-11-20 06:59:27 (UTC-8) Closed by commit rCOMM7fcd70ef376b: [keyserver][native][web] Use FCCache instead of calling… (authored by ashoat). · Explain WhyNov 22 2024, 6:13 AM2024-11-22 06:13:23 (UTC-8) This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 45899 keyserver/src/creators/farcaster-channel-tag-creator.js
keyserver/src/responders/farcaster-webhook-responders.js
keyserver/src/updaters/thread-updaters.js
keyserver/src/utils/fc-cache.js
|
Same as in the previous diff we should probably handle timestamp collision