[GitHub] [hadoop] ZanderXu commented on a diff in pull request #4821: HDFS-16750. NameNode should use NameNode.getRemoteUser() to log audit event to avoid possible NPE

2022-08-29 Thread GitBox
ZanderXu commented on code in PR #4821: URL: https://github.com/apache/hadoop/pull/4821#discussion_r957930476 ## hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/datanode/DataNode.java: ## @@ -1439,7 +1440,7 @@ private void checkSuperuserPrivilege()

[GitHub] [hadoop] ZanderXu commented on a diff in pull request #4821: HDFS-16750. NameNode should use NameNode.getRemoteUser() to log audit event to avoid possible NPE

2022-08-29 Thread GitBox
ZanderXu commented on code in PR #4821: URL: https://github.com/apache/hadoop/pull/4821#discussion_r957930476 ## hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/datanode/DataNode.java: ## @@ -1439,7 +1440,7 @@ private void checkSuperuserPrivilege()

[GitHub] [hadoop] ZanderXu commented on a diff in pull request #4821: HDFS-16750. NameNode should use NameNode.getRemoteUser() to log audit event to avoid possible NPE

2022-08-29 Thread GitBox
ZanderXu commented on code in PR #4821: URL: https://github.com/apache/hadoop/pull/4821#discussion_r957930476 ## hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/datanode/DataNode.java: ## @@ -1439,7 +1440,7 @@ private void checkSuperuserPrivilege()