Revert "HBASE-27069 Hbase SecureBulkload permission regression (apache#4475)"

wenwj0 · web-flow · commit 806a7eeeda17 · 2022-06-14T14:23:27.000+08:00
This reverts commit db0695d.
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/SecureBulkLoadManager.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/SecureBulkLoadManager.java
@@ -398,7 +398,10 @@ public String prepareBulkLoad(final byte[] family, final String srcPath, boolean
           throw new IOException("Failed to move HFile: " + p + " to " + stageP);
         }
       }
-      fs.setPermission(stageP, PERM_ALL_ACCESS);
+
+      if (StringUtils.isNotEmpty(customStaging)) {
+        fs.setPermission(stageP, PERM_ALL_ACCESS);
+      }
 
       return stageP.toString();
     }

Original file line number	Diff line number	Diff line change
`@@ -398,7 +398,10 @@ public String prepareBulkLoad(final byte[] family, final String srcPath, boolean`
`398`	`398`	`throw new IOException("Failed to move HFile: " + p + " to " + stageP);`
`399`	`399`	`}`
`400`	`400`	`}`
`401`		`- fs.setPermission(stageP, PERM_ALL_ACCESS);`
	`401`	`+`
	`402`	`+ if (StringUtils.isNotEmpty(customStaging)) {`
	`403`	`+ fs.setPermission(stageP, PERM_ALL_ACCESS);`
	`404`	`+ }`
`402`	`405`
`403`	`406`	`return stageP.toString();`
`404`	`407`	`}`