open-duelyst/duelyst

[P3] Replace node-redis with ioredis

Open

#80 ouverte le 2 oct. 2022

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)JavaScript (3 443 stars) (526 forks)batch import
backendenhancementhelp wanted

Description

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

Guide contributeur

Stack technique
javascriptnodejsredis
Domaine
backenddatabase
Type d'issue
refactor
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Node.jsRedis basics
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60
Direction de recherche
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.