[
https://issues.apache.org/jira/browse/OOZIE-818?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13273103#comment-13273103
]
[email protected] commented on OOZIE-818:
-----------------------------------------------------
bq. On 2012-05-10 16:44:25, Alejandro Abdelnur wrote:
bq. > I like the approach, minor corrections through out the patch.
bq. >
bq. > I've applied the patch and run into 2 issues:
bq. >
bq. > * NIT: patch is done at trunk/core level, it should be done a trunk/
level
bq. >
bq. > * Test errors:
bq. >
bq. >
bq. > Results :
bq. >
bq. > Failed tests:
testCoordActionDelete(org.apache.oozie.command.coord.TestCoordChangeXCommand):
Error code should be E1022
bq. >
bq. > Tests in error:
bq. >
testCoordChangePauseTime(org.apache.oozie.command.coord.TestCoordChangeXCommand):
E0603: SQL error in operation [java.lang.ClassCastException:
[Ljava.lang.Object; cannot be cast to java.lang.String], {1}
bq. >
bq. > Tests run: 4, Failures: 1, Errors: 1, Skipped: 0
bq. >
bq. >
bq. > Once this is taken care it is should ready for +1
bq. >
bq. > Thanks for all the work.
Test error was because of some typo. Fixed
Removed all trailing spaces
bq. On 2012-05-10 16:44:25, Alejandro Abdelnur wrote:
bq. >
trunk/core/src/main/java/org/apache/oozie/command/coord/CoordChangeXCommand.java,
line 151
bq. > <https://reviews.apache.org/r/4842/diff/5/?file=108315#file108315line151>
bq. >
bq. > getLastActionTime() seems abetter name, 'actual' may give some
connotation to when the action is running and actually this is the nominal
time of the action.
Changed. The reason i made it getActualLastActionTime is because
coordJob.getLastActionTime() actually points to one instance after the last
action time. Its confusing!
- shwethags
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4842/#review7775
-----------------------------------------------------------
On 2012-05-10 09:09:35, shwethags wrote:
bq.
bq. -----------------------------------------------------------
bq. This is an automatically generated e-mail. To reply, visit:
bq. https://reviews.apache.org/r/4842/
bq. -----------------------------------------------------------
bq.
bq. (Updated 2012-05-10 09:09:35)
bq.
bq.
bq. Review request for oozie.
bq.
bq.
bq. Summary
bq. -------
bq.
bq. 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
bq.
bq.
bq. This addresses bug OOZIE-818.
bq. https://issues.apache.org/jira/browse/OOZIE-818
bq.
bq.
bq. Diffs
bq. -----
bq.
bq. trunk/core/src/main/java/org/apache/oozie/CoordinatorActionBean.java
1335944
bq. trunk/core/src/main/java/org/apache/oozie/ErrorCode.java 1335944
bq.
trunk/core/src/main/java/org/apache/oozie/command/coord/CoordChangeXCommand.java
1335944
bq.
trunk/core/src/main/java/org/apache/oozie/executor/jpa/CoordActionRemoveJPAExecutor.java
1335944
bq.
trunk/core/src/test/java/org/apache/oozie/command/coord/TestCoordChangeXCommand.java
1335944
bq.
trunk/core/src/test/java/org/apache/oozie/executor/jpa/TestCoordActionRemoveJPAExecutor.java
1335944
bq.
bq. Diff: https://reviews.apache.org/r/4842/diff
bq.
bq.
bq. Testing
bq. -------
bq.
bq. UT - TestCoordChangeXCommand
bq.
bq.
bq. Thanks,
bq.
bq. shwethags
bq.
bq.
> CoordChangeXCommand deletes past coordinator actions
> ----------------------------------------------------
>
> Key: OOZIE-818
> URL: https://issues.apache.org/jira/browse/OOZIE-818
> Project: Oozie
> Issue Type: Bug
> Reporter: Shwetha G S
> Assignee: Shwetha G S
> Priority: Critical
>
> 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
--
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
