V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
V2EX 提问指南
china521
V2EX  ›  问与答

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

  •  
  •   china521 · 2015-12-28 19:39:24 +08:00 · 2523 次点击
    这是一个创建于 3254 天前的主题,其中的信息可能已经有所发展或是发生改变。

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

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

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

    7 条回复    2015-12-28 23:18:17 +08:00
    binux
        1
    binux  
       2015-12-28 20:06:22 +08:00   ❤️ 1
    人家已经告诉你了
    「 OnHandshake functionality may be implemented with custom Listener as outlined above.」
    你非要理解成不喜欢别人提交代码
    china521
        2
    china521  
    OP
       2015-12-28 21:30:37 +08:00
    @binux 自定义 Listener 上面实现,太不优雅了。加一堆代码, 还不能重用. 哈哈, 水贴.
    hei1000
        3
    hei1000  
       2015-12-28 21:35:13 +08:00 via Android
    @binux 都聊了那么多了,可不是简单的拒绝
    binux
        4
    binux  
       2015-12-28 21:45:38 +08:00
    @china521 人家就是这么设计的呢, handshake 已经入侵到协议内部了,你重用 TCP 就好了。
    jasontse
        5
    jasontse  
       2015-12-28 21:51:01 +08:00 via iPad
    看了 https://github.com/valyala/gorpc/pull/2 确实觉得有点别扭
    china521
        6
    china521  
    OP
       2015-12-28 22:04:45 +08:00
    @binux 嗯,如果按他的改的话,我一堆线上业务都得更新客户端源码.. 基本上是不可能的事.如果加入 OnHandshake 的话,就是无疼切换了. 只能是自己用了.
    yeyeye
        7
    yeyeye  
       2015-12-28 23:18:17 +08:00
    我就想问问,全球数十亿人,中国人只有十几亿,您一开口就是“老外”怎么怎么怎么样……貌似国人也只是一小撮吧……
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   2415 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 23ms · UTC 00:24 · PVG 08:24 · LAX 16:24 · JFK 19:24
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.