----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36812/#review93136 -----------------------------------------------------------
prism/src/main/java/org/apache/falcon/resource/proxy/SchedulableEntityManagerProxy.java (line 234) <https://reviews.apache.org/r/36812/#comment147397> This is to ensure that the delete from the prism happens only when the status is succeeded. Earlier in case one of the falcon servers threw a FalconException it will be caught and handled and the entity will be deleted from the prism. - Ajay Yadava On July 25, 2015, 11:47 a.m., Ajay Yadava wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36812/ > ----------------------------------------------------------- > > (Updated July 25, 2015, 11:47 a.m.) > > > Review request for Falcon. > > > Bugs: FALCON-1329 > https://issues.apache.org/jira/browse/FALCON-1329 > > > Repository: falcon-git > > > Description > ------- > > In certain cases Idempotent behaviour of falcon is broken. Sometimes, > entities go in a zombie state. > > > Diffs > ----- > > prism/src/main/java/org/apache/falcon/resource/channel/HTTPChannel.java > 3d3a58a > > prism/src/main/java/org/apache/falcon/resource/proxy/SchedulableEntityManagerProxy.java > eb281d3 > > Diff: https://reviews.apache.org/r/36812/diff/ > > > Testing > ------- > > Manually tested in distributed mode. Unit tests or integration tests for this > are not possible. > > > Thanks, > > Ajay Yadava > >
