apache / brpc

brpc is an Industrial-grade RPC framework using C++ Language, which is often used in high performance system such as Search, Storage, Machine learning, Advertisement, Recommendation etc. "brpc" means "better RPC".
https://brpc.apache.org
Apache License 2.0
16.56k stars 3.98k forks source link

feat: add a service option to disable check eovercrowded on server side #2774

Closed superhail closed 1 month ago

superhail commented 1 month ago

What problem does this PR solve?

Issue Number:

Problem Summary: Sometimes we need to disable check eovercrowded for some methods

What is changed and the side effects?

Changed: add a method/service level option to disable check eovercrowded on server side

Side effects:


Check List:

wwbmmm commented 1 month ago

What is the usage scenario of this feature?

superhail commented 1 month ago

@wwbmmm we have some control-frame methods which we hope will not fail in case of eovercrowded.

superhail commented 1 month ago

@chenBright please help check

superhail commented 1 month ago

@chenBright http2 needed as well?

chenBright commented 1 month ago

@chenBright http2 needed as well?

http2 also use ProcessHttpRequest.

Protocols that use the is_overcrowded function need as well.

superhail commented 1 month ago

@wwbmmm @chenBright all done

wwbmmm commented 1 month ago

LGTM