china521
V2EX  ›  问与答

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

  •  
  •   china521 · Dec 28, 2015 · 2898 views
    This topic created in 3818 days ago, the information mentioned may be changed or developed.

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

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

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

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