open-duelyst/duelyst

[P3] Replace node-redis with ioredis

Open

Aperta il 2 ott 2022

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)JavaScript (3443 star) (526 fork)batch import
backendenhancementhelp wanted

Descrizione

We're currently using node-redis@2.8.0, which has poor support for connection timeouts.

Replacing this with ioredis@5.2.3 fixes the above, but requires some changes. For example, the zrevrankAsync function does not exist, so we need to use zrevrank instead and remove the .then(result) chain.

Alternatively, upgrading node-redis may help with these issues.

See https://github.com/luin/ioredis/wiki/Migrating-from-node_redis

Guida contributor

Tech stack
javascriptnodejsredis
Dominio
backenddatabase
Tipo issue
refactor
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 hours
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
stale
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
clear
Prerequisiti
Node.jsRedis basics
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60
Direzione di ricerca
Review the migration guide at https://github.com/luin/ioredis/wiki/Migrating from node redis. Replace node redis with ioredis in the codebase, particularly update methods like `zrevrankAsync` to `zrevrank` and remove `.then()` chains. Check all Redis calls to ensure compatibility.