HDFS-10455. Logging the username when deny the setOwner operation. Contributed by Rakesh R.

brahmareddybattula · brahmareddybattula · commit c895f0893621 · 2016-11-01T10:53:37.000+05:30
(cherry picked from commit 1c5708f)
diff --git a/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/FSDirAttrOp.java b/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/FSDirAttrOp.java
@@ -83,10 +83,12 @@ static HdfsFileStatus setOwner(
       fsd.checkOwner(pc, iip);
       if (!pc.isSuperUser()) {
         if (username != null && !pc.getUser().equals(username)) {
-          throw new AccessControlException("Non-super user cannot change owner");
+          throw new AccessControlException("User " + username
+              + " is not a super user (non-super user cannot change owner).");
         }
         if (group != null && !pc.isMemberOfGroup(group)) {
-          throw new AccessControlException("User does not belong to " + group);
+          throw new AccessControlException(
+              "User " + username + " does not belong to " + group);
         }
       }
       unprotectedSetOwner(fsd, iip, username, group);

Original file line number	Diff line number	Diff line change
`@@ -83,10 +83,12 @@ static HdfsFileStatus setOwner(`
`83`	`83`	`fsd.checkOwner(pc, iip);`
`84`	`84`	`if (!pc.isSuperUser()) {`
`85`	`85`	`if (username != null && !pc.getUser().equals(username)) {`
`86`		`- throw new AccessControlException("Non-super user cannot change owner");`
	`86`	`+ throw new AccessControlException("User " + username`
	`87`	`+ + " is not a super user (non-super user cannot change owner).");`
`87`	`88`	`}`
`88`	`89`	`if (group != null && !pc.isMemberOfGroup(group)) {`
`89`		`- throw new AccessControlException("User does not belong to " + group);`
	`90`	`+ throw new AccessControlException(`
	`91`	`+ "User " + username + " does not belong to " + group);`
`90`	`92`	`}`
`91`	`93`	`}`
`92`	`94`	`unprotectedSetOwner(fsd, iip, username, group);`