Skip to content

[CORRUPTED] Synthetic Benchmark PR #14562 - Clean up lookahead-related code#94

Open
tomerqodo wants to merge 7 commits intobase_pr_14562_20260121_8346from
corrupted_pr_14562_20260121_8346
Open

[CORRUPTED] Synthetic Benchmark PR #14562 - Clean up lookahead-related code#94
tomerqodo wants to merge 7 commits intobase_pr_14562_20260121_8346from
corrupted_pr_14562_20260121_8346

Conversation

@tomerqodo
Copy link

Benchmark PR redis#14562

Type: Corrupted (contains bugs)

Original PR Title: Clean up lookahead-related code
Original PR Description: ## Summary

Clean up lookahead-related(redis#14440) code by consolidating slot extraction logic.

Changes

  • Replace GETSLOT_NOKEYS with INVALID_CLUSTER_SLOT
  • Refactor getSlotFromCommand() to reuse extractSlotFromKeysResult()
  • Let extractSlotFromKeysResult () behavior more unified and more readable
  • Fix comment alignment
    Original PR URL: Clean up lookahead-related code redis/redis#14562

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants