This is an automated email from the ASF dual-hosted git repository.

wchevreuil pushed a commit to branch branch-2
in repository https://gitbox.apache.org/repos/asf/hbase.git


The following commit(s) were added to refs/heads/branch-2 by this push:
     new 7fbfb10649 HBASE-27069 Hbase SecureBulkload permission regression 
(#4475)
7fbfb10649 is described below

commit 7fbfb10649d9ce1365ecb26934a216970e2e09c6
Author: Istvan Toth <[email protected]>
AuthorDate: Tue May 31 10:52:35 2022 +0200

    HBASE-27069 Hbase SecureBulkload permission regression (#4475)
    
    Signed-off-by: Rajeshbabu Chintaguntla <[email protected]>
    Signed-off-by: Wellington Chevreuil <[email protected]>
    
    (cherry picked from commit 4ce85aa39ffe353ec0d23dd7dd96e1052ac6af81)
---
 .../org/apache/hadoop/hbase/regionserver/SecureBulkLoadManager.java  | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

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
index 5b1a3302f3..d45f5cf54d 100644
--- 
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 class SecureBulkLoadManager {
           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();
     }

Reply via email to