这是一个创建于 2996 天前的主题，其中的信息可能已经有所发展或是发生改变。
同事是架构师，我们目的是做新旧系统的对比测试。 为了对比新旧消息处理是否异同，希望可以转发生产环境的流量做转发测试，侵入代码写了一个 proxy 。架构师大大在没有跟我打招呼的情况下，直接修改了线上代码在生产环境做测试。生产环境代码啊！两个人消息居然没有同步！觉得好不爽啊！测试期间出现一些 bug 导致线上环境不断重启。如果不是运维发现监控异常提醒，大家都没有发现。
其实应该有 gor 或 duplicator 或 tcpcopy 可以做到不侵入转发。架构师大大之前也让我去做过非常深度的测试，我甚至给出了图表，还横向对比了几款常用的流量转发工具。觉得自己非常尴尬，自知工作一年能力欠佳，表达能力欠佳，多次想表达自己，都被截断。已经非常努力的去适应架构师大大的工作模式，可是还是做不到他满意。我很尊敬技术人，遇到大神眼里发光，可是同事之间为什么偏偏觉得多了一层“傲慢”。很迷茫，现在都害怕跟他交流，三句不到就换来责问的语句，迷之尴尬。
 |
1
Hallujah 2016-10-16 08:10:57 +08:00 via Android  1
为了解恨:你把他的代码也改了。
为了面子:宽容地原谅他。 为了以后的工作:狠狠地艹他。 |
 |
2
wsy2220 2016-10-16 08:11:29 +08:00 via Android
谁改谁负责嘛
|
 |
3
bsidb 2016-10-16 08:36:20 +08:00 via Android 1
和非常忙的大大们交流，我自己的一点体会，也不知道适用不适用，权当一点经验交流吧~
跟非常忙的 boss 交流，要先抛出他最关心的问题的结论出来，如果他感兴趣再解释细节，如果他不感兴趣，直接忽略就好了。 虽然详细介绍一个事情的前因后果的过程是: 先 blablabla 介绍背景，再分析原因，最后得到结论。 但是 boss 一般都挺忙的，大多数时间只想听一个结论，不想听比较冗杂的分析过程。 所以和他们交流，先抛出你的结论，然后问他“您想听一下具体的原因么？” 如果他有兴趣，再简单介绍一下很概括的原因。如果他还有兴趣，再详细介绍一些细节的原因。如果他没兴趣，那就不用说了，他也不关心。 另外有人是天生的责问体质，不光对你，对所有的下属都是责问的状态，自己不要往心里去就好了~ |
 |
4
shoaly 2016-10-16 08:53:56 +08:00
我建议还是要求他之后跟你多沟通, 或者让他写一个接口给你 你去调用..把他需要知道的信息都通过接口传进去, 至少这样才能让你自己的代码可控.
随便被插入一行"其他人"的代码, 先不说尊重不尊重, 出了问题 debug 上都是一个麻烦. 莫非你要先全部下载线上版本, 然后跟你自己的做 diff 然后再来找原因? 如果都已经开始线上线下 diff 了, 不如邀请你们同事一起跟你在开发版本中开一个 branch. 你来负责合并 |
 |
5
binux 2016-10-16 08:54:01 +08:00
在线上做这种事情，责任比怎么做更重要，这种活，责任重产出还低，躲还躲不急呢。
作为小兵，做了足够的调研线下测试，更多的时候不是说保证万无一失，而是表明一个态度，你看我做了那么充足的准备，出错了责任也没那么大。 现在好了，有人愿意担这个责任，干这个活，那就让他干去呗。 |
 |
6
ebony0319 2016-10-16 08:57:51 +08:00 via Android
喂，你的代码。
不是公司的代码。 |
|
7
ebony0319 2016-10-16 08:58:46 +08:00 via Android
少了一个逗号。
喂，你的代码。 不，是公司的代码。 |
 |
8
laoyuan 2016-10-16 09:00:33 +08:00
你也傲慢呗，来啊互相伤害
|
 |
9
songchao 2016-10-16 09:40:16 +08:00 via Android
怎么没有 Git ？
|
 |
10
hasbug 2016-10-16 09:51:02 +08:00
1 楼已经说了
|
 |
11
tabris17 2016-10-16 09:51:37 +08:00 via iPhone
撕逼呗
|
 |
12
newworld 2016-10-16 10:13:28 +08:00
不要怂 就是艹他 干得他不要不要的！
开玩笑 O(∩_∩)O~ 沟通很重要 一定要沟通。。。 |
 |
13
kouryu 2016-10-16 10:39:57 +08:00 via iPhone
生产环境这种改动没有操作规范吗？
报备一下，下次出问题就直接说好像代码被改了，给自己争取缓冲时间 23333333 |
 |
14
sutra 2016-10-16 10:40:02 +08:00
没有经过 QA 部门签字的代码也能进入生产环境？药丸。
|
 |
15
ldbC5uTBj11yaeh5 2016-10-16 10:40:43 +08:00 via Android
不要怂，干他啊。
具体干的姿势温和一点就行了。 |
 |
16
ooh 2016-10-16 10:50:23 +08:00
扔杯子
|
 |
17
geew 2016-10-16 10:53:30 +08:00
同意五楼
反正有人愿意来背锅求之不得呢 爱改就改呗 谁改谁负责 |
 |
18
davidlau 2016-10-16 11:04:39 +08:00
code review 很重要
|
 |
19
xw 2016-10-16 11:04:40 +08:00
（吐槽下--）
之前生产环境代码都是 sftp 手动传上去，然后上头领导（以前是开发，现在不写了）经常私自改动，而且也不把改动提交到 svn 或 git （只开发环境用了版本控制，生产环境一直是手动传上去），也不告知几个开发人员，经常出现莫名其妙情况，也是无语了、、、 |
 |
20
goodryb 2016-10-16 11:09:18 +08:00
这种情况难道不是运维去叼他么，线上的东西开发怎么会有权限动
|
 |
21
zonghua 2016-10-16 11:10:55 +08:00
DevOps
|
 |
22
youxiachai 2016-10-16 11:14:23 +08:00
好同事啊......
出了..问题..负责任的是他啊.... |
 |
23
k9982874 2016-10-16 11:29:02 +08:00
他是你的上司，他有这个权限，他改你的代码自然有他的道理。
至于跟不跟你打招呼或者给不给你说明，看你在他眼中值不值的这么做。 对于这个问题，你也不用很纠结，毕竟是工作，做好本职工作，这是本职工作外的事了，不要让这些事乱了心情。 他改出问题他背锅就是。你的工资奖金不能少。 |
 |
24
forestyuan 2016-10-16 12:00:17 +08:00
这个架构师很不专业
|
 |
25
bk201 2016-10-16 15:17:23 +08:00 via iPhone
相处不长不要对别人指手画脚，好好沟通都是能沟通的，能坐上架构师自然不会太差.
|
 |
26
SmiteChow 2016-10-16 16:02:08 +08:00
沟通出现问题就改沟通方式方法，同时用自动化工具来保证以后不会出现这种 situation
|
 |
27
jhaohai 2016-10-16 18:29:16 +08:00 via iPhone
谁改谁负责，出了问题让他背锅就行了
|
 |
28
millson 2016-10-16 19:13:48 +08:00
php ，曾经也有这样的问题，现在线上都是 ZendGuard 加密的代码 :lol
|
 |
29
neoblackcap 2016-10-16 19:18:16 +08:00
你们撕逼直接在生产环节撕逼也是厉害，我们撕逼最多也就是在 code review 的环节
|
 |
30
chunqiuyiyu 2016-10-16 20:20:17 +08:00
谁修改，谁负责。
|
 |
31
iyaozhen 2016-10-16 20:26:20 +08:00 via Android
也是 6 啊，居然能直接修过线上代码。
楼主不要激动，这是公司的代码，有责任心很好，但这个时候不要强出头， QA 找过来你就实话实说，你们架构师胆子真大，这情况 QA 还不肛死他。 |
 |
32
chilaoqi 2016-10-16 21:12:45 +08:00
有人想抗雷，直接禅让吧 ：）
|
 |
33
Niphor 2016-10-16 23:34:25 +08:00
外包 /小公司吧... 居然有权限该线上代码
谁改谁负责 很简单的 没楼主什么事 |
 |
34
zjuster 2016-10-17 10:13:04 +08:00
1 、这是公司的代码。
2 、谁改谁负责。 你就看看是不是这样改比自己写的要有效率，可别再改回来哦。 |
 |
35
PEP4JASON 2016-10-17 14:36:56 +08:00
能动手的尽量动手
|
Select Language