MAPREDUCE-3280. Removed the unnecessary job user-name configuration in mapred-site.xml. (vinodkv)

git-svn-id: https://svn.apache.org/repos/asf/hadoop/common/trunk@1200441 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
Vinod Kumar Vavilapalli 2011-11-10 17:28:16 +00:00
parent 2df83e5b4d
commit 15150cd997
4 changed files with 5 additions and 11 deletions

View File

@ -70,6 +70,9 @@ Release 0.23.1 - Unreleased
MAPREDUCE-3336. Replaced guice internal.Preconditions api usage with the
public Preconditions API. (Thomas Graves via vinodkv)
MAPREDUCE-3280. Removed the unnecessary job user-name configuration in
mapred-site.xml. (vinodkv)
OPTIMIZATIONS
BUG FIXES

View File

@ -130,8 +130,8 @@ class YarnChild {
// Initiate Java VM metrics
JvmMetrics.initSingleton(jvmId.toString(), job.getSessionId());
LOG.debug("Remote user: " + job.get("user.name"));
childUGI = UserGroupInformation.createRemoteUser(job.get("user.name"));
childUGI = UserGroupInformation.createRemoteUser(System
.getenv(ApplicationConstants.Environment.USER.toString()));
// Add tokens to new user so that it may execute its task correctly.
for(Token<?> token : UserGroupInformation.getCurrentUser().getTokens()) {
childUGI.addToken(token);

View File

@ -1224,12 +1224,4 @@
mapreduce.job.end-notification.max.retry.interval</description>
</property>
<property>
<name>mapreduce.job.user.name</name>
<value>${user.name}</value>
<description>The user name for the job submitter, configurable only in
non-secure mode. In secure mode Kerberos authentication is necessary.
</description>
</property>
</configuration>

View File

@ -71,7 +71,6 @@ public class MiniMRYarnCluster extends MiniYARNCluster {
@Override
public void init(Configuration conf) {
conf.set(MRConfig.FRAMEWORK_NAME, MRConfig.YARN_FRAMEWORK_NAME);
conf.set(MRJobConfig.USER_NAME, System.getProperty("user.name"));
conf.set(MRJobConfig.MR_AM_STAGING_DIR, new File(getTestWorkDir(),
"apps_staging_dir/${user.name}/").getAbsolutePath());
conf.set(MRConfig.MASTER_ADDRESS, "test"); // The default is local because of