valkey-io/valkey

[BUG] PUBSUB SHARDNUMSUB doesn't return -CROSSSLOT for cross slot shard-channels

Ouverte

#560 ouverte le 27 mai 2024

Voir sur GitHubGuide contributeur
 (5 commentaires) (0 réaction) (1 personne assignée)C (1 138 forks)batch import
breaking-changebughelp wanted

Métriques du dépôt

Stars
 (25 779 étoiles)
Métriques de merge PR
 (Merge moyen 17j 7h) (74 PRs mergées en 30 j)

Description

Describe the bug

In cluster mode, calling this command with two channels mapping to different slots, like PUBSUB SHARDNUMSUB ch1 ch2, succeeds. It should return a -CROSSSLOT error, like any command with keys of two different slots.

To reproduce

Call the command with two channels mapping to the same node, but different cluster slots.

Expected behavior

-CROSSSLOT error reply.

Additional information

I believe it's enough to add a key specification in the command's JSON file under src/commands/. That would make the generic machinery for cluster crosslot detect the error early and return an error.

Guide contributeur

Direction de recherche
Localisez le fichier JSON de spécification de commande pour PUBSUB SHARDNUMSUB et ajoutez les arguments de clé manquants pour activer la détection d'erreur crossslot.
Stack technique
c
Domaine
database
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitC
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80