-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/2386/
-----------------------------------------------------------

(Updated 2011-10-14 22:16:23.602791)


Review request for oozie, Mohammad Islam and Angelo K. Huang.


Changes
-------

Adding comment to rationalize the change. Reverting redundant wait added to two 
additional cases in the first diff.


Summary
-------

Test case 
TestStatusTransitService::testCoordStatusTransitServiceKilledByUser1() fails 
due to race condition between status service and the check assertEquals(true, 
coordJob.isPending()). The kill action has already been completed by the time 
assertion is checked.


This addresses bug OOZIE-581.
    https://issues.apache.org/jira/browse/OOZIE-581


Diffs (updated)
-----

  
http://svn.apache.org/repos/asf/incubator/oozie/trunk/core/src/test/java/org/apache/oozie/service/TestStatusTransitService.java
 1181872 

Diff: https://reviews.apache.org/r/2386/diff


Testing
-------


Thanks,

Prakhar

Reply via email to