我们仍需心怀敬畏的面对每一行代码 - V2EX
V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
请不要在回答技术问题时复制粘贴 AI 生成的内容
lucifer1108
V2EX    程序员

我们仍需心怀敬畏的面对每一行代码

  •  1
     
  •   lucifer1108 2022-02-23 17:02:43 +08:00 3375 次点击
    这是一个创建于 1377 天前的主题,其中的信息可能已经有所发展或是发生改变。

    起因是昨天写一个实时流任务,订阅一个 topic,处理后写到下游.框架搭起来想当然就上线了.任务起来之后可以看到消费,但下游迟迟看不到数据.

    刷了十几分钟后,瞟了眼数据源的监控,消息数指数级上升,几个 consumer 全部 1000W+的 lag.

    立马停了任务,review 了下代码,发现 sink 的 topic 直接取了 message 的 topic.于是变成了消费->生产->消费的死循环.并且每次消费和生产,消息数翻倍.

    联系了队列的生产和消费的同学,重启了消费任务,重复数据影响也不是很大,没有影响到业务核心流程,也没有用户反馈.提心吊胆了一下午,但也还是抹过去了.

    整个晚上人都挺懵的.今天一度不敢上线.想了下,还好只跑了十几分钟,并且非核心流程.假如是晚上上线,这样一个放大器跑一晚上.怕是整个集群的存储都要崩掉...

    公司的队列没有鉴权机制,生产和消费都很随意.感觉这样也是有问题的.

    不过,最主要的是,对待代码还是要敬畏,工作久了,就没有刚开始的那种初心了,感觉自己可以 cover 的住,变的傲慢.

    12 条回复    2022-02-24 11:14:57 +08:00
    bigbyto
        1
    bigbyto  
       2022-02-23 17:07:11 +08:00
    其实总有麻木大意的时候,gitlab 那些大佬们应该够厉害了,当初还是发生了低级错误导致数据被删。公司内部要有一套机制去对付这类事发生才能从根本上去杜绝。
    mxT52CRuqR6o5
        2
    mxT52CRuqR6o5  
       2022-02-23 17:09:03 +08:00   1
    靠敬畏可没法保证不出问题,不出问题得靠合理的流程,我读下来感觉你们好像没有完善的测试环境与测试流程
    mxT52CRuqR6o5
        3
    mxT52CRuqR6o5  
       2022-02-23 17:10:08 +08:00
    应该去敬畏流程和制度,而不是去敬畏代码
    oxoxoxox
        4
    oxoxoxox  
       2022-02-23 17:12:01 +08:00
    1.代码 review 流程需要改善
    2.代码提交前需要自测
    3.需要独立的测试验证流程
    4.系统需要异常通知机制
    lucifer1108
        5
    lucifer1108  
    OP
       2022-02-23 17:15:20 +08:00
    上线流程的问题:公司有比较完善的 ci cd 流程和平台,但是实时流任务和服务不太一样,允许本地打包上传,于是有了上面的骚操作.
    @oxoxoxox
    @mxT52CRuqR6o5
    darkengine
        6
    darkengine  
       2022-02-23 17:16:29 +08:00   2
    不是对代码敬畏,要对上线敬畏
    3dwelcome
        7
    3dwelcome  
       2022-02-23 17:37:32 +08:00
    所以我更喜欢写客户端代码,崩了就崩了,让客户重启一下就行,这叫 BUG 。

    而线上服务奔溃,那就属于严重事故,要担责任的。

    问题是你线上服务,有时候并不知道什么时候会挂,提心吊胆的感觉,真是很糟糕。
    abigeater
        8
    abigeater  
       2022-02-23 17:39:05 +08:00
    赞同,但要求快速上线的前提下,总是忽视了这些问题。
    popbones
        9
    popbones  
       2022-02-23 18:08:45 +08:00 via iPhone   1
    恰恰相反,无论多么小心,人其实是开发流程里最薄弱的环节,一个人的敬畏可以走多远?每个人都能做到吗?浑身是铁能打几根钉子?这个时候要依靠系统和流程,单元测试、集成测试、Peer Review 和 QA ,CI/CD 这些。否则每天要么麻木不仁,要么诚惶诚恐,加班、复查、猝死。
    hejw19970413
        10
    hejw19970413  
       2022-02-23 18:12:13 +08:00
    以前我实习的时候,就是因为把一个判断写错了,就匆匆忙忙上了线,然后就没有然后了。从那时起,无论多紧急,都要再三确认,才能上线
    arvinsilm
        11
    arvinsilm  
       2022-02-24 09:16:05 +08:00
    这。。。。哪怕自测跑一条消息也能发现吧
    ezrealrao
        12
    ezrealrao  
       2022-02-24 11:14:57 +08:00
    让我想起了我之前做订单,一直和 increment 和 decrement 分不清,然后在扣减库存的时候一直 increment ,商户发现后质问我们,为什么订单越多,我这库存越多?
    关于     帮助文档     自助推广系统     博客     API     FAQ     Solana     905 人在线   最高记录 6679       Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 25ms UTC 21:40 PVG 05:40 LAX 13:40 JFK 16:40
    Do have faith in what you're doing.
    ubao msn snddm index pchome yahoo rakuten mypaper meadowduck bidyahoo youbao zxmzxm asda bnvcg cvbfg dfscv mmhjk xxddc yybgb zznbn ccubao uaitu acv GXCV ET GDG YH FG BCVB FJFH CBRE CBC GDG ET54 WRWR RWER WREW WRWER RWER SDG EW SF DSFSF fbbs ubao fhd dfg ewr dg df ewwr ewwr et ruyut utut dfg fgd gdfgt etg dfgt dfgd ert4 gd fgg wr 235 wer3 we vsdf sdf gdf ert xcv sdf rwer hfd dfg cvb rwf afb dfh jgh bmn lgh rty gfds cxv xcv xcs vdas fdf fgd cv sdf tert sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf shasha9178 shasha9178 shasha9178 shasha9178 shasha9178 liflif2 liflif2 liflif2 liflif2 liflif2 liblib3 liblib3 liblib3 liblib3 liblib3 zhazha444 zhazha444 zhazha444 zhazha444 zhazha444 dende5 dende denden denden2 denden21 fenfen9 fenf619 fen619 fenfe9 fe619 sdf sdf sdf sdf sdf zhazh90 zhazh0 zhaa50 zha90 zh590 zho zhoz zhozh zhozho zhozho2 lislis lls95 lili95 lils5 liss9 sdf0ty987 sdft876 sdft9876 sdf09876 sd0t9876 sdf0ty98 sdf0976 sdf0ty986 sdf0ty96 sdf0t76 sdf0876 df0ty98 sf0t876 sd0ty76 sdy76 sdf76 sdf0t76 sdf0ty9 sdf0ty98 sdf0ty987 sdf0ty98 sdf6676 sdf876 sd876 sd876 sdf6 sdf6 sdf9876 sdf0t sdf06 sdf0ty9776 sdf0ty9776 sdf0ty76 sdf8876 sdf0t sd6 sdf06 s688876 sd688 sdf86