hadoop-mapreduce-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "zhihai xu (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (MAPREDUCE-6339) Job history file is not flushed correctly because isTimerActive flag is not set true when flushTimerTask is scheduled.
Date Tue, 28 Apr 2015 07:18:07 GMT

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

zhihai xu commented on MAPREDUCE-6339:
--------------------------------------

thanks [~devaraj.k] for the review, I am not sure what cause the checkstyle issue. I uploaded
a new patch, which fix 80-column-limit violations at line 259 of JobHistoryEventHandler.java.

> Job history file is not flushed correctly because isTimerActive flag is not set true
when flushTimerTask is scheduled.
> ----------------------------------------------------------------------------------------------------------------------
>
>                 Key: MAPREDUCE-6339
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-6339
>             Project: Hadoop Map/Reduce
>          Issue Type: Bug
>          Components: mrv2
>    Affects Versions: 2.7.0
>            Reporter: zhihai xu
>            Assignee: zhihai xu
>            Priority: Critical
>         Attachments: MAPREDUCE-6339.000.patch, MAPREDUCE-6339.001.patch
>
>
> Job history file is not flushed correctly because isTimerActive flag is not set true
when flushTimerTask is scheduled.
> It looks like we should set isTimerActive to true when flushTimerTask is scheduled. Otherwise
if a new qualified event comes before the current flush timer is expired,  flushTimerTask
will be canceled and rescheduled.
> Also I didn't find any code which set isTimerActive flag to true, So isTimerActive is
useless in current code.



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

Mime
View raw message