已合并
【PR】: 重复加载op时打印的日志从error修改为warning #4181
嵇锴创建于 7月29日
【PR】: 重复加载op时打印的日志从error修改为warning #4181
已合并
嵇锴创建于 7月29日
嵇锴
嵇锴成员
7月29日

Pull Request

描述

重复加载op时打印的日志从error修改为warning。

变更类型

请选择本次引入的变更类型:

关联的Issue

如何测试

描述测试此变更的步骤和前提条件:
1.
2.

核对清单

其他信息

在此添加任何其他关于本次 PR 的说明。

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 嵇锴 的贡献)
嵇锴嵇锴成员
7月29日 添加了label:enhancement
嵇锴嵇锴成员
7月29日 创建了 pull request,commit 4a119a1b
atomgit-bot
atomgit-bot
7月29日 评论:

变更摘要

本次变更将 graph_metadef/graph/normal_graph/graph.cc 中重复加载算子时的日志级别从 ERROR 下调为 WARNING。在 GraphImpl 类中,原先使用 GE_CHK_BOOL_RET_STATUS 宏检测算子重复添加并输出错误日志,现改为显式的 if 判断,改用 GELOGW 输出警告日志后返回 GRAPH_FAILED

主要改动

  • 日志级别降级:将重复添加算子时的日志从 GE_CHK_BOOL_RET_STATUS(输出 ERROR 级别日志)改为使用 GELOGW(输出 WARNING 级别日志),避免非致命的重复加载场景触发错误级别告警。
  • 控制流重构:将宏调用 GE_CHK_BOOL_RET_STATUS 替换为显式的 if (!ret.second) 判断和 return GRAPH_FAILED 语句,功能行为保持一致,仅日志级别和代码写法发生变化。
likedislike
atomgit-bot
atomgit-bot
7月29日 评论:

代码审查

✅ 未发现问题

likedislike
CANN-robotCANN-robot成员
7月29日 添加了label:cann-cla/yes
此处折叠了74条消息 查看更多
嵇锴嵇锴成员
7月30日 修改了pull request 的描述
嵇锴嵇锴成员
7月30日 解决了最后一个问题
zhanj
zhanj成员
7月30日 评论:

/approve

likedislike
CANN-robotCANN-robot成员
7月30日 添加了label:approved
CANN-robotCANN-robot成员
7月30日 合入了pull request