Differential D13636

[keyserver] redis cache
ClosedPublic
Actions

Authored by varun on Tue, Oct 8, 1:23 PM.

Tags

None

Referenced Files

	F2929570: D13636.diff
	Wed, Oct 9, 3:17 PM

	F2928325: D13636.diff
	Wed, Oct 9, 2:59 AM

	F2927087: D13636.id44975.diff
	Tue, Oct 8, 8:23 PM

	F2926990: D13636.id.diff
	Tue, Oct 8, 7:25 PM

	F2926963: D13636.diff
	Tue, Oct 8, 6:56 PM

Subscribers

Details

Reviewers

Commits

rCOMM757a38090721: [keyserver] redis cache

Summary

this cache is used in the next diff to try to retrieve info about a particular fc channel without making a neynar API call.

Depends on D13635

Test Plan

tested in next diff

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

varun created this revision.Tue, Oct 8, 1:23 PM

Herald added a subscriber: tomek. · View Herald TranscriptTue, Oct 8, 1:23 PM

varun added a child revision: D13637: [keyserver][lib] replace neynar client cache with redis cache.Tue, Oct 8, 1:27 PM

Harbormaster completed remote builds in B32110: Diff 44975.Tue, Oct 8, 1:44 PM

varun requested review of this revision.Tue, Oct 8, 1:44 PM

ashoat accepted this revision.Tue, Oct 8, 6:44 PM

This revision is now accepted and ready to land.Tue, Oct 8, 6:44 PM

Closed by commit rCOMM757a38090721: [keyserver] redis cache (authored by varun). · Explain WhyTue, Oct 8, 9:17 PM

This revision was automatically updated to reflect the committed changes.

varun added a commit: rCOMM757a38090721: [keyserver] redis cache.

in case someone in the future wonders why i created a new Redis client here... it's because once a client is in pub/sub mode, it can't be used for regular Redis operations like managing a cache

Revision Contents
Changeset List

Path

Size

keyserver/

src/

utils/

51 lines

Diff 44985

keyserver/src/utils/redis-cache.js

Loading...