Skip to content

Comments

Replace calls to the Redis command KEYS with calls to SCAN#9

Open
dcoleman17 wants to merge 2 commits intobbangert:masterfrom
DataDog:replace_keys_with_scan
Open

Replace calls to the Redis command KEYS with calls to SCAN#9
dcoleman17 wants to merge 2 commits intobbangert:masterfrom
DataDog:replace_keys_with_scan

Conversation

@dcoleman17
Copy link

Notes:

  • SCAN was added in Redis 2.8; if this change is merged, openid-redis will only work with Redis 2.8 or later.
  • The results from the calls to SCAN are stored in a set() instead of a list because SCAN might return the same key in multiple calls

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.

1 participant