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

stoty · wenwj0 · commit db0695d7ae1c · 2022-06-14T10:31:29.000+08:00
Signed-off-by: Rajeshbabu Chintaguntla &lt;rajeshbabu@apache.org&gt;
Signed-off-by: Wellington Chevreuil &lt;wchevreuil@apache.org&gt;
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,10 +398,7 @@ public String prepareBulkLoad(final byte[] family, final String srcPath, boolean
           throw new IOException("Failed to move HFile: " + p + " to " + stageP);
         }
       }
-
-      if (StringUtils.isNotEmpty(customStaging)) {
-        fs.setPermission(stageP, PERM_ALL_ACCESS);
-      }
+      fs.setPermission(stageP, PERM_ALL_ACCESS);
 
       return stageP.toString();
     }

Original file line number	Diff line number	Diff line change
`@@ -398,10 +398,7 @@ 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`		`-`
`402`		`- if (StringUtils.isNotEmpty(customStaging)) {`
`403`		`- fs.setPermission(stageP, PERM_ALL_ACCESS);`
`404`		`- }`
	`401`	`+ fs.setPermission(stageP, PERM_ALL_ACCESS);`
`405`	`402`
`406`	`403`	`return stageP.toString();`
`407`	`404`	`}`