Skip to content

[8.2] [MOD-11756] - make sure all FT.SUG* are hashslot aware on Enterprise#7023

Merged
JoanFM merged 1 commit into8.2from
backport-7016-to-8.2
Oct 9, 2025
Merged

[8.2] [MOD-11756] - make sure all FT.SUG* are hashslot aware on Enterprise#7023
JoanFM merged 1 commit into8.2from
backport-7016-to-8.2

Conversation

@redisearch-backport-pull-request
Copy link
Contributor

@redisearch-backport-pull-request redisearch-backport-pull-request bot commented Oct 9, 2025

Description

Backport of #7016 to 8.2.


Note

Adds FT.SUGADD and FT.SUGDEL to overide_command in pack/ramp-enterprise.yml to make them hashslot-aware on Enterprise.

  • Config (ramp-enterprise):
    • Add FT.SUGADD and FT.SUGDEL to overide_command in pack/ramp-enterprise.yml.
    • Existing read-only overrides for FT.AGGREGATE, FT.CURSOR, and FT.SEARCH retained.

Written by Cursor Bugbot for commit 9c9dd2b. This will update automatically on new commits. Configure here.

…7016)

fix: fix ramp enterprise so that all SUG* are hashslot aware
(cherry picked from commit d992ba7)
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: Cursor Command Key Routing Inconsistency

The FT.CURSOR command override has an inconsistent key routing configuration where first_key (3) is greater than last_key (1). This can lead to incorrect hashslot routing in clustered environments.

pack/ramp-enterprise.yml#L59-L60

- {"command_arity": -1, "command_name": "FT.AGGREGATE", "first_key": 0, "flags": ["readonly" ], "last_key": 1, "step": -2}
- {"command_arity": -1, "command_name": "FT.CURSOR", "first_key": 3, "flags": ["readonly"], "last_key": 1, "step": -3}

Fix in Cursor Fix in Web


@redisearch-backport-pull-request redisearch-backport-pull-request bot added this pull request to the merge queue Oct 9, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 9, 2025
@JoanFM JoanFM added this pull request to the merge queue Oct 9, 2025
Merged via the queue into 8.2 with commit 67f4374 Oct 9, 2025
19 checks passed
@JoanFM JoanFM deleted the backport-7016-to-8.2 branch October 9, 2025 20:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant