[jira] [Commented] (MAPREDUCE-4355) Add startTime to RunningJob
[ https://issues.apache.org/jira/browse/MAPREDUCE-4355?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13402465#comment-13402465 ] Arun C Murthy commented on MAPREDUCE-4355: -- bq. Arun, it might be cleaner to add RunningJob.getJobStatus() instead of adding startTime, endTime fields to RunningJob and redundantly maintaining them. +1, good point! > Add startTime to RunningJob > --- > > Key: MAPREDUCE-4355 > URL: https://issues.apache.org/jira/browse/MAPREDUCE-4355 > Project: Hadoop Map/Reduce > Issue Type: New Feature > Components: mrv1, mrv2 >Affects Versions: 1.0.3, 2.0.0-alpha >Reporter: Karthik Kambatla >Assignee: Karthik Kambatla > Fix For: 1.1.0, 2.0.1-alpha > > Attachments: MR-4355_mr1.patch, MR-4355_mr2.patch > > > To read the start-time of a particular job, one should not need to > getAllJobs() and iterate through them. > getJob(JobID) returns RunningJob, which doesn't hold the job's start time. > Hence, we need to either add getJobStatus(JobID) to the API or add startTime > to RunningJob. Doing the latter. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (MAPREDUCE-4355) Add startTime to RunningJob
[ https://issues.apache.org/jira/browse/MAPREDUCE-4355?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13402450#comment-13402450 ] Alejandro Abdelnur commented on MAPREDUCE-4355: --- reverted from trunk and branch-2 > Add startTime to RunningJob > --- > > Key: MAPREDUCE-4355 > URL: https://issues.apache.org/jira/browse/MAPREDUCE-4355 > Project: Hadoop Map/Reduce > Issue Type: New Feature > Components: mrv1, mrv2 >Affects Versions: 1.0.3, 2.0.0-alpha >Reporter: Karthik Kambatla >Assignee: Karthik Kambatla > Fix For: 1.1.0, 2.0.1-alpha > > Attachments: MR-4355_mr1.patch, MR-4355_mr2.patch > > > To read the start-time of a particular job, one should not need to > getAllJobs() and iterate through them. > getJob(JobID) returns RunningJob, which doesn't hold the job's start time. > Hence, we need to either add getJobStatus(JobID) to the API or add startTime > to RunningJob. Doing the latter. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (MAPREDUCE-4355) Add startTime to RunningJob
[ https://issues.apache.org/jira/browse/MAPREDUCE-4355?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13401941#comment-13401941 ] Alejandro Abdelnur commented on MAPREDUCE-4355: --- I'll revert the current commit WED morning. I think adding a getJobStatus() method to RunningJob would be more comprehensive that just adding startTime(). Arun, would you agree with that? > Add startTime to RunningJob > --- > > Key: MAPREDUCE-4355 > URL: https://issues.apache.org/jira/browse/MAPREDUCE-4355 > Project: Hadoop Map/Reduce > Issue Type: New Feature > Components: mrv1, mrv2 >Affects Versions: 1.0.3, 2.0.0-alpha >Reporter: Karthik Kambatla >Assignee: Karthik Kambatla > Fix For: 1.1.0, 2.0.1-alpha > > Attachments: MR-4355_mr1.patch, MR-4355_mr2.patch > > > To read the start-time of a particular job, one should not need to > getAllJobs() and iterate through them. > getJob(JobID) returns RunningJob, which doesn't hold the job's start time. > Hence, we need to either add getJobStatus(JobID) to the API or add startTime > to RunningJob. Doing the latter. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (MAPREDUCE-4355) Add startTime to RunningJob
[ https://issues.apache.org/jira/browse/MAPREDUCE-4355?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13401616#comment-13401616 ] Karthik Kambatla commented on MAPREDUCE-4355: - Arun, it might be cleaner to add RunningJob.getJobStatus() instead of adding startTime, endTime fields to RunningJob and redundantly maintaining them. Thanks. > Add startTime to RunningJob > --- > > Key: MAPREDUCE-4355 > URL: https://issues.apache.org/jira/browse/MAPREDUCE-4355 > Project: Hadoop Map/Reduce > Issue Type: New Feature > Components: mrv1, mrv2 >Affects Versions: 1.0.3, 2.0.0-alpha >Reporter: Karthik Kambatla >Assignee: Karthik Kambatla > Fix For: 1.1.0, 2.0.1-alpha > > Attachments: MR-4355_mr1.patch, MR-4355_mr2.patch > > > To read the start-time of a particular job, one should not need to > getAllJobs() and iterate through them. > getJob(JobID) returns RunningJob, which doesn't hold the job's start time. > Hence, we need to either add getJobStatus(JobID) to the API or add startTime > to RunningJob. Doing the latter. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira