[jira] [Commented] (MAPREDUCE-6499) Add More Informations Of Jobs In JobHistory Main Page

2015-10-28 Thread Lin Yiqun (JIRA)

[ 
https://issues.apache.org/jira/browse/MAPREDUCE-6499?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14978292#comment-14978292
 ] 

Lin Yiqun commented on MAPREDUCE-6499:
--

Thanks [~ajisakaa] advices. So should I update my patch and only leave the 
things as you said ?

> Add More Informations Of Jobs In JobHistory Main Page
> -
>
> Key: MAPREDUCE-6499
> URL: https://issues.apache.org/jira/browse/MAPREDUCE-6499
> Project: Hadoop Map/Reduce
>  Issue Type: Improvement
>Affects Versions: 2.7.1
>Reporter: Lin Yiqun
>Assignee: Lin Yiqun
>Priority: Trivial
> Attachments: HADOOP-MAPREDUCE-6499.2.patch, 
> HADOOP-MAPREDUCE-6499.3.patch, HADOOP-MAPREDUCE-6499.4.patch, 
> HADOOP-MAPREDUCE-6499.5.patch, HADOOP-MAPREDUCE-6499.patch
>
>
> Now in  JobHistory Main Page show too little information about finished 
> jobs,even don't have the job's running total time,only 
> startTime,finishedTime.So,in jobHistory main page,we can add more 
> informations about jobs, that we can better analyze jobs.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (MAPREDUCE-6526) Remove usage of metrics v1 from hadoop-mapreduce

2015-10-28 Thread Tsuyoshi Ozawa (JIRA)

[ 
https://issues.apache.org/jira/browse/MAPREDUCE-6526?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14978349#comment-14978349
 ] 

Tsuyoshi Ozawa commented on MAPREDUCE-6526:
---

Do you think LocalJobRunnerMetrics and ShuffleClientMetrics are not useful 
Metrics? IMHO, they are still useful to analyse performance. I prefer to 
rewrite them to use Metrics v2. Compatibility itself is second priority. 
However, I think third party tools use the values of counters. I also prefer to 
use same counter name at least.

> Remove usage of metrics v1 from hadoop-mapreduce
> 
>
> Key: MAPREDUCE-6526
> URL: https://issues.apache.org/jira/browse/MAPREDUCE-6526
> Project: Hadoop Map/Reduce
>  Issue Type: Improvement
>Reporter: Akira AJISAKA
>Assignee: Akira AJISAKA
>Priority: Blocker
>
> LocalJobRunnerMetrics and ShuffleClientMetrics are still using metrics v1. We 
> should remove these metrics or rewrite them to use metrics v2.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (MAPREDUCE-6499) Add More Informations Of Jobs In JobHistory Main Page

2015-10-28 Thread Lin Yiqun (JIRA)

 [ 
https://issues.apache.org/jira/browse/MAPREDUCE-6499?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Lin Yiqun updated MAPREDUCE-6499:
-
Attachment: HADOOP-MAPREDUCE-6499.6.patch

Simplified the patch, only leave the elapsed time and modfiy some wrong hint 
text in search input.

> Add More Informations Of Jobs In JobHistory Main Page
> -
>
> Key: MAPREDUCE-6499
> URL: https://issues.apache.org/jira/browse/MAPREDUCE-6499
> Project: Hadoop Map/Reduce
>  Issue Type: Improvement
>Affects Versions: 2.7.1
>Reporter: Lin Yiqun
>Assignee: Lin Yiqun
>Priority: Trivial
> Attachments: HADOOP-MAPREDUCE-6499.2.patch, 
> HADOOP-MAPREDUCE-6499.3.patch, HADOOP-MAPREDUCE-6499.4.patch, 
> HADOOP-MAPREDUCE-6499.5.patch, HADOOP-MAPREDUCE-6499.6.patch, 
> HADOOP-MAPREDUCE-6499.patch
>
>
> Now in  JobHistory Main Page show too little information about finished 
> jobs,even don't have the job's running total time,only 
> startTime,finishedTime.So,in jobHistory main page,we can add more 
> informations about jobs, that we can better analyze jobs.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (MAPREDUCE-6525) Fix test failure of TestMiniMRClientCluster.testRestart

2015-10-28 Thread Akira AJISAKA (JIRA)

[ 
https://issues.apache.org/jira/browse/MAPREDUCE-6525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14977859#comment-14977859
 ] 

Akira AJISAKA commented on MAPREDUCE-6525:
--

Triggered Jenkins build: 
https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/6095

> Fix test failure of TestMiniMRClientCluster.testRestart
> ---
>
> Key: MAPREDUCE-6525
> URL: https://issues.apache.org/jira/browse/MAPREDUCE-6525
> Project: Hadoop Map/Reduce
>  Issue Type: Bug
>  Components: test
>Reporter: Masatake Iwasaki
>Assignee: Masatake Iwasaki
>Priority: Minor
> Attachments: MAPREDUCE-6525.001.patch
>
>
> MiniMRYarnClusterAdapter#restart creates new MiniMRYarnCluster with 
> configuration of existing MiniMRYarnCluster but the address of HistoryServer 
> is properly set.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (MAPREDUCE-6525) Fix test failure of TestMiniMRClientCluster.testRestart

2015-10-28 Thread Akira AJISAKA (JIRA)

[ 
https://issues.apache.org/jira/browse/MAPREDUCE-6525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14977857#comment-14977857
 ] 

Akira AJISAKA commented on MAPREDUCE-6525:
--

bq. The copy and logging of configuration should be done after all services are 
started.
Make sense to me. +1 for the patch.

bq. The error message of TestNonExistentJob is similar to MAPREDUCE-6524 and it 
implies that the fix is not enough.
I could not reproduce the error locally.

> Fix test failure of TestMiniMRClientCluster.testRestart
> ---
>
> Key: MAPREDUCE-6525
> URL: https://issues.apache.org/jira/browse/MAPREDUCE-6525
> Project: Hadoop Map/Reduce
>  Issue Type: Bug
>  Components: test
>Reporter: Masatake Iwasaki
>Assignee: Masatake Iwasaki
>Priority: Minor
> Attachments: MAPREDUCE-6525.001.patch
>
>
> MiniMRYarnClusterAdapter#restart creates new MiniMRYarnCluster with 
> configuration of existing MiniMRYarnCluster but the address of HistoryServer 
> is properly set.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (MAPREDUCE-6519) hadoop-mapreduce - Avoid unsafe split and append on fields that might be IPv6 literals

2015-10-28 Thread Nemanja Matkovic (JIRA)

 [ 
https://issues.apache.org/jira/browse/MAPREDUCE-6519?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Nemanja Matkovic updated MAPREDUCE-6519:

Attachment: MAPREDUCE-6519-HADOOP-11890.2.patch

Identical code as in HADOOP-11890.1.patch, only difference is that YARN changes 
are committed.
All tests passing locally as with HADOOP-11890.1.patch, re-ran tests that 
failed with patch 1 and they still passed, let's see if things repro again or 
not.

> hadoop-mapreduce - Avoid unsafe split and append on fields that might be IPv6 
> literals
> --
>
> Key: MAPREDUCE-6519
> URL: https://issues.apache.org/jira/browse/MAPREDUCE-6519
> Project: Hadoop Map/Reduce
>  Issue Type: Task
>Reporter: Nemanja Matkovic
>Assignee: Nemanja Matkovic
>  Labels: ipv6
> Attachments: MAPREDUCE-6519-HADOOP-11890.1.patch, 
> MAPREDUCE-6519-HADOOP-11890.2.patch
>
>   Original Estimate: 48h
>  Remaining Estimate: 48h
>
> mapreduce portion of patch in HADOOP-12122 that couldn't run due to number of 
> components touched.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Created] (MAPREDUCE-6527) Data race on field org.apache.hadoop.mapred.JobConf.credentials

2015-10-28 Thread Ali Kheradmand (JIRA)
Ali Kheradmand created MAPREDUCE-6527:
-

 Summary: Data race on field 
org.apache.hadoop.mapred.JobConf.credentials
 Key: MAPREDUCE-6527
 URL: https://issues.apache.org/jira/browse/MAPREDUCE-6527
 Project: Hadoop Map/Reduce
  Issue Type: Bug
Reporter: Ali Kheradmand


I am running the test suite against a dynamic race detector called RV-Predict. 
Here is a race report that I got: 

Data race on field org.apache.hadoop.mapred.JobConf.credentials: {{{
Concurrent read in thread T327 (locks held: {})
 >  at org.apache.hadoop.mapred.JobConf.getCredentials(JobConf.java:505)
at 
org.apache.hadoop.mapreduce.task.JobContextImpl.(JobContextImpl.java:70)
at 
org.apache.hadoop.mapred.LocalJobRunner$Job.run(LocalJobRunner.java:524)
T327 is created by T22
at 
org.apache.hadoop.mapred.LocalJobRunner$Job.(LocalJobRunner.java:218)

Concurrent write in thread T22 (locks held: {Monitor@496c673a, 
Monitor@496319b0})
 >  at org.apache.hadoop.mapred.JobConf.setCredentials(JobConf.java:510)
at 
org.apache.hadoop.mapred.LocalJobRunner.submitJob(LocalJobRunner.java:787)
at 
org.apache.hadoop.mapreduce.JobSubmitter.submitJobInternal(JobSubmitter.java:241)
at org.apache.hadoop.mapreduce.Job$10.run(Job.java:1290)
at javax.security.auth.Subject.doAs(Subject.java:422)
at 
org.apache.hadoop.security.UserGroupInformation.doAs(UserGroupInformation.java:1669)
at org.apache.hadoop.mapreduce.Job.submit(Job.java:1287)
at 
org.apache.hadoop.mapreduce.lib.jobcontrol.ControlledJob.submit(ControlledJob.java:335)
- locked Monitor@496319b0 at 
org.apache.hadoop.mapreduce.lib.jobcontrol.ControlledJob.submit(ControlledJob.java:n/a)
 
at 
org.apache.hadoop.mapreduce.lib.jobcontrol.JobControl.run(JobControl.java:245)
- locked Monitor@496c673a at 
org.apache.hadoop.mapreduce.lib.jobcontrol.JobControl.run(JobControl.java:229) 
T22 is created by T1
at 
org.apache.hadoop.mapred.jobcontrol.TestJobControl.doJobControlTest(TestJobControl.java:111)
}}}

In the source code of org.apache.hadoop.mapreduce.JobStatus.submitJob function, 
we have the following lines:
Job job = new Job(JobID.downgrade(jobid), jobSubmitDir);
job.job.setCredentials(credentials);

It looks a bit suspicious: Job extends thread and at the end of its constructor 
it starts a new thread which creates a new instance of JobContextImpl which 
reads credentials. However, the first thread concurrently sets credentials 
after a creating the Job instance. 





--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (MAPREDUCE-6527) Data race on field org.apache.hadoop.mapred.JobConf.credentials

2015-10-28 Thread Ali Kheradmand (JIRA)

 [ 
https://issues.apache.org/jira/browse/MAPREDUCE-6527?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Ali Kheradmand updated MAPREDUCE-6527:
--
Description: 
I am running the test suite against a dynamic race detector called RV-Predict. 
Here is a race report that I got: 

Data race on field org.apache.hadoop.mapred.JobConf.credentials: {{{
Concurrent read in thread T327 (locks held: {})
 >  at org.apache.hadoop.mapred.JobConf.getCredentials(JobConf.java:505)
at 
org.apache.hadoop.mapreduce.task.JobContextImpl.(JobContextImpl.java:70)
at 
org.apache.hadoop.mapred.LocalJobRunner$Job.run(LocalJobRunner.java:524)
T327 is created by T22
at 
org.apache.hadoop.mapred.LocalJobRunner$Job.(LocalJobRunner.java:218)

Concurrent write in thread T22 (locks held: {Monitor@496c673a, 
Monitor@496319b0})
 >  at org.apache.hadoop.mapred.JobConf.setCredentials(JobConf.java:510)
at 
org.apache.hadoop.mapred.LocalJobRunner.submitJob(LocalJobRunner.java:787)
at 
org.apache.hadoop.mapreduce.JobSubmitter.submitJobInternal(JobSubmitter.java:241)
at org.apache.hadoop.mapreduce.Job$10.run(Job.java:1290)
at javax.security.auth.Subject.doAs(Subject.java:422)
at 
org.apache.hadoop.security.UserGroupInformation.doAs(UserGroupInformation.java:1669)
at org.apache.hadoop.mapreduce.Job.submit(Job.java:1287)
at 
org.apache.hadoop.mapreduce.lib.jobcontrol.ControlledJob.submit(ControlledJob.java:335)
 locked Monitor@496319b0 at 
org.apache.hadoop.mapreduce.lib.jobcontrol.ControlledJob.submit(ControlledJob.java:n/a)
 
at 
org.apache.hadoop.mapreduce.lib.jobcontrol.JobControl.run(JobControl.java:245)
 locked Monitor@496c673a at 
org.apache.hadoop.mapreduce.lib.jobcontrol.JobControl.run(JobControl.java:229) 
T22 is created by T1
at 
org.apache.hadoop.mapred.jobcontrol.TestJobControl.doJobControlTest(TestJobControl.java:111)
}}}

In the source code of org.apache.hadoop.mapreduce.JobStatus.submitJob function, 
we have the following lines:
Job job = new Job(JobID.downgrade(jobid), jobSubmitDir);
job.job.setCredentials(credentials);

It looks a bit suspicious: Job extends thread and at the end of its constructor 
it starts a new thread which creates a new instance of JobContextImpl which 
reads credentials. However, the first thread concurrently sets credentials 
after a creating the Job instance. 



  was:
I am running the test suite against a dynamic race detector called RV-Predict. 
Here is a race report that I got: 

Data race on field org.apache.hadoop.mapred.JobConf.credentials: {{{
Concurrent read in thread T327 (locks held: {})
 >  at org.apache.hadoop.mapred.JobConf.getCredentials(JobConf.java:505)
at 
org.apache.hadoop.mapreduce.task.JobContextImpl.(JobContextImpl.java:70)
at 
org.apache.hadoop.mapred.LocalJobRunner$Job.run(LocalJobRunner.java:524)
T327 is created by T22
at 
org.apache.hadoop.mapred.LocalJobRunner$Job.(LocalJobRunner.java:218)

Concurrent write in thread T22 (locks held: {Monitor@496c673a, 
Monitor@496319b0})
 >  at org.apache.hadoop.mapred.JobConf.setCredentials(JobConf.java:510)
at 
org.apache.hadoop.mapred.LocalJobRunner.submitJob(LocalJobRunner.java:787)
at 
org.apache.hadoop.mapreduce.JobSubmitter.submitJobInternal(JobSubmitter.java:241)
at org.apache.hadoop.mapreduce.Job$10.run(Job.java:1290)
at javax.security.auth.Subject.doAs(Subject.java:422)
at 
org.apache.hadoop.security.UserGroupInformation.doAs(UserGroupInformation.java:1669)
at org.apache.hadoop.mapreduce.Job.submit(Job.java:1287)
at 
org.apache.hadoop.mapreduce.lib.jobcontrol.ControlledJob.submit(ControlledJob.java:335)
- locked Monitor@496319b0 at 
org.apache.hadoop.mapreduce.lib.jobcontrol.ControlledJob.submit(ControlledJob.java:n/a)
 
at 
org.apache.hadoop.mapreduce.lib.jobcontrol.JobControl.run(JobControl.java:245)
- locked Monitor@496c673a at 
org.apache.hadoop.mapreduce.lib.jobcontrol.JobControl.run(JobControl.java:229) 
T22 is created by T1
at 
org.apache.hadoop.mapred.jobcontrol.TestJobControl.doJobControlTest(TestJobControl.java:111)
}}}

In the source code of org.apache.hadoop.mapreduce.JobStatus.submitJob function, 
we have the following lines:
Job job = new Job(JobID.downgrade(jobid), jobSubmitDir);
job.job.setCredentials(credentials);

It looks a bit suspicious: Job extends thread and at the end of its constructor 
it starts a new thread which creates a new instance of JobContextImpl which 
reads credentials. However, the first thread concurrently sets credentials 
after a creating the Job instance. 




> Data race on field org.apache.hadoop.mapred.JobConf.credentials
> ---
>
> Key: MAPREDUCE-6527
> 

[jira] [Commented] (MAPREDUCE-6519) hadoop-mapreduce - Avoid unsafe split and append on fields that might be IPv6 literals

2015-10-28 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/MAPREDUCE-6519?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14979490#comment-14979490
 ] 

Hadoop QA commented on MAPREDUCE-6519:
--

| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 10s 
{color} | {color:blue} docker + precommit patch detected. {color} |
| {color:green}+1{color} | {color:green} @author {color} | {color:green} 0m 0s 
{color} | {color:green} The patch does not contain any @author tags. {color} |
| {color:green}+1{color} | {color:green} test4tests {color} | {color:green} 0m 
0s {color} | {color:green} The patch appears to include 5 new or modified test 
files. {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 9m 
59s {color} | {color:green} HADOOP-11890 passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green} 2m 55s 
{color} | {color:green} HADOOP-11890 passed with JDK v1.8.0_60 {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green} 2m 54s 
{color} | {color:green} HADOOP-11890 passed with JDK v1.7.0_79 {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 
28s {color} | {color:green} HADOOP-11890 passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 1m 
2s {color} | {color:green} HADOOP-11890 passed {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 2m 
57s {color} | {color:green} HADOOP-11890 passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 1m 10s 
{color} | {color:green} HADOOP-11890 passed with JDK v1.8.0_60 {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 1m 12s 
{color} | {color:green} HADOOP-11890 passed with JDK v1.7.0_79 {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 1m 
28s {color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green} 2m 53s 
{color} | {color:green} the patch passed with JDK v1.8.0_60 {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green} 2m 53s 
{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green} 2m 48s 
{color} | {color:green} the patch passed with JDK v1.7.0_79 {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green} 2m 48s 
{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 
24s {color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 0m 
47s {color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} whitespace {color} | {color:green} 0m 
0s {color} | {color:green} Patch has no whitespace issues. {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 3m 
13s {color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 1m 9s 
{color} | {color:green} the patch passed with JDK v1.8.0_60 {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 1m 9s 
{color} | {color:green} the patch passed with JDK v1.7.0_79 {color} |
| {color:red}-1{color} | {color:red} unit {color} | {color:red} 2m 12s {color} 
| {color:red} hadoop-mapreduce-client-core in the patch failed with JDK 
v1.8.0_60. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green} 6m 54s 
{color} | {color:green} hadoop-mapreduce-client-hs in the patch passed with JDK 
v1.8.0_60. {color} |
| {color:red}-1{color} | {color:red} unit {color} | {color:red} 118m 9s {color} 
| {color:red} hadoop-mapreduce-client-jobclient in the patch failed with JDK 
v1.8.0_60. {color} |
| {color:red}-1{color} | {color:red} unit {color} | {color:red} 2m 19s {color} 
| {color:red} hadoop-mapreduce-client-core in the patch failed with JDK 
v1.7.0_79. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green} 6m 31s 
{color} | {color:green} hadoop-mapreduce-client-hs in the patch passed with JDK 
v1.7.0_79. {color} |
| {color:red}-1{color} | {color:red} unit {color} | {color:red} 116m 31s 
{color} | {color:red} hadoop-mapreduce-client-jobclient in the patch failed 
with JDK v1.7.0_79. {color} |
| {color:red}-1{color} | {color:red} asflicense {color} | {color:red} 0m 31s 
{color} | {color:red} Patch generated 1 ASF License warnings. {color} |
| {color:black}{color} | {color:black} {color} | {color:black} 291m 35s {color} 
| {color:black} {color} |
\\
\\
|| Reason || Tests ||
| JDK v1.7.0_79 Failed junit tests | 
hadoop.mapreduce.filecache.TestClientDistributedCacheManager |
|   | 

[jira] [Commented] (MAPREDUCE-6525) Fix test failure of TestMiniMRClientCluster.testRestart

2015-10-28 Thread Masatake Iwasaki (JIRA)

[ 
https://issues.apache.org/jira/browse/MAPREDUCE-6525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14979525#comment-14979525
 ] 

Masatake Iwasaki commented on MAPREDUCE-6525:
-

bq. .. we should use Configuration to enable timeline server instead of using 
enableAHS

I agree but the constructer has already marked as deprecated by YARN-3573. I 
think there is nothing we can do more. We should not remove public constructor 
of mini cluster because it is possible to be used by downstream projects.

This jira is addressing the issue about not AHS but JHS in MiniMRYarnCluster:-) 
Though {{JHAdminConfig}} does not have on/off switch like 
{{YarnConfiguration.TIMELINE_SERVICE_ENABLED}}, I don't think we need to add it.

> Fix test failure of TestMiniMRClientCluster.testRestart
> ---
>
> Key: MAPREDUCE-6525
> URL: https://issues.apache.org/jira/browse/MAPREDUCE-6525
> Project: Hadoop Map/Reduce
>  Issue Type: Bug
>  Components: test
>Reporter: Masatake Iwasaki
>Assignee: Masatake Iwasaki
>Priority: Minor
> Attachments: MAPREDUCE-6525.001.patch
>
>
> MiniMRYarnClusterAdapter#restart creates new MiniMRYarnCluster with 
> configuration of existing MiniMRYarnCluster but the address of HistoryServer 
> is properly set.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (MAPREDUCE-6527) Data race on field org.apache.hadoop.mapred.JobConf.credentials

2015-10-28 Thread Ali Kheradmand (JIRA)

 [ 
https://issues.apache.org/jira/browse/MAPREDUCE-6527?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Ali Kheradmand updated MAPREDUCE-6527:
--
Description: 
I am running the test suite against a dynamic race detector called RV-Predict. 
Here is a race report that I got: 
{noformat}
Data race on field org.apache.hadoop.mapred.JobConf.credentials: {{{
Concurrent read in thread T327 (locks held: {})
 >  at org.apache.hadoop.mapred.JobConf.getCredentials(JobConf.java:505)
at 
org.apache.hadoop.mapreduce.task.JobContextImpl.(JobContextImpl.java:70)
at 
org.apache.hadoop.mapred.LocalJobRunner$Job.run(LocalJobRunner.java:524)
T327 is created by T22
at 
org.apache.hadoop.mapred.LocalJobRunner$Job.(LocalJobRunner.java:218)

Concurrent write in thread T22 (locks held: {Monitor@496c673a, 
Monitor@496319b0})
 >  at org.apache.hadoop.mapred.JobConf.setCredentials(JobConf.java:510)
at 
org.apache.hadoop.mapred.LocalJobRunner.submitJob(LocalJobRunner.java:787)
at 
org.apache.hadoop.mapreduce.JobSubmitter.submitJobInternal(JobSubmitter.java:241)
at org.apache.hadoop.mapreduce.Job$10.run(Job.java:1290)
at javax.security.auth.Subject.doAs(Subject.java:422)
at 
org.apache.hadoop.security.UserGroupInformation.doAs(UserGroupInformation.java:1669)
at org.apache.hadoop.mapreduce.Job.submit(Job.java:1287)
at 
org.apache.hadoop.mapreduce.lib.jobcontrol.ControlledJob.submit(ControlledJob.java:335)
 locked Monitor@496319b0 at 
org.apache.hadoop.mapreduce.lib.jobcontrol.ControlledJob.submit(ControlledJob.java:n/a)
 
at 
org.apache.hadoop.mapreduce.lib.jobcontrol.JobControl.run(JobControl.java:245)
 locked Monitor@496c673a at 
org.apache.hadoop.mapreduce.lib.jobcontrol.JobControl.run(JobControl.java:229) 
T22 is created by T1
at 
org.apache.hadoop.mapred.jobcontrol.TestJobControl.doJobControlTest(TestJobControl.java:111)
}}}
{noformat}

In the source code of org.apache.hadoop.mapreduce.JobStatus.submitJob function, 
we have the following lines:
{code}
Job job = new Job(JobID.downgrade(jobid), jobSubmitDir);
job.job.setCredentials(credentials);
{code}
It looks a bit suspicious: Job extends thread and at the end of its constructor 
it starts a new thread which creates a new instance of JobContextImpl which 
reads credentials. However, the first thread concurrently sets credentials 
after a creating the Job instance. 



  was:
I am running the test suite against a dynamic race detector called RV-Predict. 
Here is a race report that I got: 

Data race on field org.apache.hadoop.mapred.JobConf.credentials: {{{
Concurrent read in thread T327 (locks held: {})
 >  at org.apache.hadoop.mapred.JobConf.getCredentials(JobConf.java:505)
at 
org.apache.hadoop.mapreduce.task.JobContextImpl.(JobContextImpl.java:70)
at 
org.apache.hadoop.mapred.LocalJobRunner$Job.run(LocalJobRunner.java:524)
T327 is created by T22
at 
org.apache.hadoop.mapred.LocalJobRunner$Job.(LocalJobRunner.java:218)

Concurrent write in thread T22 (locks held: {Monitor@496c673a, 
Monitor@496319b0})
 >  at org.apache.hadoop.mapred.JobConf.setCredentials(JobConf.java:510)
at 
org.apache.hadoop.mapred.LocalJobRunner.submitJob(LocalJobRunner.java:787)
at 
org.apache.hadoop.mapreduce.JobSubmitter.submitJobInternal(JobSubmitter.java:241)
at org.apache.hadoop.mapreduce.Job$10.run(Job.java:1290)
at javax.security.auth.Subject.doAs(Subject.java:422)
at 
org.apache.hadoop.security.UserGroupInformation.doAs(UserGroupInformation.java:1669)
at org.apache.hadoop.mapreduce.Job.submit(Job.java:1287)
at 
org.apache.hadoop.mapreduce.lib.jobcontrol.ControlledJob.submit(ControlledJob.java:335)
 locked Monitor@496319b0 at 
org.apache.hadoop.mapreduce.lib.jobcontrol.ControlledJob.submit(ControlledJob.java:n/a)
 
at 
org.apache.hadoop.mapreduce.lib.jobcontrol.JobControl.run(JobControl.java:245)
 locked Monitor@496c673a at 
org.apache.hadoop.mapreduce.lib.jobcontrol.JobControl.run(JobControl.java:229) 
T22 is created by T1
at 
org.apache.hadoop.mapred.jobcontrol.TestJobControl.doJobControlTest(TestJobControl.java:111)
}}}

In the source code of org.apache.hadoop.mapreduce.JobStatus.submitJob function, 
we have the following lines:
Job job = new Job(JobID.downgrade(jobid), jobSubmitDir);
job.job.setCredentials(credentials);

It looks a bit suspicious: Job extends thread and at the end of its constructor 
it starts a new thread which creates a new instance of JobContextImpl which 
reads credentials. However, the first thread concurrently sets credentials 
after a creating the Job instance. 




> Data race on field org.apache.hadoop.mapred.JobConf.credentials
> ---
>
> Key: 

[jira] [Commented] (MAPREDUCE-6525) Fix test failure of TestMiniMRClientCluster.testRestart

2015-10-28 Thread Tsuyoshi Ozawa (JIRA)

[ 
https://issues.apache.org/jira/browse/MAPREDUCE-6525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14977991#comment-14977991
 ] 

Tsuyoshi Ozawa commented on MAPREDUCE-6525:
---

This is not related to this issue directly, but we should use Configuration to 
enable timeline server instead of using enableAHS.
{code}
  public MiniMRYarnCluster(String testName, int noOfNMs, boolean enableAHS) {
super(testName, 1, noOfNMs, 4, 4, enableAHS);
  }
{code}


> Fix test failure of TestMiniMRClientCluster.testRestart
> ---
>
> Key: MAPREDUCE-6525
> URL: https://issues.apache.org/jira/browse/MAPREDUCE-6525
> Project: Hadoop Map/Reduce
>  Issue Type: Bug
>  Components: test
>Reporter: Masatake Iwasaki
>Assignee: Masatake Iwasaki
>Priority: Minor
> Attachments: MAPREDUCE-6525.001.patch
>
>
> MiniMRYarnClusterAdapter#restart creates new MiniMRYarnCluster with 
> configuration of existing MiniMRYarnCluster but the address of HistoryServer 
> is properly set.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Created] (MAPREDUCE-6526) Remove usage of metrics v1 from hadoop-mapreduce

2015-10-28 Thread Akira AJISAKA (JIRA)
Akira AJISAKA created MAPREDUCE-6526:


 Summary: Remove usage of metrics v1 from hadoop-mapreduce
 Key: MAPREDUCE-6526
 URL: https://issues.apache.org/jira/browse/MAPREDUCE-6526
 Project: Hadoop Map/Reduce
  Issue Type: Improvement
Reporter: Akira AJISAKA
Priority: Blocker


LocalJobRunnerMetrics and ShuffleClientMetrics are still using metrics v1. We 
should remove these metrics or rewrite them to use metrics v2.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Assigned] (MAPREDUCE-6526) Remove usage of metrics v1 from hadoop-mapreduce

2015-10-28 Thread Akira AJISAKA (JIRA)

 [ 
https://issues.apache.org/jira/browse/MAPREDUCE-6526?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Akira AJISAKA reassigned MAPREDUCE-6526:


Assignee: Akira AJISAKA

> Remove usage of metrics v1 from hadoop-mapreduce
> 
>
> Key: MAPREDUCE-6526
> URL: https://issues.apache.org/jira/browse/MAPREDUCE-6526
> Project: Hadoop Map/Reduce
>  Issue Type: Improvement
>Reporter: Akira AJISAKA
>Assignee: Akira AJISAKA
>Priority: Blocker
>
> LocalJobRunnerMetrics and ShuffleClientMetrics are still using metrics v1. We 
> should remove these metrics or rewrite them to use metrics v2.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)