[GitHub] [incubator-brpc] qicosmos opened a new issue #1223: 能否在server中增加SetMinLogLevel的接口以及禁用日志的宏开关

2020-08-27 Thread GitBox
qicosmos opened a new issue #1223: URL: https://github.com/apache/incubator-brpc/issues/1223 现在logging里面有一个SetMinLogLevel接口,能否在server里面增加一个代理接口,让用户可以通过server去设置日志级别。 另外能否增加一个宏开关用来控制是否输出日志?因为测试的时候发现日志刷得很频繁,有时候不希望输出日志。 This

Error while running github feature from .asf.yaml in incubator-brpc!

2020-08-27 Thread Apache Infrastructure
An error occurred while running github feature in .asf.yaml!: .asf.yaml: Invalid GitHub label 'RPC' - must be lowercase alphanumerical and <= 35 characters! - To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org For addition

Error while running github feature from .asf.yaml in incubator-brpc!

2020-08-27 Thread Apache Infrastructure
An error occurred while running github feature in .asf.yaml!: .asf.yaml: Invalid GitHub label 'RPC' - must be lowercase alphanumerical and <= 35 characters! - To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org For addition

[incubator-brpc] branch master updated: add .asf.yaml to set github about info

2020-08-27 Thread jamesge
This is an automated email from the ASF dual-hosted git repository. jamesge pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-brpc.git The following commit(s) were added to refs/heads/master by this push: new 414d712 add .asf.yaml to set github ab

[incubator-brpc] branch master updated: add .asf.yaml to set github about info

2020-08-27 Thread jamesge
This is an automated email from the ASF dual-hosted git repository. jamesge pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-brpc.git The following commit(s) were added to refs/heads/master by this push: new 414d712 add .asf.yaml to set github ab

[GitHub] [incubator-brpc] jamesge merged pull request #1222: add .asf.yaml to set github about info

2020-08-27 Thread GitBox
jamesge merged pull request #1222: URL: https://github.com/apache/incubator-brpc/pull/1222 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [incubator-brpc] tanzhongyi003 opened a new pull request #1222: add .asf.yaml to set github about info

2020-08-27 Thread GitBox
tanzhongyi003 opened a new pull request #1222: URL: https://github.com/apache/incubator-brpc/pull/1222 update brpc info on github about page This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-brpc] lacey1102 commented on pull request #1157: fix socket conn stat err and memory leak when discard socket

2020-08-27 Thread GitBox
lacey1102 commented on pull request #1157: URL: https://github.com/apache/incubator-brpc/pull/1157#issuecomment-682295379 > hi,主要是指_numinflight这个计数不对么? 主要是要做sharepart->RemoveRefManually()。未降低引用个数,会导致这部分内存无法释放。

[GitHub] [incubator-brpc] feifeiiong commented on pull request #536: Multiple connection

2020-08-27 Thread GitBox
feifeiiong commented on pull request #536: URL: https://github.com/apache/incubator-brpc/pull/536#issuecomment-682294119 好的,感谢,也想知道当初给multi下每个socket 共享main socket sharepart的原因,看了代码暂时没法理解,谢谢! This is an automated message from

[GitHub] [incubator-brpc] cdjingit commented on pull request #536: Multiple connection

2020-08-27 Thread GitBox
cdjingit commented on pull request #536: URL: https://github.com/apache/incubator-brpc/pull/536#issuecomment-682292810 > 添加ShareStatsForNewSocket的目的是?目前看该函数的引入可能造成bug,具体表现是: > 在multi模式下,名字服务更新ip port, 删除旧main socket时,由于multi下socket创建时调用ShareStatsForNewSocket导致share part引用计数无法归零,无法删除main

[GitHub] [incubator-brpc] songlong commented on issue #1199: 动态运行错误RegisterValidateFunction

2020-08-27 Thread GitBox
songlong commented on issue #1199: URL: https://github.com/apache/incubator-brpc/issues/1199#issuecomment-682282946 不能,升级了pb2.5,gflags2.2.1, 通过preload加载,能避免flags重新reload注册问题 This is an automated message from the Apache Git Se

[GitHub] [incubator-brpc] gydong commented on issue #1199: 动态运行错误RegisterValidateFunction

2020-08-27 Thread GitBox
gydong commented on issue #1199: URL: https://github.com/apache/incubator-brpc/issues/1199#issuecomment-682270904 是说升级到protobuf2.5 后即可解决您的问题? This is an automated message from the Apache Git Service. To respond to the message

[GitHub] [incubator-brpc] gydong commented on issue #1217: 是否支持Curl通过Http发送大量数据至brpc server?

2020-08-27 Thread GitBox
gydong commented on issue #1217: URL: https://github.com/apache/incubator-brpc/issues/1217#issuecomment-682267046 server支持接收大body,但不支持持续接收(边接收边处理),所以只要内存足够用,是可以传输大body的 This is an automated message from the Apache Git Service

[GitHub] [incubator-brpc] songlong commented on issue #1199: 动态运行错误RegisterValidateFunction

2020-08-27 Thread GitBox
songlong commented on issue #1199: URL: https://github.com/apache/incubator-brpc/issues/1199#issuecomment-682022520 修改cmakefile文件 set(Protobuf_INCLUDE_DIR /usr/include/protobuf25) set(Protobuf_LIBRARY /usr/lib64/protobuf25) include_directories(${Protobuf_INCLUDE_DIR}) include

[GitHub] [incubator-brpc] zyearn commented on a change in pull request #1220: replace HealthCheckThread to StartHealthCheck in docs/comments #1209

2020-08-27 Thread GitBox
zyearn commented on a change in pull request #1220: URL: https://github.com/apache/incubator-brpc/pull/1220#discussion_r478481513 ## File path: src/brpc/socket.h ## @@ -333,7 +333,7 @@ friend class policy::H2GlobalStreamCreator; // Mark this Socket or the Socket associat

[GitHub] [incubator-brpc] zyearn closed issue #1216: 支持pb2.5以上编译太费尽了,很多问题

2020-08-27 Thread GitBox
zyearn closed issue #1216: URL: https://github.com/apache/incubator-brpc/issues/1216 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[GitHub] [incubator-brpc] zyearn commented on pull request #1219: replace HealthCheckThread to StartHealthCheck in docs/comments

2020-08-27 Thread GitBox
zyearn commented on pull request #1219: URL: https://github.com/apache/incubator-brpc/pull/1219#issuecomment-681989084 多谢领取这个[issue](https://github.com/apache/incubator-brpc/issues/1209)。 改的时候需要注意一点:不仅仅只是把HealthCheckThread替换成StartHealthCheck。 由于原来的HealthCheckThread是个bthread,所以在原来的

[GitHub] [incubator-brpc] songlong commented on issue #1216: 支持pb2.5以上编译太费尽了,很多问题

2020-08-27 Thread GitBox
songlong commented on issue #1216: URL: https://github.com/apache/incubator-brpc/issues/1216#issuecomment-681985150 手动处理了一下config.sh,己安装成功 This is an automated message from the Apache Git Service. To respond to the message, p

[incubator-brpc] branch master updated: fix typo of 'RPC'

2020-08-27 Thread jiashunzhu
This is an automated email from the ASF dual-hosted git repository. jiashunzhu pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-brpc.git The following commit(s) were added to refs/heads/master by this push: new 0deb4f8 fix typo of 'RPC' new

[GitHub] [incubator-brpc] zyearn merged pull request #1221: fix typo of 'RPC'

2020-08-27 Thread GitBox
zyearn merged pull request #1221: URL: https://github.com/apache/incubator-brpc/pull/1221 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [incubator-brpc] zyearn commented on pull request #1221: fix typo of 'RPC'

2020-08-27 Thread GitBox
zyearn commented on pull request #1221: URL: https://github.com/apache/incubator-brpc/pull/1221#issuecomment-681979718 Thanks for the contribution. This is an automated message from the Apache Git Service. To respond to the m

[GitHub] [incubator-brpc] zyearn commented on issue #1216: 支持pb2.5以上编译太费尽了,很多问题

2020-08-27 Thread GitBox
zyearn commented on issue #1216: URL: https://github.com/apache/incubator-brpc/issues/1216#issuecomment-681979409 这是因为环境的protoc和protobuf runtime的版本对应不上导致的,一般是由于装了多个版本的protobuf This is an automated message from the Apache Git

[GitHub] [incubator-brpc] feifeiiong commented on pull request #536: Multiple connection

2020-08-27 Thread GitBox
feifeiiong commented on pull request #536: URL: https://github.com/apache/incubator-brpc/pull/536#issuecomment-681937900 添加ShareStatsForNewSocket的目的是?目前看该函数的引入可能造成bug,具体表现是: 在multi模式下,名字服务更新ip port, 删除旧main socket时,由于multi下socket创建时调用ShareStatsForNewSocket导致share part引用计数无法归零,无法删除main s

[GitHub] [incubator-brpc] lsjbd opened a new pull request #1221: fix typo of 'RPC'

2020-08-27 Thread GitBox
lsjbd opened a new pull request #1221: URL: https://github.com/apache/incubator-brpc/pull/1221 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abov