Skip to content

[feat] swap: mark unsupported commands (XADD non-deterministic varian… - #457

Merged
qiongtubao merged 1 commit into
feature/ror-update-8.2.0from
feature/swap-unsupport-cmd
Oct 16, 2026
Merged

qiongtubao merged 1 commit into
feature/ror-update-8.2.0from
feature/swap-unsupport-cmd

Conversation

@qiongtubao

Copy link
Copy Markdown

…ts) as swap-error

  • xadd.json.merge: deprecate non-deterministic XADD variants for swap
  • config: register unsupported-command swap error code
  • ctrip_swap_*: detect and reject non-deterministic XADD/KILL/VVERBATIM/etc. via swap_unsupport_cmd path
  • script.c: propagate swap error into script reply
  • server.c: route unsupported command to swap-unsupport reply
  • module.c: filter unsupported commands from module command table
  • tests/swap/unit/swap_unsupport_cmd.tcl: cover new behavior

@qiongtubao
qiongtubao force-pushed the feature/swap-unsupport-cmd branch 4 times, most recently from 248a328 to 33b5091 Compare October 14, 2026 10:49
…ts) as swap-error

- xadd.json.merge: deprecate non-deterministic XADD variants for swap
- config: register unsupported-command swap error code
- ctrip_swap_*: detect and reject non-deterministic XADD/KILL/VVERBATIM/etc. via swap_unsupport_cmd path
- script.c: propagate swap error into script reply
- server.c: route unsupported command to swap-unsupport reply
- module.c: filter unsupported commands from module command table
- tests/swap/unit/swap_unsupport_cmd.tcl: cover new behavior
@qiongtubao
qiongtubao force-pushed the feature/swap-unsupport-cmd branch from 33b5091 to 7ff8dc1 Compare October 15, 2026 03:17
@evazca

evazca commented Oct 16, 2026

Copy link
Copy Markdown

lgtm

@qiongtubao
qiongtubao merged commit c9b6a59 into feature/ror-update-8.2.0 Oct 16, 2026
5 checks passed
@qiongtubao
qiongtubao deleted the feature/swap-unsupport-cmd branch October 16, 2026 04:22
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.

2 participants