[GitHub] [incubator-brpc] mistyliu commented on issue #954: 支持腾讯开源rpc框架tars使用的tars协议

2019-12-09 Thread GitBox
mistyliu commented on issue #954: 支持腾讯开源rpc框架tars使用的tars协议 URL: https://github.com/apache/incubator-brpc/issues/954#issuecomment-563741404 仅验证使用的话,可以参考目前brpc中支持thrift的模式,通过变通的方式引入tars协议。 This is an automated message from the

[VOTE] Release Apache brpc(incubating) 0.9.7-rc02

2019-12-09 Thread tan zhongyi
Hi, guys, 0.9.7-rc02 is ready for vote, here it is. I am pleased to be calling this vote for the release of apache brpc(incubating) 0.9.7-rc02. The source code can be found at: https://dist.apache.org/repos/dist/dev/incubator/brpc/0.9.7-rc02/incubator-brpc-0.9.7-rc02.tar.gz The release

[GitHub] [incubator-brpc] jamesge commented on issue #990: Support server protocol parsing order without changing ProtocolType(options.proto)

2019-12-09 Thread GitBox
jamesge commented on issue #990: Support server protocol parsing order without changing ProtocolType(options.proto) URL: https://github.com/apache/incubator-brpc/issues/990#issuecomment-563229628 the number for ordering should be passed to RegisterProtocol

[GitHub] [incubator-brpc] zyearn opened a new issue #990: Support server protocol parsing order without changing ProtocolType(options.proto)

2019-12-09 Thread GitBox
zyearn opened a new issue #990: Support server protocol parsing order without changing ProtocolType(options.proto) URL: https://github.com/apache/incubator-brpc/issues/990 One possible plan: define an order for every protocol, ``` TypeOrder[] = { {PROTOCOL_BAIDU_STD , 100} ,

[GitHub] [incubator-brpc] jamesge commented on a change in pull request #972: Redis server protocol

2019-12-09 Thread GitBox
jamesge commented on a change in pull request #972: Redis server protocol URL: https://github.com/apache/incubator-brpc/pull/972#discussion_r355398301 ## File path: src/brpc/server.cpp ## @@ -1588,7 +1589,8 @@ void Server::GenerateVersionIfNeeded() { if

[GitHub] [incubator-brpc] jamesge commented on a change in pull request #972: Redis server protocol

2019-12-09 Thread GitBox
jamesge commented on a change in pull request #972: Redis server protocol URL: https://github.com/apache/incubator-brpc/pull/972#discussion_r355392122 ## File path: src/brpc/policy/redis_protocol.cpp ## @@ -52,62 +58,249 @@ struct InputResponse : public InputMessageBase {

[GitHub] [incubator-brpc] jamesge commented on a change in pull request #972: Redis server protocol

2019-12-09 Thread GitBox
jamesge commented on a change in pull request #972: Redis server protocol URL: https://github.com/apache/incubator-brpc/pull/972#discussion_r355397799 ## File path: src/brpc/redis_reply.h ## @@ -160,6 +212,87 @@ inline int64_t RedisReply::integer() const { return 0;

[GitHub] [incubator-brpc] jamesge commented on a change in pull request #972: Redis server protocol

2019-12-09 Thread GitBox
jamesge commented on a change in pull request #972: Redis server protocol URL: https://github.com/apache/incubator-brpc/pull/972#discussion_r355393947 ## File path: src/brpc/policy/redis_protocol.h ## @@ -33,6 +33,11 @@ ParseResult ParseRedisMessage(butil::IOBuf* source,

[GitHub] [incubator-brpc] jamesge commented on a change in pull request #972: Redis server protocol

2019-12-09 Thread GitBox
jamesge commented on a change in pull request #972: Redis server protocol URL: https://github.com/apache/incubator-brpc/pull/972#discussion_r355397430 ## File path: src/brpc/redis_reply.h ## @@ -160,6 +212,87 @@ inline int64_t RedisReply::integer() const { return 0;

[GitHub] [incubator-brpc] jamesge commented on a change in pull request #972: Redis server protocol

2019-12-09 Thread GitBox
jamesge commented on a change in pull request #972: Redis server protocol URL: https://github.com/apache/incubator-brpc/pull/972#discussion_r355397364 ## File path: src/brpc/redis_reply.h ## @@ -160,6 +212,87 @@ inline int64_t RedisReply::integer() const { return 0;

[GitHub] [incubator-brpc] zyearn opened a new issue #989: Disable some specific protocol when it is not enabled in server

2019-12-09 Thread GitBox
zyearn opened a new issue #989: Disable some specific protocol when it is not enabled in server URL: https://github.com/apache/incubator-brpc/issues/989 If server doesn't support rtmp, redis, etc., we want to skip the request parsing process using these protocols.

Re: [incubator-brpc] annotated tag 0.9.7-rc02 updated (4cb1334 -> c8b1709)

2019-12-09 Thread tan zhongyi
Cool, I will make a release vote tonight. 在 2019/12/8 下午8:40,“jiashun...@apache.org” 写入: This is an automated email from the ASF dual-hosted git repository. jiashunzhu pushed a change to annotated tag 0.9.7-rc02 in repository