老外为什么不喜欢别人提交代码, 是个例吗, 第一次 pull...

2015-12-28 19:39:24 +08:00
 china521

https://github.com/valyala/gorpc/pull/16

如上, patch 了以后,只需要几行就能搞定, 不 patch, 要几十行,客户端跟服务端都得改动好多... 而且这个 OnHandshake 功能,我感觉是所有服务端程序都应该导出的, 可以实现会话重用,
是看不上我提交的代码吗?

另外,他这个项目是真不错, 生产环境一直在用.

2523 次点击
所在节点    问与答
7 条回复
binux
2015-12-28 20:06:22 +08:00
人家已经告诉你了
「 OnHandshake functionality may be implemented with custom Listener as outlined above.」
你非要理解成不喜欢别人提交代码
china521
2015-12-28 21:30:37 +08:00
@binux 自定义 Listener 上面实现,太不优雅了。加一堆代码, 还不能重用. 哈哈, 水贴.
hei1000
2015-12-28 21:35:13 +08:00
@binux 都聊了那么多了,可不是简单的拒绝
binux
2015-12-28 21:45:38 +08:00
@china521 人家就是这么设计的呢, handshake 已经入侵到协议内部了,你重用 TCP 就好了。
jasontse
2015-12-28 21:51:01 +08:00
看了 https://github.com/valyala/gorpc/pull/2 确实觉得有点别扭
china521
2015-12-28 22:04:45 +08:00
@binux 嗯,如果按他的改的话,我一堆线上业务都得更新客户端源码.. 基本上是不可能的事.如果加入 OnHandshake 的话,就是无疼切换了. 只能是自己用了.
yeyeye
2015-12-28 23:18:17 +08:00
我就想问问,全球数十亿人,中国人只有十几亿,您一开口就是“老外”怎么怎么怎么样……貌似国人也只是一小撮吧……

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/246755

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX