Yearning icon indicating copy to clipboard operation
Yearning copied to clipboard

需求:sql 提交后,期望 “审核”和“执行” 分开,由提交人自主选择执行时间

Open alingse opened this issue 2 years ago • 5 comments

未命名

从现在的 “提交” --> "审核并执行" 改成 “提交” --> “审核” --> "执行"

sql 提交的人知道的事情

需求出发点 ,sql 合理性, sql 执行的时机,sql 执行前需要执行的操作,sql 执行后的后续操作(清理cache /关闭开关/上线前端)

sql 审核的人知道的事情

需求出发点, sql 合理性

结论

执行还是由提交人自己执行会好点。只是走一个审核流程。

alingse avatar Aug 29 '22 03:08 alingse

提交人是研发,审核人是研发leader,执行人是DBA,很合理呀

pakatv avatar Sep 17 '22 08:09 pakatv

目前是最后一个审批人审批结束后,自动开始执行。感觉执行时间改成人工选择会更好一点。有的时候,研发白天提交申请,但为了安全,应该放到晚上或者业务低峰期执行更好

blylei avatar Oct 26 '22 01:10 blylei

本来就可以分开呀

Blylei @.***> 于 2022年10月26日周三 上午9:50写道:

目前是最后一个审批人审批结束后,自动开始执行。感觉执行时间改成人工选择会更好一点。有的时候,研发白天提交申请,但为了安全,应该放到晚上或者业务低峰期执行更好

— Reply to this email directly, view it on GitHub https://github.com/cookieY/Yearning/issues/675#issuecomment-1291380203, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOFFQHYPPEHIEXQKFRLF6ITWFCE7HANCNFSM574GX7UA . You are receiving this because you commented.Message ID: @.***>

pakatv avatar Oct 26 '22 07:10 pakatv

而且本来也可以由提交人选择执行时间

kai fedora @.***> 于 2022年10月26日周三 下午3:02写道:

本来就可以分开呀

Blylei @.***> 于 2022年10月26日周三 上午9:50写道:

目前是最后一个审批人审批结束后,自动开始执行。感觉执行时间改成人工选择会更好一点。有的时候,研发白天提交申请,但为了安全,应该放到晚上或者业务低峰期执行更好

— Reply to this email directly, view it on GitHub https://github.com/cookieY/Yearning/issues/675#issuecomment-1291380203, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOFFQHYPPEHIEXQKFRLF6ITWFCE7HANCNFSM574GX7UA . You are receiving this because you commented.Message ID: @.***>

pakatv avatar Oct 26 '22 07:10 pakatv

而且本来也可以由提交人选择执行时间 kai fedora @.> 于 2022年10月26日周三 下午3:02写道: 本来就可以分开呀 Blylei @.> 于 2022年10月26日周三 上午9:50写道: > > 目前是最后一个审批人审批结束后,自动开始执行。感觉执行时间改成人工选择会更好一点。有的时候,研发白天提交申请,但为了安全,应该放到晚上或者业务低峰期执行更好 > > — > Reply to this email directly, view it on GitHub > <#675 (comment)>, > or unsubscribe > https://github.com/notifications/unsubscribe-auth/AOFFQHYPPEHIEXQKFRLF6ITWFCE7HANCNFSM574GX7UA > . > You are receiving this because you commented.Message ID: > @.***> >

请问下怎么设置的?

airstring avatar Feb 26 '24 08:02 airstring