Re: About explicitly listing all third-part licensed files

2022-10-30 Thread Wang,Weibing
I think we can use `*` to simplify the file list, that will not cause ambiguity. Like attachment `bsd.txt`. Best regards Weibing On Mon, Oct 31, 2022 at 2:38 PM Xiaofeng wrote: > > Hi Trista and the community, > > During the release of 1.3.0, Trista suggested [1] explicitly listing > all the fil

[incubator-brpc] branch master updated: Update release_en.md

2022-10-30 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/incubator-brpc.git The following commit(s) were added to refs/heads/master by this push: new 6fc88595 Update release_en.md 6fc88595 i

About explicitly listing all third-part licensed files

2022-10-30 Thread Xiaofeng
Hi Trista and the community, During the release of 1.3.0, Trista suggested [1] explicitly listing all the files in the LICENSE[2] file. After expanding the details list in .licenserc.yaml[3], about 320 files need to be listed. How to add this large file list to the LICENSE file? [1] https://list

[GitHub] [incubator-brpc-website] Huixxi merged pull request #106: discard all useless commits

2022-10-30 Thread GitBox
Huixxi merged PR #106: URL: https://github.com/apache/incubator-brpc-website/pull/106 -- 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...@br

[GitHub] [incubator-brpc] wwbmmm commented on pull request #1973: fix HTTP invalid host issue for channel not inited by http(s)

2022-10-30 Thread GitBox
wwbmmm commented on PR #1973: URL: https://github.com/apache/incubator-brpc/pull/1973#issuecomment-1296584959 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 unsubs

[incubator-brpc] branch master updated: community: adjust markdown format (#1977)

2022-10-30 Thread xiaofeng
This is an automated email from the ASF dual-hosted git repository. xiaofeng 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 8187bfcb community: adjust markdown fo

[GitHub] [incubator-brpc] wasphin merged pull request #1977: community: adjust markdown format

2022-10-30 Thread GitBox
wasphin merged PR #1977: URL: https://github.com/apache/incubator-brpc/pull/1977 -- 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.ap

[incubator-brpc] branch wasphin-patch-1 updated (95b1fbb1 -> 566355a2)

2022-10-30 Thread xiaofeng
This is an automated email from the ASF dual-hosted git repository. xiaofeng pushed a change to branch wasphin-patch-1 in repository https://gitbox.apache.org/repos/asf/incubator-brpc.git from 95b1fbb1 community: adjust markdown format add 218c3ed0 Update newcommitter_en.md add 566

[GitHub] [incubator-brpc] wasphin opened a new pull request, #1977: community: adjust markdown format

2022-10-30 Thread GitBox
wasphin opened a new pull request, #1977: URL: https://github.com/apache/incubator-brpc/pull/1977 ### What problem does this PR solve? Issue Number: Problem Summary: ### What is changed and the side effects? Changed: Side effects: - Performance effects(性能

[incubator-brpc] 01/01: community: adjust markdown format

2022-10-30 Thread xiaofeng
This is an automated email from the ASF dual-hosted git repository. xiaofeng pushed a commit to branch wasphin-patch-1 in repository https://gitbox.apache.org/repos/asf/incubator-brpc.git commit 95b1fbb1aaccc7ca679b28cf695c4d60c25e4b49 Author: Xiaofeng Wang AuthorDate: Mon Oct 31 13:27:30 2022 +

[incubator-brpc] branch wasphin-patch-1 created (now 95b1fbb1)

2022-10-30 Thread xiaofeng
This is an automated email from the ASF dual-hosted git repository. xiaofeng pushed a change to branch wasphin-patch-1 in repository https://gitbox.apache.org/repos/asf/incubator-brpc.git at 95b1fbb1 community: adjust markdown format This branch includes the following new commits: n

[incubator-brpc] branch master updated: Update newcommitter_en.md

2022-10-30 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/incubator-brpc.git The following commit(s) were added to refs/heads/master by this push: new 218c3ed0 Update newcommitter_en.md 218c3

[GitHub] [incubator-brpc] Lhfcws commented on issue #1390: memory leak if binding failed

2022-10-30 Thread GitBox
Lhfcws commented on issue #1390: URL: https://github.com/apache/incubator-brpc/issues/1390#issuecomment-1296559317 > > server.ReleaseKeytableForUT(); > > 这个方法是哪来的,源码里没找到呀 我修改了下描述。“我的版本是内部改过一些的版本,如果没有这个方法的可以自己加下。” -- This is an automated message from the Apache Git Service. T

[GitHub] [incubator-brpc] kof02guy commented on issue #1976: 请问在使用single这种连接方式时,如何限制服务端的最大连接数

2022-10-30 Thread GitBox
kof02guy commented on issue #1976: URL: https://github.com/apache/incubator-brpc/issues/1976#issuecomment-1296513098 > > 多谢回复。请问对于服务端的连接数,是否有一个推荐的值的范围。如果连接数过多,除了linux本身的limit限制之外,还会有别的影响么? > > 连接数一般来说没什么影响,只要不超过linux本身limit限制即可。 对服务可用性影响更大的是并发请求数,这个一般通过ServerOptions的max_concurrency来限

[GitHub] [incubator-brpc] cdjingit closed issue #1874: 熔断无法恢复,错误码为112

2022-10-30 Thread GitBox
cdjingit closed issue #1874: 熔断无法恢复,错误码为112 URL: https://github.com/apache/incubator-brpc/issues/1874 -- 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: de

[GitHub] [incubator-brpc] wwbmmm commented on issue #1976: 请问在使用single这种连接方式时,如何限制服务端的最大连接数

2022-10-30 Thread GitBox
wwbmmm commented on issue #1976: URL: https://github.com/apache/incubator-brpc/issues/1976#issuecomment-1296508761 > 多谢回复。请问对于服务端的连接数,是否有一个推荐的值的范围。如果连接数过多,除了linux本身的limit限制之外,还会有别的影响么? 连接数一般来说没什么影响,只要不超过linux本身limit限制即可。 对服务可用性影响更大的是并发请求数,这个一般通过ServerOptions的max_concurrency来限制 --

[GitHub] [incubator-brpc] kof02guy commented on issue #1976: 请问在使用single这种连接方式时,如何限制服务端的最大连接数

2022-10-30 Thread GitBox
kof02guy commented on issue #1976: URL: https://github.com/apache/incubator-brpc/issues/1976#issuecomment-1296494080 多谢回复。请问对于服务端的连接数,是否有一个推荐的值的范围。如果连接数过多,除了linux本身的limit限制之外,还会有别的影响么? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [incubator-brpc] wwbmmm commented on issue #1976: 请问在使用single这种连接方式时,如何限制服务端的最大连接数

2022-10-30 Thread GitBox
wwbmmm commented on issue #1976: URL: https://github.com/apache/incubator-brpc/issues/1976#issuecomment-1296491288 max_connection_pool_size是限制客户端连接数的。 single模式,每个后端server(ip+port)只会建立一个连接。所以只要限制server个数,就可以限制客户端的连接数。 服务端的连接数目前无法限制 -- This is an automated message from the Apache Git

[GitHub] [incubator-brpc] wwbmmm commented on pull request #1975: support SharedMutex.

2022-10-30 Thread GitBox
wwbmmm commented on PR #1975: URL: https://github.com/apache/incubator-brpc/pull/1975#issuecomment-1296488949 CI is failed -- 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. T

[GitHub] [incubator-brpc] wwbmmm commented on pull request #1975: support SharedMutex.

2022-10-30 Thread GitBox
wwbmmm commented on PR #1975: URL: https://github.com/apache/incubator-brpc/pull/1975#issuecomment-1296487575 related: https://github.com/apache/incubator-brpc/pull/1031 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [incubator-brpc] kof02guy opened a new issue, #1976: 请问在使用single这种连接方式时,如何限制服务端的最大连接数

2022-10-30 Thread GitBox
kof02guy opened a new issue, #1976: URL: https://github.com/apache/incubator-brpc/issues/1976 对于连接池来说,有max_connection_pool_size。而对于baidu_std默认的single这种方式,是否有方法去限制服务端创建的最大连接数呢? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub a

[incubator-brpc] branch master updated: Update oncall.md

2022-10-30 Thread wwbmmm
This is an automated email from the ASF dual-hosted git repository. wwbmmm 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 406a0ee1 Update oncall.md 406a0ee1 is de

Oncall report from Oct 24th to Oct 30th

2022-10-30 Thread Weibing Wang
Hi Last week we had 6 issues closed and 11 PRs merged. And the following PR looks good to me and need your double check: * https://github.com/apache/incubator-brpc/pull/1958 Best regards Weibing - To unsubscribe, e-mail: dev-u

[GitHub] [incubator-brpc] wwbmmm closed issue #1957: butil::WriteFile()出错,如何返回具体的错误信息,目前所有异常情况都只会返回一个-1的错误码。

2022-10-30 Thread GitBox
wwbmmm closed issue #1957: butil::WriteFile()出错,如何返回具体的错误信息,目前所有异常情况都只会返回一个-1的错误码。 URL: https://github.com/apache/incubator-brpc/issues/1957 -- 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 speci

[GitHub] [incubator-brpc-website] serverglen commented on a diff in pull request #105: try to fix external resources stackpath.bootstrapcdn.com

2022-10-30 Thread GitBox
serverglen commented on code in PR #105: URL: https://github.com/apache/incubator-brpc-website/pull/105#discussion_r1008988463 ## themes/docsy/assets/vendor/bootstrap/_config.yml: ## @@ -49,13 +49,13 @@ download: cdn: # See https://www.srihash.org for info on how to gener

[GitHub] [incubator-brpc] wwbmmm closed issue #1850: 使用 brpc-java 的 client 访问 bprc-cpp 的server 可行么?

2022-10-30 Thread GitBox
wwbmmm closed issue #1850: 使用 brpc-java 的 client 访问 bprc-cpp 的server 可行么? URL: https://github.com/apache/incubator-brpc/issues/1850 -- 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 commen

[GitHub] [incubator-brpc] wwbmmm closed issue #1788: Feature request: Merge RDMA branch into master

2022-10-30 Thread GitBox
wwbmmm closed issue #1788: Feature request: Merge RDMA branch into master URL: https://github.com/apache/incubator-brpc/issues/1788 -- 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 commen

[incubator-brpc] branch master updated: Add release_en doc (#1972)

2022-10-30 Thread wwbmmm
This is an automated email from the ASF dual-hosted git repository. wwbmmm 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 f03a4aa9 Add release_en doc (#1972) f03a

[GitHub] [incubator-brpc] wwbmmm merged pull request #1972: Add release_en doc

2022-10-30 Thread GitBox
wwbmmm merged PR #1972: URL: https://github.com/apache/incubator-brpc/pull/1972 -- 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.apa

[GitHub] [incubator-brpc-website] wwbmmm commented on a diff in pull request #105: try to fix external resources stackpath.bootstrapcdn.com

2022-10-30 Thread GitBox
wwbmmm commented on code in PR #105: URL: https://github.com/apache/incubator-brpc-website/pull/105#discussion_r1008985463 ## themes/docsy/assets/vendor/bootstrap/_config.yml: ## @@ -49,13 +49,13 @@ download: cdn: # See https://www.srihash.org for info on how to generate

[GitHub] [incubator-brpc] serverglen commented on pull request #1972: Add release_en doc

2022-10-30 Thread GitBox
serverglen commented on PR #1972: URL: https://github.com/apache/incubator-brpc/pull/1972#issuecomment-1296445417 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 un

[GitHub] [incubator-brpc-website] serverglen commented on pull request #105: try to fix external resources stackpath.bootstrapcdn.com

2022-10-30 Thread GitBox
serverglen commented on PR #105: URL: https://github.com/apache/incubator-brpc-website/pull/105#issuecomment-1296441351 这样修改应该有问题 -- 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 com

[GitHub] [incubator-brpc-website] serverglen closed pull request #105: try to fix external resources stackpath.bootstrapcdn.com

2022-10-30 Thread GitBox
serverglen closed pull request #105: try to fix external resources stackpath.bootstrapcdn.com URL: https://github.com/apache/incubator-brpc-website/pull/105 -- 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

Re: new committer: Chen Guangming

2022-10-30 Thread Shuai Liu
Welcome Best Regards, serverglen Jerry Tan 于2022年10月30日周日 11:53写道: > The Project Management Committee (PMC) for Apache brpc > > has invited Chen Guangming to become a committer and we are pleased > > to announce that they have accepted. > > > > Thanks and welcome, Guangming。 >

[GitHub] [incubator-brpc-website] wasphin commented on a diff in pull request #105: try to fix external resources stackpath.bootstrapcdn.com

2022-10-30 Thread GitBox
wasphin commented on code in PR #105: URL: https://github.com/apache/incubator-brpc-website/pull/105#discussion_r1008966832 ## themes/docsy/assets/vendor/bootstrap/_config.yml: ## @@ -49,13 +49,13 @@ download: cdn: # See https://www.srihash.org for info on how to generate

[incubator-brpc] branch master updated: add warning for rdma invoke when brpc is not compiled with rdma

2022-10-30 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 87552c13 add warning for rdma invoke

[GitHub] [incubator-brpc] zyearn merged pull request #1967: add warning for rdma invoke when brpc is not compiled with rdma

2022-10-30 Thread GitBox
zyearn merged PR #1967: URL: https://github.com/apache/incubator-brpc/pull/1967 -- 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.apa

[incubator-brpc] branch master updated (123be90c -> 96c690a0)

2022-10-30 Thread jiashunzhu
This is an automated email from the ASF dual-hosted git repository. jiashunzhu pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-brpc.git from 123be90c Merge pull request #1974 from tanzhongyi003/master new ee446ee0 Macos workflow (#10) ne

[GitHub] [incubator-brpc] zyearn closed issue #1842: Integrate MacOS compilation into github actions

2022-10-30 Thread GitBox
zyearn closed issue #1842: Integrate MacOS compilation into github actions URL: https://github.com/apache/incubator-brpc/issues/1842 -- 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 comme

[GitHub] [incubator-brpc] zyearn merged pull request #1899: Macos workflow

2022-10-30 Thread GitBox
zyearn merged PR #1899: URL: https://github.com/apache/incubator-brpc/pull/1899 -- 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.apa

[GitHub] [incubator-brpc] liyichao opened a new pull request, #1975: support SharedMutex.

2022-10-30 Thread GitBox
liyichao opened a new pull request, #1975: URL: https://github.com/apache/incubator-brpc/pull/1975 support SharedMutex that is compatible with c++17 std::shared_mutex, migration from golang, see https://github.com/golang/go/blob/master/src/sync/rwmutex.go -- This is an automated messag