----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41926/#review112939 -----------------------------------------------------------
rerun/src/main/java/org/apache/falcon/rerun/handler/AbstractRerunHandler.java (line 72) <https://reviews.apache.org/r/41926/#comment173428> If entity is defined, the wfEngine should be the workflow endpoint defined in the cluster of this entity. If for any reason getting this wfEngine fails, shouldnt we propagate the error up to the user? Please help me understand why we ignore the exception here... - Balu Vellanki On Jan. 5, 2016, 12:42 p.m., pavan kumar kolamuri wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/41926/ > ----------------------------------------------------------- > > (Updated Jan. 5, 2016, 12:42 p.m.) > > > Review request for Falcon. > > > Bugs: FALCON-1719 > https://issues.apache.org/jira/browse/FALCON-1719 > > > Repository: falcon-git > > > Description > ------- > > Retry is not updating the state of the instance in the databases > > > Diffs > ----- > > > rerun/src/main/java/org/apache/falcon/rerun/handler/AbstractRerunConsumer.java > 582cb15 > > rerun/src/main/java/org/apache/falcon/rerun/handler/AbstractRerunHandler.java > 64c566e > rerun/src/main/java/org/apache/falcon/rerun/handler/LateRerunConsumer.java > ee31952 > rerun/src/main/java/org/apache/falcon/rerun/handler/LateRerunHandler.java > 64177a4 > rerun/src/main/java/org/apache/falcon/rerun/handler/RetryConsumer.java > 61aa3e1 > > Diff: https://reviews.apache.org/r/41926/diff/ > > > Testing > ------- > > > Thanks, > > pavan kumar kolamuri > >
