Skip to content

Add documentation documenting read-only scripts#1953

Merged
oranagra merged 3 commits intoredis:masterfrom
madolson:ro-scripts
May 29, 2022
Merged

Add documentation documenting read-only scripts#1953
oranagra merged 3 commits intoredis:masterfrom
madolson:ro-scripts

Conversation

@madolson
Copy link
Contributor

Based on discussion from redis/redis#10728 (comment), want to better define when to use EVAL et. al vs EVAL_RO et al.

@enjoy-binbin enjoy-binbin added the waiting-for-upstream waiting for a redis PR to be merged label May 22, 2022
@oranagra oranagra added the to-be-merged should probably be merged soon label May 22, 2022
@oranagra
Copy link
Member

i thought that we want to mention something in the command pages themselves.
i.e.

  1. they were created for the purpose of client routing (that's what makes them different than other types of RO scripts).
  2. they might get deprecated some day.

maybe all of that isn't needed, just trying to stress what their purpose, and what's they're not meant to be.

@oranagra oranagra mentioned this pull request May 29, 2022
@netlify
Copy link

netlify bot commented May 29, 2022

👷 Deploy request for redis-doc pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 05ff4d6

@oranagra oranagra merged commit 5a5bb51 into redis:master May 29, 2022
oranagra added a commit to oranagra/redis-doc that referenced this pull request May 29, 2022
slorello89 added a commit to slorello89/redis-doc that referenced this pull request Jun 3, 2022
README updates, moving from dictionary.txt -> wordlist

adding spellchecker-cli to wordlist. . . so meta

Update README about clients (redis#1886)

Add documentation documenting read-only scripts (redis#1953)

Based on discussion from redis/redis#10728 (comment),
want to better define when to use `EVAL` et. al vs `EVAL_RO` et al.
to be modified again soon in redis#1954

Add info fields about network bytes during replication. (redis#1966)

Co-authored-by: jiangyujie.jyj <[email protected]>

Update key-specs.md (redis#1963)

Fix "incomplete flag section" internal page link

Update install-redis-on-windows.md (redis#1967)

Duplicate "install" word.

Update command-getkeysandflags.md (redis#1946)

Update install-redis-on-mac-os.md (redis#1938)

Small grammar fixes

fix incorrect link (redis#1926)

Add CLUSTER INFO fields (redis#1798)

script flags update (redis#1954)

updating docs about read only scripts and oom related flag changes in 7.0.1

minor fixes

adding spellcheck config and commands file

adding workflow dispatch

newline

newline

debug

newline

duh

whitespace

more whitespace

this ought to do it.

this ought to do it.

alright, this really ought to do it.

no clobber

just ignore

COMMAND_FILES*

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

Labels

to-be-merged should probably be merged soon waiting-for-upstream waiting for a redis PR to be merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants