Skip to content

fix(client): FunctionKillAsync routes to AllPrimaries instead of AllNodes in cluster mode #494

Description

@currantw

Describe the bug

GlideClusterClient.FunctionKillAsync() explicitly routes to Route.AllPrimaries, but glide-core's default routing for FUNCTION KILL is AllNodes. A runaway function could be executing on a replica, so the kill command should reach all nodes.

Expected Behavior

FUNCTION KILL should be sent to all nodes (AllNodes) so it can terminate a running function regardless of which node is executing it.

Current Behavior

Only primary nodes receive the FUNCTION KILL command, leaving any runaway function on a replica unaffected.

Possible Solution

Remove the explicit Route.AllPrimaries parameter from FunctionKillAsync() so glide-core's correct AllNodes default applies.

Related

Metadata

Metadata

Assignees

Labels

coreCore library (`sources/Valkey.Glide/`)triageNeeds triage — not yet reviewed

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions