Minor revision.

- Updated minor comments.

Signed-off-by: Kyle Kim <[email protected]>

src/cluster_slot_stats.c

@@ -45,8 +45,8 @@ static uint64_t getSlotStat(int slot, slotStatType stat_type) {
     case CPU_USEC: slot_stat = server.cluster->slot_stats[slot].cpu_usec; break;
     case NETWORK_BYTES_IN: slot_stat = server.cluster->slot_stats[slot].network_bytes_in; break;
     case NETWORK_BYTES_OUT: slot_stat = server.cluster->slot_stats[slot].network_bytes_out; break;
-    default: /* SLOT_STAT_COUNT, INVALID */
-        serverPanic("Invalid slot stat type %d was found.", stat_type);
+    case SLOT_STAT_COUNT:
+    case INVALID: serverPanic("Invalid slot stat type %d was found.", stat_type);
     }
     return slot_stat;
 }

src/db.c

@@ -231,9 +231,13 @@ int calculateKeySlot(sds key) {
 int getKeySlot(sds key) {
     /* This is performance optimization that uses pre-set slot id from the current command,
      * in order to avoid calculation of the key hash.
+     *
      * This optimization is only used when current_client flag `CLIENT_EXECUTING_COMMAND` is set.
      * It only gets set during the execution of command under `call` method. Other flows requesting
      * the key slot would fallback to calculateKeySlot.
+     *
+     * Modules and scripts executed on the primary may get replicated as multi-execs that operate on multiple slots,
+     * so we must always recompute the slot for commands coming from the primary.
      */
     if (server.current_client && server.current_client->slot >= 0 && server.current_client->flag.executing_command &&
         !server.current_client->flag.primary) {

src/multi.c

@@ -28,7 +28,6 @@
  */
 
 #include "server.h"
-#include "cluster_slot_stats.h"
 
 /* ================================ MULTI/EXEC ============================== */