求问 如果senior要求很高 block住你的进度怎么办

kanaz 发表于 2021-2-20 14:35
开个task记下跟你diff不相关的comment 然后把task贴上到comment 说之后会处理 diff只想focus在project相关 ...

我能理解senior的行为 只是觉得如果要求做到完美就应该多给时间 在时间就这么短并且没法往后拖的情况下 真的不可能尽善尽美。

这senior明显是brilliant jerk，仗着自己懂legacy code，很有可能自己写一堆shiiit，然后要小兵来擦屁股。lz学会怎么push back吧，分jira，每次focus on one thing。

可以拿LP撑腰 他是insist on the highest standards 你是 bias for action 和 disagree and commit

补充内容 (2021-2-20 16:02):
但说真的我们组senior也是这样的。。而且曾经有过revision x让我这样改 revision x+2又说让我改回revision x的经历 我说这不是我之前的写法吗？他给我来了一句 哦我刚才喝了点红酒 给忘了。。
我觉得最好的方法就是摸清这个人的风格，然后按照他的风格写就是了，然后每次发CR前反复检查，虽然不能杜绝吹毛求疵但是可以减少

曾去过十分类似的组，最后以转组了事。现复盘如下：
1. 回复将会在专门开一个code cleaning的ticket里一并解决，这个ticket ddl就在眼前，不希望延误工期
1.1 可以在weekly meeting上提出开code cleaning session的事，比如两个月来一次，专门处理legacy code。这样不仅可以满足senior的嗜好，还可以增加你的visibility
2. 1 on 1和老板反映此事，批评该senior做事情不professional，分不清top priority/low priority issue
3. 如果老板的态度是支持此人，或者表示无力改变现状，说点冠冕堂皇的话来打发你，那尽早换组。这样的组呆久了只会伤害你自己的career path
. 1point3acres

特别解释一下1.1。不要忘记，legacy code是前人写的，senior没资格点名要求你在你的PR里解决该问题。应该在专门的session改进此类问题，而不是越改越乱、PR越写越大。这样对谁都不好。

以及大家也不要单纯觉得Senior给你抓bug是无私、对你好的表现。在商言商，作为Senior，除了lead的project以外，自己对组里的impact都是每次performance review的时候需要着重考量的点，以及我记得亚麻的Senior是要担任一定management的任务的，所以对于junior engineer是有lead的义务的。这种review都是要量化自己的成果的，所以怎么体现呢？PR comment的数量其实就是一个很好的point。所以有时候如果发现PR comment有些故意找茬的意味在里面，不要怀疑自己，你没有什么问题，是给你comment的兄弟想升职了或者保住饭碗而已。所以勇敢地怼回去是正确的做法。



补充内容 (2022-07-30 16:21 +8:00):
把我楼下某个补充的观点放在这里，大家自行判断。对于Jr来讲，ddl也好，quality也好，都不如个人成长（promotion）重要。在Promote到Senior之前，你的一切行为就应该为了promotion服务，如果赶ddl对你更有帮助那就多赶ddl，反之如果在一个ddl让位于quality的组，那就好好管quality

亚麻有个烦人的LP Disagree and commit。
有趣的是，你可以disagree，但你最后还是得给我commit

csushin1992 发表于 2021-2-20 16:13
亚麻有个烦人的LP Disagree and commit。
有趣的是，你可以disagree，但你最后还是得给我commit

这个就很有趣了。senior作为和我不存在上下级关系的同事给我一个算不上decision的decision，我作为平辈给了一个不同意该decision的decision，那最后我应该commit哪一个decision呢？

csushin1992 发表于 2021-2-20 16:13
亚麻有个烦人的LP Disagree and commit。
有趣的是，你可以disagree，但你最后还是得给我commit

可不可以问一下怎么做到让disagree的人commit呀。。他如果不approve我的code 我就没有办法commit呀

匿名用户 发表于 2021-02-20 00:18:38
可不可以问一下怎么做到让disagree的人commit呀。。他如果不approve我的code 我就没有办法commit呀

disagree and git push.

匿名用户 发表于 2021-02-19 22:24:36
但是他是待了很久的senior 他可以不approve你的cr 一直拖着你直到你改
至少我观察到的 组里的人被senior留了很多comments的 都是加班加点的在改 没看到几个真的push ba

公司/组不行，走人

laptopkek 发表于 2021-2-20 15:57
可以拿LP撑腰 他是insist on the highest standards 你是 bias for action 和 disagree and commit

. check 1point3acres for more.补充 ...

re摸清对方脾气，虽然是个不得已的办法，但是确实是生存下去的一个方法。
有的人就是比较喜欢找存在感，建议存在感给足，那个专门session的方法感觉不错，可以考虑在开clean up session的时候多提是xx发现的。

我的办法是低头（不是特别推荐），一个pr评论多了马上和对方chat，争取了解ta的rationale，摸清哪些comment是必须address的，因为其实你pushback不了这些ta觉得必须address的评论（并且有可能会吵起来，情况越来越糟）。 如果真的完全pushback不了，又影响deadline建议考虑cc老板或者把两个人的chat内容做成notes，每次抄送留记录。