KeeProMise commented on PR #6871: URL: https://github.com/apache/hadoop/pull/6871#issuecomment-2342733716
Hi, @goiri @simbadzina @Hexiaoqiao @sjlee @ayushtkn @haiyang1987 @ZanderXu the PR has been blocked for 2 months, and if everyone has time, please help to review it because several subtask PRs need to depend on this PR. This PR does not modify the existing synchronous router logic. To facilitate review, my **main modifications** are as follows: - RouterRpcClient.java: The original functionality and logic have not been changed; I only have extracted some common methods. - RouterAsyncRpcClient.java: An asynchronous implementation of RouterRpcClient. - Added configuration for the asynchronous feature toggle, as well as the number of asynchronous handlers and responders. - Using ThreadLocalContext to maintain thread local variables, ensuring that thread local variables can be correctly passed between handler, asyncRouterHandler, and asyncRouterResponder. The PRs that depend on this PR are: https://github.com/apache/hadoop/pull/6994 https://github.com/apache/hadoop/pull/6988 https://github.com/apache/hadoop/pull/6986 https://github.com/apache/hadoop/pull/6983 -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
