用 sentinel 替换 hasattr 惰性初始化
值得精读的示例性重构:展示了如何用 sentinel 消除 `hasattr` 的隐式依赖,使缓存语义明确。适合作为代码可维护性改进的参考。
SGLang is a high-performance serving framework for large language models and multimodal models.
用 sentinel 替换 hasattr 惰性初始化
值得精读的示例性重构:展示了如何用 sentinel 消除 `hasattr` 的隐式依赖,使缓存语义明确。适合作为代码可维护性改进的参考。
移除防御性 getattr,直接访问属性
值得精读:这是一个教科书级的机械重构案例,展示了如何安全地将防御性 `getattr` 替换为直接属性访问,提升代码健壮性。可作为团队代码清理的参考。
移除 5 个文件中未使用的实例属性
值得合并,属低风险技术债务清理。建议审核时确认这些字段确实无外部引用(如通过 grep 验证)。此外,此 PR 可作为后续更大范围属性清理(如 scheduler 中类似字段)的参考模式。
移除 managers 模块中 6 个文件的未使用字段
该 PR 是清理死代码的良好实践,值得合并。对于团队新手,可以借此 PR 了解如何安全地识别和删除未使用字段。建议在合并后运行完整的 CI 测试套件以确认无回归。
清理调度器中的死代码赋值
该 PR 是低风险的清理工作,无需深入审查。但可作为理解 Scheduler 中 prefill 统计信息演进历史的参考:从 `self.adder` 等临时存储迁移到 `new_batch.prefill_stats` 对象化。
移除未使用的 self.X 赋值,改为局部变量
此 PR 属于小范围重构,设计意图清晰但存在一处被指出的潜在 GC 风险尚未解决。建议精读 reviewer 评论并评估 tokenizer_manager.py 的更改是否需要保留引用。对于关注代码清理和属性生命周期管理的工程师有学习价值。整体重要性不高,合并前应确保 disagg 测试通过。
修复 SWA 逐出边界 env 变量未生效 bug
值得精读的场景:关注 SWA cache 逐出策略的开发者;希望了解多分支间 env 变量移植实践的读者。推荐与 #24857 和 `swa_radix_cache.py` 中 `_insert_helper` 的 case 3 对照阅读。
修复 Mistral Large 3 测试因 scale shape 不匹配失败
建议合并。该修复针对性强,改动极小且经过测试验证。可考虑在后续 PR 中增强切片安全性,例如对空 tensor 做 fallback 处理。
参与讨论