Skip to content

Nested MULTI/EXEC may replicate in different cases.#4467

Merged
antirez merged 1 commit intoredis:unstablefrom
yossigo:fix-nested-multi
Nov 23, 2017
Merged

Nested MULTI/EXEC may replicate in different cases.#4467
antirez merged 1 commit intoredis:unstablefrom
yossigo:fix-nested-multi

Conversation

@yossigo
Copy link
Collaborator

@yossigo yossigo commented Nov 22, 2017

For example:

  1. A module command called within a MULTI section.
  2. A Lua script with replicate_commands() called within a MULTI section.
  3. A module command called from a Lua script in the above context.

For example:
1. A module command called within a MULTI section.
2. A Lua script with replicate_commands() called within a MULTI section.
3. A module command called from a Lua script in the above context.
@dvirsky
Copy link
Contributor

dvirsky commented Nov 23, 2017

FYI This doesn't fix the issue described in #4466 which is similar

@antirez
Copy link
Contributor

antirez commented Nov 23, 2017

Hello @yossigo, thank you, I did a review of your work can it looks good to me as well. Merging.

@antirez antirez merged commit ada2065 into redis:unstable Nov 23, 2017
antirez added a commit that referenced this pull request Nov 23, 2017
antirez added a commit that referenced this pull request Nov 23, 2017
JackieXie168 pushed a commit to JackieXie168/redis that referenced this pull request Jan 13, 2018
JackieXie168 pushed a commit to JackieXie168/redis that referenced this pull request Jan 13, 2018
JackieXie168 pushed a commit to JackieXie168/redis that referenced this pull request Jan 13, 2018
Nested MULTI/EXEC may replicate in different cases.
JackieXie168 pushed a commit to JackieXie168/redis that referenced this pull request Jan 13, 2018
pulllock pushed a commit to pulllock/redis that referenced this pull request Jun 28, 2023
Nested MULTI/EXEC may replicate in different cases.
pulllock pushed a commit to pulllock/redis that referenced this pull request Jun 28, 2023
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.

3 participants