You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by te...@apache.org on 2014/03/01 01:44:21 UTC

svn commit: r1573115 - /hbase/branches/0.98/hbase-server/src/main/java/org/apache/hadoop/hbase/security/token/FsDelegationToken.java

Author: tedyu
Date: Sat Mar  1 00:44:21 2014
New Revision: 1573115

URL: http://svn.apache.org/r1573115
Log:
HBASE-10644 TestSecureExportSnapshot#testExportFileSystemState fails on hadoop-1


Modified:
    hbase/branches/0.98/hbase-server/src/main/java/org/apache/hadoop/hbase/security/token/FsDelegationToken.java

Modified: hbase/branches/0.98/hbase-server/src/main/java/org/apache/hadoop/hbase/security/token/FsDelegationToken.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.98/hbase-server/src/main/java/org/apache/hadoop/hbase/security/token/FsDelegationToken.java?rev=1573115&r1=1573114&r2=1573115&view=diff
==============================================================================
--- hbase/branches/0.98/hbase-server/src/main/java/org/apache/hadoop/hbase/security/token/FsDelegationToken.java (original)
+++ hbase/branches/0.98/hbase-server/src/main/java/org/apache/hadoop/hbase/security/token/FsDelegationToken.java Sat Mar  1 00:44:21 2014
@@ -68,7 +68,12 @@ public class FsDelegationToken {
                                                       fs.getCanonicalServiceName());
       if (userToken == null) {
         hasForwardedToken = false;
-        userToken = fs.getDelegationToken(renewer);
+        try {
+          userToken = fs.getDelegationToken(renewer);
+        } catch (NullPointerException npe) {
+          // we need to handle NullPointerException in case HADOOP-10009 is missing
+          LOG.error("Failed to get token for " + renewer);
+        }
       } else {
         hasForwardedToken = true;
         LOG.info("Use the existing token: " + userToken);