已合并
memcache接口错误码整改 #457
lichunyang创建于 26 天前
memcache接口错误码整改 #457
已合并
lichunyang创建于 26 天前
lichunyang
lichunyang成员
26 天前

合入来源

问题/功能描述

修改方案描述

是否涉及UT/ST

开发自检

likedislike
Pull Request已成功合入, 合并人@zhangjinshi
(感谢 lichunyang 的贡献)
atomgit-bot
atomgit-bot
26 天前 评论:

变更摘要

本次变更对 memcache 接口的错误码使用进行了规范化整改。主要将批量操作(BatchGetBatchPut 等)中结果容器的默认初始化值从通用错误码 MMC_ERROR 或硬编码 -1 替换为语义更明确的 MMC_INVALID_PARAM,并在 BatchGet 中为空/不存在对象的情况补充返回 MMC_OBJECT_NOT_EXISTS,使错误码能更准确地反映实际失败原因。

主要改动

  • 批量结果容器默认错误码统一为 MMC_INVALID_PARAM:在 src/memcache/csrc/mmc_client.cppmmcc_batch_getmmcc_batch_put 中,将 batchResult 初始化值从 MMC_ERROR 改为 MMC_INVALID_PARAM;在 src/memcache/csrc/mmcache_store.cppBatchPutFromBatchGetIntoPutBatchGetBatch 四个函数中,将 results 初始化值从 -1 改为 MMC_INVALID_PARAM
  • MmcClientDefault::BatchGet 默认结果初始化调整:在 src/memcache/csrc/client/mmc_client_default.cpp 中,将 batchResult 的初始化值从 MMC_ERROR 改为 MMC_INVALID_PARAM
  • BatchGet 新增对象不存在错误码:在 MmcClientDefault::BatchGet 中,当 blob 数量无效(numBlobs <= 0blobs 大小不匹配)以及数据区大小与预期不符时,除日志记录外,显式将对应 key 的 batchResult[i] 设置为 MMC_OBJECT_NOT_EXISTS
likedislike
atomgit-bot
atomgit-bot
26 天前 评论:

代码审查

✅ 未发现问题

likedislike
ascend-robot
ascend-robot成员
26 天前 评论:
流水线 PR-pipeline_memcache#2253 [ commitID:9a1f6416 ] 已完成
阶段 任务名 状态 详情
编译构建 Build_memcache_arm >>>
恶意代码检查 Antipoison_memcache >>>
编码安全与规范检查 pre-commit >>>
CodeCheck_memcache >>>
开源片段检查 SCA_memcache >>>
开发者测试 UT_memcache >>>
流水线 PR-pipeline_memcache >>>
此流水线已支持下列评论快捷指令,仅PR创建者和白名单成员评论有效
  • compile : 运行流水线
  • retry : 重试流水线所有失败子任务
  • retry <任务名> : 仅重试指定失败子任务
  • stop : 停止流水线
likedislike
Xxiangjie10成员
26 天前 添加了label:pr-audit-failed
此处折叠了318条消息 查看更多
ascend-robot
ascend-robot成员
4 天前 评论:

重试流水线失败:PR-pipeline_memcache 系统繁忙,请稍后再试

likedislike
ascend-robotascend-robot成员
4 天前 删除了label:ci-pipeline-running
ascend-robotascend-robot成员
4 天前 添加了label:ci-pipeline-passed
Zzhangjinshi成员
4 天前 关闭了关联的issue
Zzhangjinshi成员
4 天前 合入了pull request