存在检测错误或警告时，拒绝提交

[peixubin]

存在检测错误或警告时，拒绝提交,避免一些错误的SQL提交执行

[peixubin]

"Codecov token not found. Please provide Codecov token with -t flag." 是什么意思？是不是构建脚本有问题

[codecov[bot]]

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 77.15%. Comparing base (52ce759) to head (059f172). Report is 16 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #2692      +/-   ##
==========================================
+ Coverage   77.14%   77.15%   +0.01%     
==========================================
  Files         117      117              
  Lines       16174    16186      +12     
==========================================
+ Hits        12477    12489      +12     
  Misses       3697     3697              

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

[woshiyanghai]

我是没有开启自动审核，sql检查如果有错误，或者警告，其实我认为应该可以提交，只是在审核区域不在展示，工单日志展示sql审核不通过。个人感觉稍微好一点？ 比如有一个场景，研发写了很大一段sql准备让提工单。但是一直审核有错误，如果不让他提交，他找dba看的时候，还需要把sql发送dba，然后好像比较麻烦？

[woshiyanghai]

对，自动驳回并不是自动审核流程这两个是独立的。 所以在提交工单时，如果发现时自动驳回的单子，就直接设置为2，后续没有自动审核逻辑了。这样这个单子系统也能看到，不同级别有权限的也能看到，也又对应的日志。

[LeoQuote]

这一版还是和自动通过结合了, 在这一步之前再做一个处理, 如果 workflow status 是 autoreview_wrong ,那么就提前保存, return , 你看能不能解决你的问题.

https://github.com/hhyo/Archery/blob/52ce7595d4b3b6e294b49c4d9a4ce10a0992d60f/sql/utils/workflow_audit.py#L303-L316

[peixubin]

这一版还是和自动通过结合了, 在这一步之前再做一个处理, 如果 workflow status 是 autoreview_wrong ,那么就提前保存, return , 你看能不能解决你的问题.

https://github.com/hhyo/Archery/blob/52ce7595d4b3b6e294b49c4d9a4ce10a0992d60f/sql/utils/workflow_audit.py#L303-L316

对于自动驳回和自动通过是否结合，我不能理解。驳回或通过本来就是审核的两个动作，在create_audit里面必定会存在驳回或通过两个场景，这有什么值得疑问的？ 大部分代码是原来就有的，我还不想去大动。我只是增加了对工单状态的判断。 ` 239 def generate_audit_setting(self) -> AuditSetting: 240 if self.is_auto_review(): 241 if self.workflow_type in [WorkflowType.SQL_REVIEW, WorkflowType.QUERY]: 242 if self.workflow.status != "workflow_autoreviewwrong": 243 return AuditSetting(auto_pass=True)

`

[LeoQuote]

可能之前表达的不清晰, 我重复表达下我的建议:

1. 是否自动驳回和是否自动通过是两个逻辑, 自动驳回优先, 先判断是否自动驳回, 再判断是否自动通过, 这样逻辑清晰, 容易理解, 不要混在一起.
2. 驳回还是不驳回, 不适宜放在 generate_audit_settings 内, 就如这个函数名字描述的一样, 是生成工单对应的审批流, 是否自动通过, 审核节点是哪些, 等等. 驳回还是不驳回适宜放在 create_audit 内.

希望你能理解.

[hhyo]

#2772 已处理