Default versions: assume v2.8 as a minimum (SCAN over KEYS) and 4.0+ in Azure by NickCraver · Pull Request #1929 · StackExchange/StackExchange.Redis
Navigation Menu
{{ message }}
StackExchange / StackExchange.Redis Public
- Notifications You must be signed in to change notification settings
- Fork 1.6k
Merged
Default versions: assume v2.8 as a minimum (SCAN over KEYS) and 4.0+ in Azure#1929
NickCraver merged 3 commits intomainfrom
Default versions: assume v2.8 as a minimum (SCAN over KEYS) and 4.0+ in Azure#1929
NickCraver merged 3 commits intomainfrom
Conversation
Copy link Copy Markdown
Collaborator
NickCraver
commented
Dec 14, 2021
NickCraver
commented
This more reflects reality today and if we cannot detect a server version (e.g. config has INFO disabled as in #1926) then we'll do smarter things like use SCAN over KEYS.
…in Azure This more reflects reality today and if we cannot detect a server version (e.g. config has INFO disabled as in #1926) then we'll do smarter things like use SCAN over KEYS.
NickCraver
requested review from
mgravell and
philon-msft
mgravell approved these changes Dec 14, 2021
philon-msft approved these changes Dec 14, 2021
NickCraver added 2 commits
December 14, 2021 11:27
NickCraver
requested review from
mgravell and
philon-msft
philon-msft approved these changes Dec 14, 2021
Copy link Copy Markdown
Collaborator Author
NickCraver
commented
Dec 14, 2021
NickCraver commented
Dec 14, 2021hahahaha, it's a tiebreaker test failing - unrelated but hilarious, the other should help here :)
NickCraver
merged commit
fd79515
into
main
NickCraver
deleted the
craver/version-bump
branch
ShaneCourtrille
mentioned this pull request
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment