git merge使用不当引发的代码丢失血案

原文: git merge使用不当引发的代码丢失血案@blog.23lab.com

背景

几年前大批量的团队都在转用gitgit的本地库和分支特性让代码管理的便利性大大增加,也因为本地库和分支的大批量使用导致了代码之间的频繁merge,我们团队以前就有遇到过git merge以后丢代码的情况,表现就是某些变更开发A提交了,经过中间以序列的commit, mergepush,最终某些改动没了,去看commit history,也没有相应的变更记录。一次次commit对比以后总是会找到某次的merge,但是也没法解释为什么会丢,最后为了保险通常会选择beyond compare全量对比一次确保所有丢失的代码找回来,那是一个痛苦而又让人睡不安稳的解决办法。

今天又遇到了一次这个问题,这次的更加诡异,在我查为什么丢的时候,另外一个同事提交代码以后竟然丢失的代码回来了,我当时脑袋里面只想到一个词,那就是: 见鬼。但是冷静下来想,git这种工具出现真丢代码的概率肯定很小,肯定是我们姿势不对。于是认认真真的研究了我们今天的提交记录,终于找到答案,就是git merge后只提交部分代码导致的,这里顺带提一下,git pull等于git fetch + git merge,所以git pull时也需要特别关注本文提到的问题。

验证过程

为了验证这个问题,我建了一个全新的git库来复现上述问题,根据我们遇到的情况模拟,整个过程我使用三个用户模拟交叉提交和合并,分别是user1,user2,user3,为了方便我没有选择注册三个号,而是在提交信息里面都带上了名字,可以制造两次代码冲突,并且两次都进行部分提交。最终的代码提交记录如下:

image.png

为了更清晰的看到这个提交过程,我将三个用户的提交分开来,并根据提交先后做了下面的图。


image.png

代码怎么丢的?

首先,第1步和第2步,user1和user2分别在本地做修改,共同修改了file1,只有user2对file2也做了修改,所以如果这两人的代码合并,可以预计合并的时file1会冲突,file2不会。user2修改完成以后做了一次git push,这时候git远端库的head应该是指向user2的最后一次提交的。接着user1做git pull,因为file1被两个用户修改了,所以需要合并,接着user1处理完冲突以后只提交了file1,没有提交file2(用命令行需要手动reset,但是用有些GUI工具是可以选择只提交file1的)。此时的库已经丢失了user2:file2:add line的那次改动,这就是血案的开始,丢代码。可以对比一下 merge前merge后

代码怎么"恢复"的?

前面只是验证了代码怎么丢,但是事情没有结束,我上面提到更诡异的是代码恢复了,这又是怎么回事呢?user1在第3步合完代码以后,自己进行了两次提交,修改了file3和file1,同时,user3加入并本地修改了file1(预计会和user1冲突),接着user1git push,然后user3 git pull,这时候会报file1冲突,同样,user3犯了user1类似的错误,只提交了冲突文件file1。按第3步丢代码的经验,file3的改动肯定是没有了,验证以后确实如此,但是再看看file2的变更呢,它竟然又回来了,所以所谓的恢复不是恢复,所谓的丢失没有丢失,就问你怕不怕。更可怕的是我们再去看file2的变更历史。

image.png

中间消失和恢复的过程没有任何变更记录,这也是之前我们经常查到没法往下查到地方。从表现上看,是第一次误操作忽略了这一行的变更,而第二次误操作,可以理解为是忽略了忽略添加这一行的那一次变更,但是,所有的都没有记录。

总结

上面的实验可能看起来有点繁琐,总结一下,核心的问题就是不要在git merge的时候只提交部分代码,特别是好多人刚刚用git的时候,运行完成一次git merge,发现自己修改了1个文件,结果git提示要提交100个,这时候小心谨慎的新同学就会选择只提交自己的变动吧,殊不知这是在犯大错啊。除了这种情况,之前还遇到有的新同学会修改git默认生成的merge信息,这也是非常不好的习惯,改了以后在查问题的时候没办法很容易的看到哪些是merge。所以请注意:

  • git merge以后不要部分提交!!

  • git merge以后不要部分提交!!

  • git merge以后不要部分提交!!

  • git merge的信息不要手动改!!

  • git merge的信息不要手动改!!

  • git merge的信息不要手动改!!

文中提到的实验库的地址是: find-the-missing-code, 有兴趣的童鞋可以自己详细看一下。

最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 151,511评论 1 330
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 64,495评论 1 273
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 101,595评论 0 225
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 42,558评论 0 190
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 50,715评论 3 270
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 39,672评论 1 192
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 31,112评论 2 291
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 29,837评论 0 181
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 33,417评论 0 228
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 29,928评论 2 232
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 31,316评论 1 242
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 27,773评论 2 234
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 32,253评论 3 220
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 25,827评论 0 8
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 26,440评论 0 180
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 34,523评论 2 249
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 34,583评论 2 249

推荐阅读更多精彩内容

  • 第一部分是文档,为了日后查看方便放到了前边[可能敲错了,详见阮一峰老师的文章]http://www.ruanyif...
    白璞1024阅读 983评论 0 49
  • 以下笔记主要参考gitgot,大致了解git使用和原理。 第一部分我们从个人的视角去研究如何用好Git,并且揭示G...
    carolwhite阅读 2,269评论 0 1
  • "常见机器学习算法实现代码(DeepLearning Tutorials/PCA/kNN/logistic reg...
    hzyido阅读 1,057评论 0 1
  • 嗨,你好哇! 不知道你现在在哪,在做些什么呢! 倘若你的身边是另外一个女孩,我也不会感到难过或生气:我这么死心塌地...
    杨爽blue阅读 1,046评论 6 10