Re: [PR] fix batch create stream and make SetHostSocket thread safe (brpc)

2025-04-18 Thread via GitHub
chenBright merged PR #2938: URL: https://github.com/apache/brpc/pull/2938 -- 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.or

Re: [PR] fix batch create stream and make SetHostSocket thread safe (brpc)

2025-04-09 Thread via GitHub
jenrryyou commented on PR #2938: URL: https://github.com/apache/brpc/pull/2938#issuecomment-2791429800 辛苦大佬帮忙merge下 @wwbmmm -- 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.

Re: [PR] fix batch create stream and make SetHostSocket thread safe (brpc)

2025-04-06 Thread via GitHub
wwbmmm commented on PR #2938: URL: https://github.com/apache/brpc/pull/2938#issuecomment-2781844799 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] fix batch create stream and make SetHostSocket thread safe (brpc)

2025-04-06 Thread via GitHub
jenrryyou commented on code in PR #2938: URL: https://github.com/apache/brpc/pull/2938#discussion_r2030132594 ## src/brpc/stream_impl.h: ## @@ -134,6 +134,7 @@ friend struct butil::DefaultDeleter; butil::IOBuf *_pending_buf; int64_t _start_idle_timer_us; bthread_t

Re: [PR] fix batch create stream and make SetHostSocket thread safe (brpc)

2025-04-05 Thread via GitHub
wwbmmm commented on code in PR #2938: URL: https://github.com/apache/brpc/pull/2938#discussion_r2030032623 ## src/brpc/stream_impl.h: ## @@ -134,6 +134,7 @@ friend struct butil::DefaultDeleter; butil::IOBuf *_pending_buf; int64_t _start_idle_timer_us; bthread_time

[PR] fix batch create stream and make SetHostSocket thread safe (brpc)

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