[GitHub] [incubator-brpc] wwbmmm commented on issue #1634: Why bRPC allows several bthreads keep running until process quit and without join them?

2021-12-14 Thread GitBox


wwbmmm commented on issue #1634:
URL: https://github.com/apache/incubator-brpc/issues/1634#issuecomment-993260959


   可以用中文讨论 @Huixxi 


-- 
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 the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



-
To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org
For additional commands, e-mail: dev-h...@brpc.apache.org



[GitHub] [incubator-brpc] mahongweichina opened a new pull request #1635: Add brpc logo

2021-12-14 Thread GitBox


mahongweichina opened a new pull request #1635:
URL: https://github.com/apache/incubator-brpc/pull/1635


   Supply a vector graphic logo of brpc(svg)


-- 
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 the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



-
To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org
For additional commands, e-mail: dev-h...@brpc.apache.org



[incubator-brpc] branch master updated: Add brpc logo

2021-12-14 Thread jerrytan
This is an automated email from the ASF dual-hosted git repository.

jerrytan 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 b7e1fa0  Add brpc logo
 new cff40a9  Merge pull request #1635 from mahongweichina/master
b7e1fa0 is described below

commit b7e1fa0d85ac1379f97f89795885063b3fe9f78d
Author: Clement Ma 
AuthorDate: Tue Dec 14 16:15:10 2021 +0800

Add brpc logo

Supply a vector graphic logo of brpc(svg)
---
 community/logo-bRPC.svg | 1 +
 1 file changed, 1 insertion(+)

diff --git a/community/logo-bRPC.svg b/community/logo-bRPC.svg
new file mode 100644
index 000..bd7c97d
--- /dev/null
+++ b/community/logo-bRPC.svg
@@ -0,0 +1 @@
+http://www.w3.org/2000/svg"; viewBox="0 0 1440 
1440">.cls-1{fill:#fff;opacity:0;}.cls-2{fill:#7f7f7f;}.cls-3{fill:#38b8ff;}

[GitHub] [incubator-brpc] tanzhongyi003 merged pull request #1635: Add brpc logo

2021-12-14 Thread GitBox


tanzhongyi003 merged pull request #1635:
URL: https://github.com/apache/incubator-brpc/pull/1635


   


-- 
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 the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



-
To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org
For additional commands, e-mail: dev-h...@brpc.apache.org



[GitHub] [incubator-brpc] wengduo commented on issue #1628: cpu profile显示不出来

2021-12-14 Thread GitBox


wengduo commented on issue #1628:
URL: https://github.com/apache/incubator-brpc/issues/1628#issuecomment-993350799


   原因找到了,fork后,系统oom了,我这个服务内存占比比较高~
   
   发自我的iPhone
   
   > 在 2021年12月14日,15:11,caidj ***@***.***> 写道:
   > 
   > 
   > 重启的原因是什么?段错误?
   > 
   > —
   > You are receiving this because you authored the thread.
   > Reply to this email directly, view it on GitHub, or unsubscribe.
   > Triage notifications on the go with GitHub Mobile for iOS or Android. 
   


-- 
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 the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



-
To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org
For additional commands, e-mail: dev-h...@brpc.apache.org



[GitHub] [incubator-brpc] wengduo commented on issue #1629: V99 1207 18:47:09.863345 1326 /home/xiaoju/src/brpc/src/brpc/policy/thrift_protocol.cpp:379] version=795373312 doesn't match THRIFT_VERSION

2021-12-14 Thread GitBox


wengduo commented on issue #1629:
URL: https://github.com/apache/incubator-brpc/issues/1629#issuecomment-993352980


   我这边使用的是thrift0.9.3
   
   发自我的iPhone
   
   > 在 2021年12月14日,15:09,caidj ***@***.***> 写道:
   > 
   > 
   > Would you please use thrift-0.9.2 for the moment? It works with 
thrift-0.9.2.
   > @kenshinxf Could you help to check this problem? Why is the thrift version 
hard coded?
   > 
   > —
   > You are receiving this because you authored the thread.
   > Reply to this email directly, view it on GitHub, or unsubscribe.
   > Triage notifications on the go with GitHub Mobile for iOS or Android. 
   


-- 
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 the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



-
To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org
For additional commands, e-mail: dev-h...@brpc.apache.org



[GitHub] [incubator-brpc] cdjingit closed issue #1628: cpu profile显示不出来

2021-12-14 Thread GitBox


cdjingit closed issue #1628:
URL: https://github.com/apache/incubator-brpc/issues/1628


   


-- 
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 the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



-
To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org
For additional commands, e-mail: dev-h...@brpc.apache.org



[GitHub] [incubator-brpc] libratiger closed issue #875: update the benchmark

2021-12-14 Thread GitBox


libratiger closed issue #875:
URL: https://github.com/apache/incubator-brpc/issues/875


   


-- 
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 the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



-
To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org
For additional commands, e-mail: dev-h...@brpc.apache.org



[GitHub] [incubator-brpc] libratiger closed issue #889: update the protos file up to proto3

2021-12-14 Thread GitBox


libratiger closed issue #889:
URL: https://github.com/apache/incubator-brpc/issues/889


   


-- 
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 the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



-
To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org
For additional commands, e-mail: dev-h...@brpc.apache.org



[incubator-brpc] branch master updated (cff40a9 -> bcf7fd8)

2021-12-14 Thread jamesge
This is an automated email from the ASF dual-hosted git repository.

jamesge pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-brpc.git.


from cff40a9  Merge pull request #1635 from mahongweichina/master
 new 91a48f4  Update gifs in docs to remove hostname and ip
 new 87bc97a  Update rpc_press doc to remove specific hostname
 new bcf7fd8  Merge pull request #1636 from wwbmmm/update_pics

The 2374 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 docs/cn/rpc_press.md|   2 +-
 docs/images/heap_profiler_3.gif | Bin 362001 -> 2247904 bytes
 docs/images/vars_1.gif  | Bin 202182 -> 1117263 bytes
 docs/images/vars_2.gif  | Bin 305617 -> 1632482 bytes
 4 files changed, 1 insertion(+), 1 deletion(-)

-
To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org
For additional commands, e-mail: dev-h...@brpc.apache.org



[GitHub] [incubator-brpc] jamesge merged pull request #1636: Update gifs in docs to remove hostname and ip

2021-12-14 Thread GitBox


jamesge merged pull request #1636:
URL: https://github.com/apache/incubator-brpc/pull/1636


   


-- 
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 the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



-
To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org
For additional commands, e-mail: dev-h...@brpc.apache.org



[incubator-brpc] branch master updated: Fix typo

2021-12-14 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 3a6a0b9  Fix typo
 new 39aa541  Merge pull request #1613 from egolearner/patch-1
3a6a0b9 is described below

commit 3a6a0b9369ca9252f44c41b3368f0b1229f9d70c
Author: egolearner <45122959+egolear...@users.noreply.github.com>
AuthorDate: Thu Nov 25 10:56:15 2021 +0800

Fix typo
---
 docs/cn/client.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/docs/cn/client.md b/docs/cn/client.md
index 566ebf1..8c52e23 100755
--- a/docs/cn/client.md
+++ b/docs/cn/client.md
@@ -755,7 +755,7 @@ http/h2 body的压缩方法见[client压缩request body](http_client.md#压缩re
 
 支持的压缩方法有:
 
-- brpc::CompressTypeSnappy : 
[snanpy压缩](http://google.github.io/snappy/),压缩和解压显著快于其他压缩方法,但压缩率最低。
+- brpc::CompressTypeSnappy : 
[snappy压缩](http://google.github.io/snappy/),压缩和解压显著快于其他压缩方法,但压缩率最低。
 - brpc::CompressTypeGzip : 
[gzip压缩](http://en.wikipedia.org/wiki/Gzip),显著慢于snappy,但压缩率高
 - brpc::CompressTypeZlib : 
[zlib压缩](http://en.wikipedia.org/wiki/Zlib),比gzip快10%~20%,压缩率略好于gzip,但速度仍明显慢于snappy。
 

-
To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org
For additional commands, e-mail: dev-h...@brpc.apache.org



[GitHub] [incubator-brpc] jamesge merged pull request #1623: Exclude mesalink/openssl/ssl.h when find openssl

2021-12-14 Thread GitBox


jamesge merged pull request #1623:
URL: https://github.com/apache/incubator-brpc/pull/1623


   


-- 
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 the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



-
To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org
For additional commands, e-mail: dev-h...@brpc.apache.org



[GitHub] [incubator-brpc] jamesge closed issue #1622: [ci: compile-with-make] wrong header path for openssl

2021-12-14 Thread GitBox


jamesge closed issue #1622:
URL: https://github.com/apache/incubator-brpc/issues/1622


   


-- 
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 the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



-
To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org
For additional commands, e-mail: dev-h...@brpc.apache.org



[incubator-brpc] branch master updated: Exclude mesalink/openssl/ssl.h when find openssl

2021-12-14 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 a7525c2  Exclude mesalink/openssl/ssl.h when find openssl
 new d3452d7  Merge pull request #1623 from wasphin/feature/exclude-mesalink
a7525c2 is described below

commit a7525c20ed7bbf4d2143e6d2c733a99da794cfe0
Author: wxf 
AuthorDate: Thu Dec 2 09:37:17 2021 +0800

Exclude mesalink/openssl/ssl.h when find openssl

Resolve #1622
---
 config_brpc.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/config_brpc.sh b/config_brpc.sh
index 1720031..f52911d 100755
--- a/config_brpc.sh
+++ b/config_brpc.sh
@@ -161,7 +161,7 @@ else
 OPENSSL_LIB=$(find_dir_of_lib ssl)
 # Inconvenient to check these headers in baidu-internal
 #PTHREAD_HDR=$(find_dir_of_header_or_die pthread.h)
-OPENSSL_HDR=$(find_dir_of_header_or_die openssl/ssl.h)
+OPENSSL_HDR=$(find_dir_of_header_or_die openssl/ssl.h 
mesalink/openssl/ssl.h)
 fi
 
 if [ $WITH_MESALINK != 0 ]; then

-
To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org
For additional commands, e-mail: dev-h...@brpc.apache.org



[GitHub] [incubator-brpc] jamesge merged pull request #1613: Fix typo

2021-12-14 Thread GitBox


jamesge merged pull request #1613:
URL: https://github.com/apache/incubator-brpc/pull/1613


   


-- 
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 the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



-
To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org
For additional commands, e-mail: dev-h...@brpc.apache.org



[GitHub] [incubator-brpc] jamesge commented on a change in pull request #1571: Support wr/wrr policy degradation

2021-12-14 Thread GitBox


jamesge commented on a change in pull request #1571:
URL: https://github.com/apache/incubator-brpc/pull/1571#discussion_r769285373



##
File path: src/brpc/load_balancer.cpp
##
@@ -24,6 +24,7 @@
 namespace brpc {
 
 DEFINE_bool(show_lb_in_vars, false, "Describe LoadBalancers in vars");
+DEFINE_int32(default_weight_of_wlb, 0, "Default weight of Weighted 
LoadBalancer");

Review comment:
   @Huixxi 这里需要补充下评论或细化下描述,否则光看选项名称,并不太清楚作用是什么。




-- 
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 the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



-
To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org
For additional commands, e-mail: dev-h...@brpc.apache.org



[GitHub] [incubator-brpc] moesakura opened a new issue #1638: 请教一下 系统错误码300,305,700大概是什么问题?

2021-12-14 Thread GitBox


moesakura opened a new issue #1638:
URL: https://github.com/apache/incubator-brpc/issues/1638


   在 服务的 ip:port/vars中
   
   请求量上升后(并发也上升) qpm 5k
   error_code_300 4k
   error_code_700 55k
   
   之前还遇到过305的错误 但是没有当时的场景信息了。
   请问一下是什么问题?
   


-- 
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 the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



-
To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org
For additional commands, e-mail: dev-h...@brpc.apache.org



[incubator-brpc] branch master updated: Replace baidu internal wiki address

2021-12-14 Thread zychen
This is an automated email from the ASF dual-hosted git repository.

zychen 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 67fd03b  Replace baidu internal wiki address
 new 8156a14  Merge pull request #1637 from wwbmmm/replace_baidu_wiki
67fd03b is described below

commit 67fd03b08ae75b0d3c211f983b07c2c616360c2a
Author: wwbmmm 
AuthorDate: Wed Dec 15 14:21:45 2021 +0800

Replace baidu internal wiki address
---
 docs/cn/client.md  | 2 +-
 src/brpc/builtin/pprof_service.cpp | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/docs/cn/client.md b/docs/cn/client.md
index 0add4d3..6b470fc 100755
--- a/docs/cn/client.md
+++ b/docs/cn/client.md
@@ -521,7 +521,7 @@ Controller的特点:
 
 ## 线程数
 
-和大部分的RPC框架不同,brpc中并没有独立的Client线程池。所有Channel和Server通过[bthread](http://wiki.baidu.com/display/RPC/bthread)共享相同的线程池.
 如果你的程序同样使用了brpc的server, 仅仅需要设置Server的线程数。 
或者可以通过[gflags](http://wiki.baidu.com/display/RPC/flags)设置[-bthread_concurrency](http://brpc.baidu.com:8765/flags/bthread_concurrency)来设置全局的线程数.
+和大部分的RPC框架不同,brpc中并没有独立的Client线程池。所有Channel和Server通过[bthread](bthread.md)共享相同的线程池.
 如果你的程序同样使用了brpc的server, 仅仅需要设置Server的线程数。 
或者可以通过[gflags](flags.md)设置[-bthread_concurrency](http://brpc.baidu.com:8765/flags/bthread_concurrency)来设置全局的线程数.
 
 ## 超时
 
diff --git a/src/brpc/builtin/pprof_service.cpp 
b/src/brpc/builtin/pprof_service.cpp
index abc2b8e..8aa7834 100644
--- a/src/brpc/builtin/pprof_service.cpp
+++ b/src/brpc/builtin/pprof_service.cpp
@@ -220,7 +220,7 @@ void PProfService::heap(
 extra_desc = " (no TCMALLOC_SAMPLE_PARAMETER in env)";
 }
 cntl->SetFailed(ENOMETHOD, "Heap profiler is not enabled%s,"
-"check out http://wiki.baidu.com/display/RPC";,
+"check out 
https://github.com/apache/incubator-brpc/blob/master/docs/cn/heap_profiler.md";,
 extra_desc);
 return;
 }

-
To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org
For additional commands, e-mail: dev-h...@brpc.apache.org



[GitHub] [incubator-brpc] chenzhangyi merged pull request #1637: Replace baidu internal wiki address

2021-12-14 Thread GitBox


chenzhangyi merged pull request #1637:
URL: https://github.com/apache/incubator-brpc/pull/1637


   


-- 
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 the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



-
To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org
For additional commands, e-mail: dev-h...@brpc.apache.org