Re: [PR] fix bug: switch between stream_rpc and baidu_std (brpc)

2024-07-06 Thread via GitHub
wwbmmm commented on PR #2678: URL: https://github.com/apache/brpc/pull/2678#issuecomment-2211691354 LGTM -- 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-m

Re: [I] server中brpc::StreamWait卡在了thread::TaskGroup::sched_to (brpc)

2024-07-06 Thread via GitHub
chenBright commented on issue #2680: URL: https://github.com/apache/brpc/issues/2680#issuecomment-2211784543 StreamWrite没有返回网络错误,一直返回EAGAIN的话,应该是client挂了,但是server并没有感知到tcp连接断开,发送的数据没有收到client的ack,随后写不进去内核缓冲区后,就一直返回EAGAIN。 -- This is an automated message from the Apache Git Service. To res

Re: [I] backup request可以支持下游错误率高于一定阈值时不生效吗? (brpc)

2024-07-06 Thread via GitHub
chenBright commented on issue #2668: URL: https://github.com/apache/brpc/issues/2668#issuecomment-2211789986 容灾方面是不是可以全面一点? (周期或者滑动窗口)统计请求的成功率,根据成功率可以做以下策略: 1. lb选实例时,减少访问可用性弱的实例; 2. 减少重试(包括backup request)。 3. 其他,待补充。 -- This is an automated message from the Apache Git Service.

[I] 【rdma问题请教】Server收到Client通信后直接core (brpc)

2024-07-06 Thread via GitHub
SimonCqk opened a new issue, #2683: URL: https://github.com/apache/brpc/issues/2683 **Describe the bug (描述bug)** 在我的应用场景中,集成了brpc的多种操作,通过request_attachment或response_attachment收发,包含append、append_with_user_data、swap等多个动作,现尝试启用rdma,但还没有端到端跑通(brpc内置的rdma example已经跑通) **To Reproduce (复

Re: [I] Questions about idle_timeout_sec of server (brpc)

2024-07-06 Thread via GitHub
chenBright commented on issue #2681: URL: https://github.com/apache/brpc/issues/2681#issuecomment-2211811304 In this case, the connection will be closed until all rpc requests are completed. Idle timeout mechanism only release the additional reference. -- This is an automated message from

[PR] Fix malloc dead lock cause by contention profiler (brpc)

2024-07-06 Thread via GitHub
chenBright opened a new pull request, #2684: URL: https://github.com/apache/brpc/pull/2684 ### What problem does this PR solve? Issue Number: Problem Summary: ### What is changed and the side effects? Changed: Side effects: - Performance effects(性能影响):

Re: [PR] add bzlmod support (brpc)

2024-07-06 Thread via GitHub
oathdruid commented on PR #2670: URL: https://github.com/apache/brpc/pull/2670#issuecomment-2212296577 any suggestion? -- 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 un

Re: [I] server中brpc::StreamWait卡在了thread::TaskGroup::sched_to (brpc)

2024-07-06 Thread via GitHub
GreateCode commented on issue #2680: URL: https://github.com/apache/brpc/issues/2680#issuecomment-2212300609 有道理,这样的话,一定时期内都是EAGAIN,while里就判定连接断开,然后return结束。 但是server的连接资源怎么释放?如果sever配置了ServerOptions.idle_timeout_sec=100, 那么超过100s后,server会把该连接释放吗? -- This is an automated message fr

Re: [I] gdb_bthread_stack.py not work. (brpc)

2024-07-06 Thread via GitHub
Huixxi commented on issue #2674: URL: https://github.com/apache/brpc/issues/2674#issuecomment-2212310641 编译的时候加了-g么? -- 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 unsu

(brpc) 01/01: Merge pull request #2678 from howarle/fix_stream_rpc

2024-07-06 Thread huixxi
This is an automated email from the ASF dual-hosted git repository. huixxi pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/brpc.git commit fdc60bab6816cc7a6268fd395c954d814ca406eb Merge: 81ae605d 9c387c0e Author: HU AuthorDate: Sun Jul 7 11:51:26 2024 +0800

Re: [PR] fix bug: switch between stream_rpc and baidu_std (brpc)

2024-07-06 Thread via GitHub
Huixxi merged PR #2678: URL: https://github.com/apache/brpc/pull/2678 -- 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 F

Re: [PR] Add debug log for Socket health check (brpc)

2024-07-06 Thread via GitHub
Huixxi commented on PR #2673: URL: https://github.com/apache/brpc/pull/2673#issuecomment-2212311550 LGTM -- 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-m

Re: [PR] Add debug log for Socket health check (brpc)

2024-07-06 Thread via GitHub
Huixxi merged PR #2673: URL: https://github.com/apache/brpc/pull/2673 -- 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 F

(brpc) branch master updated (81ae605d -> fdc60bab)

2024-07-06 Thread huixxi
This is an automated email from the ASF dual-hosted git repository. huixxi pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/brpc.git from 81ae605d Merge pull request #2610 from yanglimingcn/bugfix/remove_old_rpcz_directory add 9c387c0e fix bug: switch

(brpc) branch master updated (fdc60bab -> 7557004e)

2024-07-06 Thread huixxi
This is an automated email from the ASF dual-hosted git repository. huixxi pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/brpc.git from fdc60bab Merge pull request #2678 from howarle/fix_stream_rpc add 931ef3eb Add debug log for Socket health check

(brpc) 01/01: Merge pull request #2673 from chenBright/debug_hc

2024-07-06 Thread huixxi
This is an automated email from the ASF dual-hosted git repository. huixxi pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/brpc.git commit 7557004e7d6c7eac7b8b418ce134152f0d3e2486 Merge: fdc60bab 931ef3eb Author: HU AuthorDate: Sun Jul 7 11:52:17 2024 +0800

Re: [PR] Fix uninitialized local side of Socket (brpc)

2024-07-06 Thread via GitHub
Huixxi merged PR #2672: URL: https://github.com/apache/brpc/pull/2672 -- 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 F

(brpc) 01/01: Merge pull request #2672 from chenBright/fix_local_side

2024-07-06 Thread huixxi
This is an automated email from the ASF dual-hosted git repository. huixxi pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/brpc.git commit 8b9138ac4c9c488e236bb9dd1cce8bb3eea173ce Merge: 7557004e edfc0079 Author: HU AuthorDate: Sun Jul 7 11:53:23 2024 +0800

(brpc) branch master updated (7557004e -> 8b9138ac)

2024-07-06 Thread huixxi
This is an automated email from the ASF dual-hosted git repository. huixxi pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/brpc.git from 7557004e Merge pull request #2673 from chenBright/debug_hc add edfc0079 Fix uninitialized local side of Socket

Re: [PR] Use butil::ThreadLocal to store keytable (brpc)

2024-07-06 Thread via GitHub
Huixxi commented on PR #2645: URL: https://github.com/apache/brpc/pull/2645#issuecomment-2212314895 请问下按照这样做了设计之后,有具体的性能测试报告么?能够直观的显示这种改造在特定场景下带来的性能提升 -- 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

Re: [I] Bvar会丢失数据 (brpc)

2024-07-06 Thread via GitHub
Huixxi commented on issue #2682: URL: https://github.com/apache/brpc/issues/2682#issuecomment-2212317073 @serverglen 这个帅哥有空看下 -- 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 comm

Re: [I] brpc Stream导致普通RPC无法解析 (brpc)

2024-07-06 Thread via GitHub
Huixxi closed issue #2677: brpc Stream导致普通RPC无法解析 URL: https://github.com/apache/brpc/issues/2677 -- 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-un

Re: [I] variable.cpp error: ‘StringFlagDestructor’ does not name a type (brpc)

2024-07-06 Thread via GitHub
Huixxi closed issue #2679: variable.cpp error: ‘StringFlagDestructor’ does not name a type URL: https://github.com/apache/brpc/issues/2679 -- 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 spec