黑車要求員工一週5個diff

这还不容易？随便搞点有的没的的没用设计和抽象

匿名用户 发表于 2022-8-5 17:12
从来没听过有强制DIFF数量这个东西 人在FB好几年了 你是？

我也从来没听说过，我在选组的时候倒是关注一下，仅此而已。我觉得在infra，你要是不做新项目，而是修bug，feature enhancement啥的，一周有一两个像样的diff就算是高产的。如果从头做新项目，一天5个都行。

匿名用户 发表于 2022-08-05 17:32:22
狗家，之前在ads下面某org，新来的中国经理明确说因为我tenure相对长，所以一个quarter必须多于100个CL，否则认为不达标，“没有承担起自己的责任”  
  
然后我就立马跑了（狗头

那个经理做了多长时间？
从亚麻来的engineer manager ？

按说engineer 出生的应该不会这么没常识。

要么就是故意想赶你走。反损害自己名声也没必要用这种接口。真想赶一个人，有千种办法，然后加明说。

MareBone 发表于 2022-8-5 16:51
有没有大佬解释下diff，pr，cl，cr都是什么

diff, a command to compare two files. (https://www.google.com/search?q=diff)
pr. pull request. a request for code review in github. (https://www.google.com/search?q=pull%20request)
cl. change list. list of the changes, widely used in Google for source review/control.
cr, critique. Google's pr.

在fb diff就是一般说的pr 估计uber也是

匿名用户 发表于 2022-08-05 23:32:09
我也从来没听说过，我在选组的时候倒是关注一下，仅此而已。我觉得在infra，你要是不做新项目，而是修bug，feature enhancement啥的，一周有一两个像样的diff就算是高产的。如果从头

那你在说什么买它一直有这个东西？表达能力真的堪忧…

jy_121 发表于 2022-08-05 17:25:24
其实之前diff数量在promo和perf时都有人会去看的，虽然质量更重要但是数量也会大致看下，因为看数量确实有点low所以不会直接说明白。五个确实不是硬指标，但是有了这个dashboard就更容易就

请问perf时候，这个会看过去半年的diff count 还是过去一年的 还是所有的啊….diff质量是他们自己挑着看，还是工程师可以选择一些比较好的diff给他们看啊？

匿名用户 发表于 2022-8-6 02:48
那你在说什么买它一直有这个东西？表达能力真的堪忧…

我说的是在profile下activity，用起来非常不方便，没有统计，也没有黑车的那种一目了然的赛马图。随便挑一两个看看有点感性认识而已，连定性都难做到，定量就更甭提了。

听懂了吗？也许你说的工具我不知道（如果有我也不奇怪），但我想这种有百害的杀器还是不用的好。

黑人不是牙膏 发表于 2022-8-6 07:25
请问perf时候，这个会看过去半年的diff count 还是过去一年的 还是所有的啊….diff质量是他们自己挑着看， ...

这个没有固定的。黑车虽然每年perf两次，但是因为年中的那次如果不申请升职就几乎没什么用，所以还是看年底的review，具体怎么看diff那就是每个经理自己决定了。和promo的时候不一样，promo时是会要你自己选一些自己觉得不错的diff，同时committee成员也会去浏览你这个review周期的其它diff。

统计个metrics本身没毛病
大张旗鼓的突出一项metrics说明管理层对ic缺乏信任。和当年国内⚽️联赛要求体测一个道理。