这种情况该不该跟另一个senior正面刚

最近职场上有这个情况，想问问大家到底要不要正面刚。
最近小弟在做一个挺大的的重构项目，其中有一个模块，另外一个senior发现了问题，正在提交code review补丁，但是在审MR的时候我发现，他的补丁不但不能完全解决问题，反而block我现在在做的东西，搞得我需要对已经做完的部分进行重构（scope不确定）
于是我们两个简单交流了一下，他也意识到自己的补丁不能完全解决问题，解决问题还是需要我这边重构，但是因为他已经做了，所以想提交代码，现在只有我的review feedback不能address。
我听到就有点犹豫，一个是重构正在进行时，他这个fix可能能部分解决问题，但是不能解决全部问题不说，我以前的工作被影响，要重写，后面的重构又可以解决他所说的问题，这个fix也不是很有必要。于是没有同意他提交修改。
现在想问问大家，怎么解决这个conflict呢？如果拒绝他提交修改要什么理由比较好？
请大家不吝赐教，多谢🙏

这感觉是bq里面典型的，tell me a time when you had conflicts with your colleagues

我这里说的你们可能已经想过了: 把他的CL提交了, 你的branch里先把那个commit revert然后rebase, 然后等你的做好了再提交; 这样你们短期内有一个不完整的补丁, 长期会有一个完整的解决方案, 你也不需要重写你的branch, 感觉不存在什么冲突啊?

没有人想自己辛苦些的代码要被放弃掉。你说的太直接，他肯定不高兴。如果是死脑筋白人，我见过很多跟你杠到底的。
公事公办就是你decline他的review，说不能根本解决问题，而且你已经在解决了。私下协商的好，就让他主动撤掉，这是聪明人都会选的，两边都不会太难看。
也要看你是否得罪得起这个人，不行先让他提交，你再想办法改回来。因为这里已经确定你得罪不起他，改的时候，也不要太直接了，用复杂一点的手段改，不要直接删了他的

发的急没说清楚，senior发现的这个bug不是我的work，是以前老的code里面一个bug，我的新框架可以解决这个bug。

最直接的是，你们两个之间解决不了的问题直接escalate给你的manager。 协调组里的工作进程矛盾是他/她的本职工作。另外，哪怕是你需要让步，让manager知道为什么你之前的工作要重做也是很重要。你需要确保自己的工作受影响了以后不背不必要的锅。

侧面的问题就是作为一个team工作的优先级问题。这个漏洞不修，或者senior的patch都有什么impact？用户或者公司的业务受影响不？在这些方面比起来，你个人的工作是可以牺牲的。如果impact只是team内部的cost，那就是比较不修bug和重做你的project哪个代价更大。

确认一下事实，如果理解错了那可能后面的讨论就不合适

1 你们组有legacy code，你在refactor，会根本性的改变一些底层逻辑
2 你们的code 有一个bug，这个senior的patch能解决部分问题，你的refactor可以解决这个bug相关的底层问题
3 你的code目前还么有deploy而且短期内deploy不了，regardless这个bug
4 如果他的patch deploy了，你可能要重做很多东西


这里首先一个问题，这个bug有多重要，能否接受这个bug不fix？如果可以，escalate到你们的pm/sdm，争取postpone他这个task

如果这个bug会影响到你们的product，那么fix是必要的，不管有没有你这个refactor都要fix，你就想像成他在你refractor之前就fix了，你anyway要include这个fix。这里你提到他只能partial的fix，要么把你相关的工作isolate出来先deploy，他必须在你的基础上fix；要么他在原有基础上fix所有的问题，一个patch不能解决所有问题本身就是一个很好的comment来request change

Shubin_ren 发表于 2021-09-13 18:12:03
确认一下事实，如果理解错了那可能后面的讨论就不合适

1 你们组有legacy code，你在refactor，会根本性的改变一些底层逻辑

谢谢你的回复，您的假设基本正确的，这个bug是一个看起来非常糟糕的bug，但是在里面一年多了可能两年了，没有人报告过，impact也很小。然后刚好就是在我正在重构的这个期间，senior自己发现了，也不是他负责的，他也是本着工程师敬业的精神，看不过去想修了，但没想到牵一发动全身，他自己也发现超过了scope，从impact上说可修可不修

just say no

豹豹 发表于 2021-09-13 17:59:48
最直接的是，你们两个之间解决不了的问题直接escalate给你的manager。 协调组里的工作进程矛盾是他/她的本职工作。另外，哪怕是你需要让步，让manager知道为什么你之前的工作要重做也是很重

谢谢您的回复，很有帮助，我现在的困惑是，我对于这个fix能修多少，又会造成多少新的问题并不是很清楚，因为我看来这个fix已经造成了新的问题。但是本身figure out这件事我不知道时间上值不值得，因为dive deep也需要时间，所以还是很疑惑，本身弄透他的solution可能需要2-3天全职时间这样，也耽误我的项目进度。我现在能想到的就是，开个会，include我的manager进来，然后我会前尽量理解他的解法，present一些trade off这样。让我们manager也听一下，好做决定。

你如果自己能确认自己能 比较有效低成本的修好，就和senior说，用你的方案
如果你自己没有办法低成本有效的修好，那就听senior的。

可以把这个fix对于production和project的pros&cons都列一下，根据优先级就好选了
根据你的描述，fix对于production只解决一部分问题，但是对于project会导致延期
这就是在现在生产环境bug和项目进展之间取舍，那就不是一个纯technical的决定了，可以引入PM，EM做个决定啊