查看: 1626| 回复: 5
跳转到指定楼层
上一主题 下一主题
收起左侧

[管理] 印度经理一点点屁事喜欢挑刺咋办

全局:

注册一亩三分地论坛,查看更多干货!

您需要 登录 才可以下载或查看附件。没有帐号?注册账号

x
组里L6 data engineer,前段时间approve了一个CR,然后最近说这个CR有个地方不够好,加了一个什么东西可以增加performance。这东西加不加其实没太大区别,但是你说你不加吧又显的自己没有keep highest standard。然后跑过来问我why do you approve this?我也不好反驳。只好吃个闷亏,毕竟有的时候review的时候看个大致没问题就行了。
还有很多其他的例子,基本对工作进度不影响,但是他确实可以给你扣个不仔细的帽子,也很难反驳。请问是我的问题吗还是老板太追求完美?

上一篇:EA旗下BioWare宣布裁员50人
下一篇:two sigma面试一定要c++吗?
地里匿名用户
推荐
匿名用户-AYMSA  | 添加认证 | 2023-8-26 06:44:56
本帖最后由 匿名 于 2023-8-25 18:46 编辑

-baidu 1point3acres你觉得加不加没什么区别就和他说啊,加之后什么样,不加什么样,或者说加了有什么downside,不是要紧之后也可以加的话就说可以follow up 一个cr,先把这个release测试。

什么叫一点屁事?你觉得他麻烦就换个人review或者直接override不就行了?每个人标准不一样,你既然让人家review了人家提出意见没有任何问题,采纳不采纳合不合适是另外一回事。
回复

使用道具 举报

地里匿名用户
🔗
匿名用户-OJZCW  | 添加认证 | 2023-8-26 07:25:45
不要和**去讲道理
回复

使用道具 举报

🔗
 楼主| weihaoren 2023-8-26 08:03:10 来自APP | 只看该作者
全局:
匿名用户 发表于 2023-08-25 15:44:56
你觉得加不加没什么区别就和他说啊,加之后什么样,不加什么样,或者说加了有什么downside,不是要紧之后也可以加的话就说可以follow up 一个cr,先把这个release测试。
..
什么叫一
哦哦你理解错了,是我review了别人CR,他觉得我review的不够仔细,就很无语
回复

使用道具 举报

地里匿名用户
🔗
匿名用户-KULO3  | 添加认证 | 2023-8-26 08:14:38 来自APP
你的话其实有点前后矛盾,究竟是加了可以提高performance还是加不加没区别。code review这东西真的每个公司文化不一样,有的是无责stamp型,有的是endorsement型,
回复

使用道具 举报

🔗
 楼主| weihaoren 2023-8-26 08:36:04 来自APP | 只看该作者
全局:
匿名用户 发表于 2023-08-25 17:14:38
你的话其实有点前后矛盾,究竟是加了可以提高performance还是加不加没区别。code review这东西真的每个公司文化不一样,有的是无责stamp型,有的是endorsement型,
哈哈是有点,就是加了可能有微小的performance improvement
回复

使用道具 举报

您需要登录后才可以回帖 登录 | 注册账号
职场达人
  • ↑ 本版用于讨论职场各种干货话题,闲聊请去🔗聊聊或者🔗匿名版
  • ❌ 本版严禁水贴,引战,发布广告,拉群,贴个人联系方式,扣分无警告
  • ☑ 求职、面经等去 🔗北美求职和 🔗回国求职大区,刷题和学习请去 🔗终身学习大区
  • ☑ 请去专版发布 🔗内推, 🔗招聘信息,和讨论 🔗创业内容
  • ☑ PIP / DevList/ Need Support 等话题也已开设 🔗专版

本版积分规则

>
快速回复 返回顶部 返回列表