cockroachdb / cockroachdb/errors

Add ability to wrap an error with an Unimplemented error

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

描述

I have a bit of code that does something like this:
```
errors.UnimplementedErrorf(
link,
"unsupported expression %q: %v",
exprString,
err,
)
```

This flattens the `err` argument. What I'd really like is something like

```
errors.UnimplementedWrapf(
err,
link,
"unsupported expression %q",
exprString,
)
```

(analogous to `errors.Errorf` versus `errors.Wrapf`)

The exact piece of code I'm referring to is: https://github.com/cockroachdb/cockroach/blob/7bf398fc9c715e67ae4ab342569b26f50809fb59/pkg/ccl/importccl/read_import_mysql.go#L764

cc @knz

贡献指南

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

调研方向

从 pkg/ccl/importccl/read_import_mysql.go 中大约第 764 行的被引用调用开始,然后比较现有的 UnimplementedErrorf 和 Wrapf API。添加所请求的 UnimplementedWrapf 行为,使原始错误保持被包装,同时消息包含所提供的表达式文本;使用针对错误格式化的测试进行验证。

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

评估

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

把新 issue 发到你的邮箱

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