You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@oozie.apache.org by "Bowen Zhang (JIRA)" <ji...@apache.org> on 2014/05/01 07:31:15 UTC
[jira] [Commented] (OOZIE-1818) CoordMaterializeTransitionXCommand
verifyPrecondition doesn't verify current time
[ https://issues.apache.org/jira/browse/OOZIE-1818?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13986359#comment-13986359 ]
Bowen Zhang commented on OOZIE-1818:
------------------------------------
I actually don't see any differences except for the error message.
> CoordMaterializeTransitionXCommand verifyPrecondition doesn't verify current time
> ---------------------------------------------------------------------------------
>
> Key: OOZIE-1818
> URL: https://issues.apache.org/jira/browse/OOZIE-1818
> Project: Oozie
> Issue Type: Bug
> Reporter: Purshotam Shah
>
> {code}
> Timestamp startTime = coordJob.getNextMaterializedTimestamp();
> if (startTime == null) {
> startTime = coordJob.getStartTimestamp();
> if (startTime.after(new Timestamp(System.currentTimeMillis() + lookAheadWindow * 1000))) {
> throw new PreconditionException(ErrorCode.E1100, "CoordMaterializeTransitionXCommand for jobId="
> + jobId + " job's start time is not reached yet - nothing to materialize");
> }
> }
> {code}
> It should be should be
> {code}
> Timestamp matTime = coordJob.getNextMaterializedTimestamp();
> if (matTime == null) {
> matTime = coordJob.getStartTimestamp();
> }
> if (matTime.after(new Timestamp(System.currentTimeMillis() + lookAheadWindow * 1000))) {
> throw new PreconditionException(ErrorCode.E1100, "CoordMaterializeTransitionXCommand for jobId="
> + jobId + " Request is for future time");
> }
> {code}
--
This message was sent by Atlassian JIRA
(v6.2#6252)