----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37431/#review95421 -----------------------------------------------------------
prism/src/main/java/org/apache/falcon/resource/AbstractEntityManager.java (line 341) <https://reviews.apache.org/r/37431/#comment150354> Why not just put a null check here and do away with all other changes (and enabling asserts)? This will achieve the same effect, right? src/bin/service-start.sh (line 46) <https://reviews.apache.org/r/37431/#comment150353> Enabling assertion by default to avoid a bug is not a good idea. This will be applicable to every installation of falcon. - Ajay Yadava On Aug. 13, 2015, 5:36 a.m., pavan kumar kolamuri wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/37431/ > ----------------------------------------------------------- > > (Updated Aug. 13, 2015, 5:36 a.m.) > > > Review request for Falcon. > > > Bugs: FALCON-1331 > https://issues.apache.org/jira/browse/FALCON-1331 > > > Repository: falcon-git > > > Description > ------- > > When a user is trying to update a process update failed saying null pointer > exception and further updates are not possible since lock was not released > for that entity. > > > Diffs > ----- > > prism/src/main/java/org/apache/falcon/resource/AbstractEntityManager.java > 43461c8 > src/bin/service-start.sh 4a030de > > Diff: https://reviews.apache.org/r/37431/diff/ > > > Testing > ------- > > > Thanks, > > pavan kumar kolamuri > >
