You are viewing a plain text version of this content. The canonical link for it is here.
Posted to hdfs-commits@hadoop.apache.org by jg...@apache.org on 2010/07/09 23:14:37 UTC
svn commit: r962696 - in /hadoop/hdfs/trunk: CHANGES.txt
src/java/org/apache/hadoop/hdfs/server/datanode/DataNode.java
Author: jghoman
Date: Fri Jul 9 21:14:36 2010
New Revision: 962696
URL: http://svn.apache.org/viewvc?rev=962696&view=rev
Log:
HDFS-1289. Datanode secure mode is broken. Contributed by Kan Zhang.
Modified:
hadoop/hdfs/trunk/CHANGES.txt
hadoop/hdfs/trunk/src/java/org/apache/hadoop/hdfs/server/datanode/DataNode.java
Modified: hadoop/hdfs/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/hdfs/trunk/CHANGES.txt?rev=962696&r1=962695&r2=962696&view=diff
==============================================================================
--- hadoop/hdfs/trunk/CHANGES.txt (original)
+++ hadoop/hdfs/trunk/CHANGES.txt Fri Jul 9 21:14:36 2010
@@ -131,6 +131,8 @@ Trunk (unreleased changes)
HDFS-1045. In secure clusters, re-login is necessary for https
clients before opening connections. (jghoman)
+ HDFS-1289. Datanode secure mode is broken. (Kan Zhang via jghoman)
+
Release 0.21.0 - Unreleased
INCOMPATIBLE CHANGES
Modified: hadoop/hdfs/trunk/src/java/org/apache/hadoop/hdfs/server/datanode/DataNode.java
URL: http://svn.apache.org/viewvc/hadoop/hdfs/trunk/src/java/org/apache/hadoop/hdfs/server/datanode/DataNode.java?rev=962696&r1=962695&r2=962696&view=diff
==============================================================================
--- hadoop/hdfs/trunk/src/java/org/apache/hadoop/hdfs/server/datanode/DataNode.java (original)
+++ hadoop/hdfs/trunk/src/java/org/apache/hadoop/hdfs/server/datanode/DataNode.java Fri Jul 9 21:14:36 2010
@@ -248,11 +248,6 @@ public class DataNode extends Configured
final DatanodeProtocol namenode) throws IOException {
super(conf);
- UserGroupInformation.setConfiguration(conf);
- DFSUtil.login(conf,
- DFSConfigKeys.DFS_DATANODE_KEYTAB_FILE_KEY,
- DFSConfigKeys.DFS_DATANODE_USER_NAME_KEY);
-
DataNode.setDataNode(this);
try {
@@ -1398,6 +1393,10 @@ public class DataNode extends Configured
Collection<URI> dataDirs = getStorageDirs(conf);
dnThreadName = "DataNode: [" +
StringUtils.uriToString(dataDirs.toArray(new URI[0])) + "]";
+ UserGroupInformation.setConfiguration(conf);
+ DFSUtil.login(conf,
+ DFSConfigKeys.DFS_DATANODE_KEYTAB_FILE_KEY,
+ DFSConfigKeys.DFS_DATANODE_USER_NAME_KEY);
return makeInstance(dataDirs, conf);
}