open-duelyst/duelyst

[P3] Replace node-redis with ioredis

Open

#80 aperta il 2 ott 2022

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

Metriche repository

Star
 (3443 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

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

Direzione di ricerca
Rivedi la guida alla migrazione su https://github.com/luin/ioredis/wiki/Migrating from node redis. Sostituisci node redis con ioredis nella codebase, in particolare aggiorna metodi come `zrevrankAsync` in `zrevrank` e rimuovi le catene `.then()`. Controlla tutte le chiamate Redis per garantire la compatibilità.
Tech stack
javascriptnodejsredis
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptNode.jsRedis
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60