jianghuazhu commented on code in PR #4659:
URL: https://github.com/apache/hadoop/pull/4659#discussion_r937524227


##########
hadoop-hdfs-project/hadoop-hdfs-rbf/src/test/java/org/apache/hadoop/hdfs/server/federation/router/TestRouterRpc.java:
##########
@@ -2070,6 +2077,46 @@ public void testSetBalancerBandwidth() throws Exception {
     }, 100, 60 * 1000);
   }
 
+  @Test
+  public void testRecordRealClient() throws Exception {

Review Comment:
   Thanks @goiri  for the comment and review.
   If you want HADOOP_CALLER_CONTEXT_ENABLED_KEY to reset and take effect, 
MiniRouterDFSCluster needs to be restarted, so I refactored the process of 
starting MiniRouterDFSCluster. And use a static Configuration, which is for the 
NameNode.
   You mean keep globalSetUp() unchanged and add static Configuration?
   Another question is, do you think the new format of logging needs to be 
updated?



-- 
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]

Reply via email to