perf(redis): avoid serializing addresses during unregister

[ReubenBond]

Summary

Redis grain-directory unregister operations serialized the full GrainAddress even though the delete script only consumes its activation ID. This removes that unused serialization.

Rationale

Unregister remains activation-aware and preserves conflict handling while avoiding a JSON string allocation and serialization work on every successful call.

Microsoft Reviewers: Open in CodeFlow

[Copilot]

Pull request overview

This PR optimizes the Redis grain-directory unregister path by removing an unnecessary JSON serialization of the full GrainAddress when the Redis delete script only needs the activation id, reducing per-call allocation and serialization overhead in the common success case.

Changes:

• Remove eager JsonSerializer.Serialize(address) allocation during Unregister.
• Continue passing only ActivationId into the Redis delete script while preserving conflict handling behavior.

Show a summary per file

File	Description
src/Redis/Orleans.GrainDirectory.Redis/RedisGrainDirectory.cs	Removes unused address serialization in Unregister and keeps the script invocation based on activation id.

Review details

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

• Files reviewed: 1/1 changed files
• Comments generated: 0
• Review effort level: Lite

[github-actions]

Code coverage

77.61% line coverage - 97,578 / 125,721 lines

Coverage details

• Source files: 1,660
• Scope: every CI test matrix job, including providers, CodeGen, .NET 8/10, Linux, Windows, and macOS
• Sources: maintained repository code under src/ loaded by the complete CI test scope
• Commit: 490050e856
• View the test run and merged coverage artifact
• View the coverage reporting run