Review Board 1.7.22


OOZIE-818: CoordChangeXCommand deletes past coordinator actions

Review Request #4842 - Created April 23, 2012 and submitted

shwethags
trunk
OOZIE-818
Reviewers
oozie
oozie
CoordChangeXCommand.processLookaheadActions() doesn't use coord timeunit while calculating the action times. So, if the frequency is non-minutes(hours, days, months), setting pause time can delete past coord actions
UT - TestCoordChangeXCommand
Review request changed
Updated (May 12, 2012, 11:36 a.m.)
fixed testcase failure - Reverted query in CoordinatorActionBean
Ship it!
Posted (May 16, 2012, 5:27 p.m.)
Shwetha, gr8 job, thxs for your patience! it took me a while because I wanted to ensure we would not run into a race-condition/deadlock while deleting coord actions. it is not possible because coordinator commands lock on the coordinator job id.
I'll commit this to trunk, would you please upload the final patch to JIRA and grant license rights?