cockroachdb / cockroachdb/errors

Require a context in assertion errors

未关闭
#18 1 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
主要语言
Go
星标
2.5k
派生
74
PR 合并指标
30 天内没有已合并 PR

描述

Suggested by @andreimatei in https://github.com/cockroachdb/cockroach/pull/43032#pullrequestreview-334181315:

`AssertionFailedf` is usually called in functions with a `context.Context` available.

It would be beneficial to capture the context tags to provide more details during troubleshooting.

贡献指南

这个仓库没有索引到贡献指南

调研方向

首先定位 AssertionFailedf 以及使用 context.Context 调用它的函数。跟踪断言错误的构造方式,以及上下文标签如何在那里可用。完成的标准是:断言错误始终需要或接收上下文,并包含其标签以便排查问题,同时为更新后的行为提供覆盖测试。

由索引模型根据 Issue 内容生成。

评估

技术栈
go
领域
backend
Issue 类型
功能
难度
4/5
预计耗时
3-5 天
活跃度
活跃
描述清晰度
基本清楚
新手友好度
48/100

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。