Skip to content

Perf/remote task mainthread receiver#72

Open
gabsow wants to merge 6 commits intomasterfrom
perf/remote-task-mainthread-receiver
Open

Perf/remote task mainthread receiver#72
gabsow wants to merge 6 commits intomasterfrom
perf/remote-task-mainthread-receiver

Conversation

@gabsow
Copy link
Contributor

@gabsow gabsow commented Jan 8, 2026

Note

Optimizes remote task handling and fixes refcount usage, with test harness hardening.

  • Run REMOTE_TASK_FUNCTION_ID receiver on the Redis main thread: cluster.c defers callback via runCallbackOnUnblock and invokes it in MR_ClusterInnerCommunicationMsgUnblock; mr.c routes MR_RemoteTask to MR_RemoteTaskMainThread and logs enablement in MR_Init.
  • Correct RedisModuleString refcount ops to use a real context (mr_staticCtx/ctx) instead of NULL across message/exec paths (HoldString/FreeString).
  • Rust test module: add main-thread awareness and a global mutex around detached context; skip ThreadSafeContextLock on main thread to avoid crashes; record main thread in init.
  • Tests: add pytests/requirements.txt (redis>=5); enhance run_tests.sh with redis-py version check, per-run/test timeouts, CI heartbeat, and diagnostics on timeout.

Written by Cursor Bugbot for commit 722bd1c. This will update automatically on new commits. Configure here.

@gabsow gabsow force-pushed the perf/remote-task-mainthread-receiver branch from 0ee987b to d2377af Compare January 11, 2026 06:34
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.

1 participant