同事 A 写了个方法,同事 B 调用了 A 的方法 导致线上直接宕机 这个锅该谁背。。

2021-01-11 10:28:06 +08:00
 yasumoto
公司服务器连续几次宕机，导致有很多业务数据没进来，我们投放是花了钱的 后面找问题发现

是同事 A 写了个 sql 在 mybatis 里面用的<where><if xxx != null> xxx=xxx<if>....</where>类似这样的

同事 B 调用了这个方法，但是他的业务有的时候值没有 导致查询全表 1000 万的数据 机器直接卡死了

在阿里云显示慢 sql 执行了 500 多秒 - -

今天业务找过来说让员工负责一部分损失 ，问问大家 这个损失应该谁背呢
32648 次点击
所在节点    程序员
273 条回复
jimmyczm
2021-01-11 11:23:46 +08:00
如果让员工承担损失，那么也得让员工享受分红。
tony9413
2021-01-11 11:23:59 +08:00
讨论责任没必要，这个领导说了算。
讨论如何规避更合理。A 和 B 如何在方法调用上沟通的？公司流程上有没有代码 review 、静态检查、约束规范？测试有没有覆盖这种场景？如果没有，为什么？
PS：测试不是背锅侠，A 是逃不了责任，你写的方法有隐患，为什么不加防护或在接口使用说明标注？这是典型的给人挖坑。可以去看看防御性编程相关知识。
gam2046
2021-01-11 11:24:17 +08:00
肯定是测试啊，如果没有测试，就项目负责人。
至于为什么不找开发人员，因为没有人做事情能不犯错，不能证明是主观故意造成的损失，就需要有对应职能的人负责
FieldYu
2021-01-11 11:25:40 +08:00
测试开发工程师？
swulling
2021-01-11 11:26:36 +08:00
重大事故罚款，我司有过一次 case，具体罚款如下

总经理 10w，总监 1w，经理 5k，员工 500
ericls
2021-01-11 11:26:43 +08:00
如果都是符合流程且没有恶意就不需要负责任
keepeye
2021-01-11 11:28:08 +08:00
数据库公司的锅，谁让数据库性能这么垃圾
yuankui
2021-01-11 11:29:42 +08:00
赚了钱不给你分，出了事要你背，哪有这么好的事啊？
PureWhiteWu
2021-01-11 11:31:34 +08:00
很简单啊，出问题员工要分担责任，那么赚了钱员工按同样比例获得
ctro15547
2021-01-11 11:33:13 +08:00
开发不自测 ？ 功能测试环境有 1000W 条数据给到测试人员验证吗？ 假如测试环境 100W 能跑通没问题，正式 1000W 爆炸了，怎么怪测试？
kaedea
2021-01-11 11:33:42 +08:00
调用 1 个方法出问题了扣钱，调用 10 个方法没问题加钱，这合理吗？
dream7758522
2021-01-11 11:36:34 +08:00
@sunrain 70mw 有点夸张了啊
l00t
2021-01-11 11:36:51 +08:00
A 有什么责任? 没条件输入，那自然就查全表咯，这逻辑不是很正确吗? 总不能无中生有一个条件出来。A 要有责任，你咋不说数据库 SQL 就有责任，压根不该设计出没条件的语句出来。

B 调的接口，B 自己测试的，B 自己预期的“某些单子必有参数”，然而实际没有，显然责任都在 B 身上。
letking
2021-01-11 11:38:29 +08:00
说让测试背锅的稍微过过脑子好吗？照你们这种说法，一个公司招几个测试，不管什么狗屁问题都甩过去就行了？这样行得通的话谁还会干测试呢？大家都去做写 bug 的开发不好吗？
raaaaaar
2021-01-11 11:41:55 +08:00
@ctro15547 #70 这就是测试的职责呀，难不成测试没有，开发就有？
silianbo
2021-01-11 11:42:16 +08:00
SB 公司，还想甩锅给员工。赶快离职
yy77
2021-01-11 11:42:46 +08:00
没能力反抗的人背损失。社会就是这么残酷。
l00t
2021-01-11 11:45:54 +08:00
@tony9413 #62 你怎么不说 SQL 有隐患，怎么给设计出不需要 where 的查询语句出来；怎么不说数据库有隐患，怎么大表还能允许走出全表扫描来。数据库难辞其咎！
liian2019
2021-01-11 11:46:24 +08:00
和 A 有个啥关系
futou
2021-01-11 11:52:18 +08:00
建议 A 去仲裁
很多公司代码共享，既有 shit 山，又有宝库
按贵司逻辑，谁从宝库里搜点代码出来上线出了问题，应该把当年写宝库的员工找出来鞭尸

这是一个专为移动设备优化的页面（即为了让你能够在 Google 搜索结果里秒开这个页面），如果你希望参与 V2EX 社区的讨论，你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://tanronggui.xyz/t/743718

V2EX 是创意工作者们的社区，是一个分享自己正在做的有趣事物、交流想法，可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX