hotfix 时如果知道问题的位置,可以直接把这部分代码注释掉吗? - V2EX
1oNflow

hotfix 时如果知道问题的位置,可以直接把这部分代码注释掉吗?

  •  
  •   1oNflow Mar 22, 2020 1172 views
    This topic created in 2242 days ago, the information mentioned may be changed or developed.

    hotfix 或者出 bug 的的标准操作是怎样的?

    需要 revert 最近的提交,还是直接做出新的修改,比如直接修复,或者先注释掉新的提交(不影响核心功能),后面慢慢改?

    5 replies    2020-03-22 22:15:58 +08:00
    123444a
        1
    123444a  
       Mar 22, 2020 via Android
    注释代码能解决问题?你的意思是取消某个功能吧?一般没有人会注释代码,都是用 flag 来控制功能的启动,就是 --help 出来那些 flag
    yhxx
        2
    yhxx  
       Mar 22, 2020
    你说的是线上故障?
    标准流程应该是先回滚,然后看情况再好好修
    1oNflow
        3
    1oNflow  
    OP
       Mar 22, 2020 via iPhone
    @yhxx
    感谢回复,目前的情况是个 web 应用的前端部分( react ),有个 component 获取数据后显示的逻辑出错了,因为能定位到它,所以想直接注释掉这个 component 用户就看不见了,可以后续慢慢修。

    回滚应该更好,但是需要回滚的部分不是最新 commit 的,中间也有别人开发的部分上线,害怕回滚了某个 commit 会影响别人的代码…

    @123444a
    目前用的是 nodejs,前端也并没有设定特定功能的 flag,这种做法在 java 后端用的比较多?
    123444a
        4
    123444a  
       Mar 22, 2020 via Android
    JS 开了混淆没有,开了就注释然后加个 ToDo,没开就删代码
    1oNflow
        5
    1oNflow  
    OP
       Mar 22, 2020
    @123444a 没,就写在 jsx 里的,小公司的项目没事,主要担心大公司流程是不是不能随便删(注释)代码就来问一下
    About     Help     Advertise     Blog     API     FAQ     Solana     3418 Online   Highest 6679       Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 64ms UTC 12:12 PVG 20:12 LAX 05:12 JFK 08:12
    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