hbase-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From te...@apache.org
Subject hbase git commit: HBASE-15291 Revert due to race condition resulting in closed FileSystem
Date Thu, 21 Jul 2016 10:44:41 GMT
Repository: hbase
Updated Branches:
  refs/heads/0.98 a948b0169 -> 9f2bd55d7


HBASE-15291 Revert due to race condition resulting in closed FileSystem


Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/9f2bd55d
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/9f2bd55d
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/9f2bd55d

Branch: refs/heads/0.98
Commit: 9f2bd55d78583afd2317d75ba187b37d3a559579
Parents: a948b01
Author: tedyu <yuzhihong@gmail.com>
Authored: Thu Jul 21 03:44:31 2016 -0700
Committer: tedyu <yuzhihong@gmail.com>
Committed: Thu Jul 21 03:44:31 2016 -0700

----------------------------------------------------------------------
 .../hbase/security/access/SecureBulkLoadEndpoint.java     | 10 ----------
 1 file changed, 10 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/9f2bd55d/hbase-server/src/main/java/org/apache/hadoop/hbase/security/access/SecureBulkLoadEndpoint.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/security/access/SecureBulkLoadEndpoint.java
b/hbase-server/src/main/java/org/apache/hadoop/hbase/security/access/SecureBulkLoadEndpoint.java
index 6e53ec0..41613dd 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/security/access/SecureBulkLoadEndpoint.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/security/access/SecureBulkLoadEndpoint.java
@@ -291,16 +291,6 @@ public class SecureBulkLoadEndpoint extends SecureBulkLoadService
                 new SecureBulkLoadListener(fs, bulkToken, conf));
           } catch (Exception e) {
             LOG.error("Failed to complete bulk load", e);
-          } finally {
-            if (fs != null) {
-              try {
-                if (!UserGroupInformation.getLoginUser().equals(ugi)) {
-                  FileSystem.closeAllForUGI(ugi);
-                }
-              } catch (IOException e) {
-                LOG.error("Failed to close FileSystem for " + ugi.getUserName(), e);
-              }
-            }
           }
           return false;
         }


Mime
View raw message