执行摘要
该PR通过为Mooncake分布式存储设置默认的MC_MAX_MR_SIZE环境变量（4GiB），修复了初始化过程中可能因配置缺失导致的注册挂起问题。这是一个针对KVCache外部存储组件的关键稳定性修复，影响范围有限但重要，采用先检查后设置的保守策略确保兼容性。
功能与动机
问题背景：Mooncake存储初始化时，如果未设置MC_MAX_MR_SIZE环境变量，可能导致注册过程挂起（register hang），影响系统启动稳定性。
解决目标：在PR body中明确表述为"Set MC_MAX_MR_SIZE to avoid register hang"，旨在通过提供合理的默认值，确保存储组件可靠初始化。
实现拆解
修改文件：fastdeploy/cache_manager/transfer_factory/mooncake_store/mooncake_store.py
核心改动：在MooncakeStore.__init__方法中，在现有环境变量设置逻辑后新增以下代码：
if os.environ.get("MC_MAX_MR_SIZE") is None:
os.environ["MC_MAX_MR_SIZE"] = "4294967296" # 4GB
logger.info("MC_MAX_MR_SIZE is not set, default to 4GB.")
设计要点：
- 条件设置：仅当环境变量未设置时才应用默认值，保留用户自定义配置的能力
- 一致性：沿用同文件中
MC_TCP_BIND_ADDRESS的设置模式
- 日志记录：明确提示默认值生效，便于问题排查
评论区精华
Copilot在review中提出了有价值的代码质量建议：
"这里设置的默认值是 4294967296（= 4 * 1024^3），实际是 4GiB 而不是 4GB（十进制）。建议把注释/日志里的'4GB'改为'4GiB'，并尽量避免裸魔法数字（例如提取为常量或用 4 * 1024**3 计算）以减少误解；同时可以在注释/文档里补充为何需要该默认值以避免 register hang。"
这些建议聚焦于代码清晰度和可维护性，但未在后续讨论中得到明确回应或采纳。
风险与影响
技术风险：
- 配置误解：注释中的"4GB"与实际值4GiB（4294967296字节）存在单位混淆，可能影响用户对配置值的理解
- 覆盖风险：根据codecov报告，新增的3行代码缺少测试覆盖
影响评估：
- 用户影响：提升Mooncake存储的启动可靠性，减少因环境配置缺失导致的故障
- 系统影响：确保KVCache外部存储组件稳定工作，间接保障缓存管理功能
- 兼容性：采用条件设置策略，完全向后兼容现有配置
关联脉络
历史PR关联：
- PR #7083：同样涉及cache_manager模块，但关注缓存传输管理而非存储初始化
- PR #7125：同样涉及KVCache配置管理，但关注缓存驱逐时间而非环境变量默认值
演进趋势：近期多个PR（如#7083、#7125、#7127）显示KVCache模块正在持续完善，包括缓存管理、调度优化和测试覆盖。本PR作为其中的配置稳定性修复，体现了对基础设施可靠性的关注。
参与讨论