hfutatzhanghb commented on code in PR #7686:
URL: https://github.com/apache/hadoop/pull/7686#discussion_r2106394894
##########
hadoop-hdfs-project/hadoop-hdfs-rbf/src/main/java/org/apache/hadoop/hdfs/server/federation/router/ThreadLocalContext.java:
##########
@@ -75,6 +75,7 @@ public ThreadLocalContext() {
* that the task execution reflects the state of the original calling thread.
*/
public void transfer() {
+ Server.getCurCall().set(null);
Review Comment:
@slfan1989 Thanks a lot for your reviewing, we should reset
`Server.getCurCall` here to avoid async responder thread use wrong ThreadLocal
variable. For example, router always receives `safemode_get` response from
NameNode, if we don't reset `Server.getCurCall()`, we will find the call
information occurs in the debug log which is not accurate.
--
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: common-issues-unsubscr...@hadoop.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org
