jessun

Results 5 issues of jessun

## 存在问题 在当前的 main 分支(830352d)中,使用的 golangci-lint 版本为 v1.43.0。目前该版本下的检查未能发现部分类型断言panic的风险,比如 #1576。将该工具升级至 v1.53.2 后可以发现此类风险。 ## 版本选择 根据 https://github.com/golangci/golangci-lint 的 tag 规律发现,该工具的末尾小版本号最大一般也就是到2,因此可以考虑使用最新版本 v1.53.2。 ## 代码变更 - 业务代码调整。在更新了 golangci-lint 工具后,发现有多处代码需要优化,没有业务变更。关联 pr 为 #1578。 - 工具配置调整。在默认情况下发现较多问题时,在单次运行后...

suggestion
need_update_doc

存在的问题: 目前 model.AuditPlanSQLV2.Info 类型为 JSON,即 json.RawMessage 。那么不同的扫描任务类型会在该字段上会有各自的字段定义。每个审核任务都需要新增独立的函数来处理 Info 字段的内容填充和转换。每个独立函数都需要自行进行`json.Unmarshal()`或`json.Marshal()`操作。 可选优化方案: 方案一:将 JSON 定义变更为固定结构体 struct,类似 model 中的 AuditResults 字段 方案二:直接新增一个字段来代替 Info 字段,原字段做废弃使用。 影响: 所有审核任务的采集 SQL 格式转换部分。 对于该字段存储来说,没有变化。

suggestion
code_refactor

**SQLe Version** c06efc94583b0aa7bdd9e235d6c093040f88f78d **Describe the bug** **To Reproduce** https://github.com/actiontech/sqle/discussions/1486 在 sqle v2.2303.0 升级至 v2.2304.0 过程中,如果没有使用升级工具。会造成表 audit_plan_report_sqls_v2 列 audit_results 出现 null 情况。此时如果从页面访问审核结果,会触发 panic。 **Suggestion** 建议验证方式:直接对表 audit_plan_report_sqls_v2 列 audit_results 设置为 null。然后使用页面功能获取审核结果。观察 sqle...

bug

**SQLe Version** 476ce9745090a7a54ce92558e02232f77eb13d0f **Describe the bug** ![image](https://user-images.githubusercontent.com/13587641/236609347-03082f15-3864-4dc5-8dce-44ca724e67bc.png) 在取消`cancel`的地方,使用的方式是传值。但是在该`cancel`消费的地方可能有两处。因此其中有一处可能会产生阻塞,没有产生取消的效果。 **To Reproduce** TODO **Expected behavior** TODO **Suggestion** `cancel`定义的目的是作为在loop中作为`信号`来使用,并非作为`广播`来使用。因此在具体的扫描任务中代码实现中,不应该读取`cancel`

bug

请问作者:后期编辑器有加入VIM模式的计划吗?个人偏爱VIM模式。