Skip to content

Sort out mess document for RM_Replicate and RM_ReplicateVerbatim#13323

Merged
sundb merged 1 commit intoredis:unstablefrom
sundb:doc_for_RM_Replicate
Jun 21, 2024
Merged

Sort out mess document for RM_Replicate and RM_ReplicateVerbatim#13323
sundb merged 1 commit intoredis:unstablefrom
sundb:doc_for_RM_Replicate

Conversation

@sundb
Copy link
Collaborator

@sundb sundb commented Jun 5, 2024

Related to #12647

  1. Make clear that RM_Replicate and RM_ReplicateVerbatim are non-thread safe.
  2. Make clear that RM_Replicate and RM_ReplicateVerbatim are alwarys wrapped into MULTI in any case.

@sundb sundb changed the title Update RM_Replicate* doc to specify GIL requirement and always being wrapped in MULTI/EXEC Sort out mess document for 'RM_Replicate' and 'RM_ReplicateVerbatim' Jun 5, 2024
@sundb sundb changed the title Sort out mess document for 'RM_Replicate' and 'RM_ReplicateVerbatim' Sort out mess document for RM_Replicate and RM_ReplicateVerbatim Jun 5, 2024
@oranagra oranagra requested a review from guybe7 June 17, 2024 06:58
@sundb sundb merged commit a03b6e2 into redis:unstable Jun 21, 2024
@sundb sundb deleted the doc_for_RM_Replicate branch June 21, 2024 10:00
funny-dog pushed a commit to funny-dog/redis that referenced this pull request Sep 17, 2025
…is#13323)

Related to redis#12647

1. Make clear that `RM_Replicate` and `RM_ReplicateVerbatim` are
non-thread safe.
2. Make clear that `RM_Replicate` and `RM_ReplicateVerbatim` are alwarys
wrapped into MULTI in any case.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[CRASH] worker thread call RedisModule_ReplicateVerbatim(ctx); trigger redis crash

2 participants