-
Notifications
You must be signed in to change notification settings - Fork 8.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
HDFS-17651.[ARR] Async handler executor isolation #7244
base: HDFS-17531
Are you sure you want to change the base?
Conversation
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
@hfutatzhanghb hi, thanks for your contribution! Could you briefly introduce your specific plan, for example: the background, purpose, and solution of PR? |
@KeeProMise Thanks for reminding, sir. Will update those information these days. |
1e07b5f
to
932d69b
Compare
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
932d69b
to
1d4508b
Compare
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
@Hexiaoqiao @KeeProMise Sir, could you please help review this PR when you have free time? Thanks a lot. |
@hfutatzhanghb thanks for your contribution! LGTM |
Description of PR
The main purpose of this PR is to better isolate each nameservice by letting each nameservice has its own async handler thread pool.
Think below situation:
When a downstream nameserivce ns1 has poor performance, the threads in async handler thread pool will be occupied adding calls to the ns1's Connection#calls. There will be no available threads to handle normal nameservice's rpc request asynchronously. Therefore, it is better to isolate the async handler thread pool of different nameservices.