----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30116/#review72133 -----------------------------------------------------------
common/src/test/resources/config/process/process-duplicate-name.xml <https://reviews.apache.org/r/30116/#comment118147> Do you need another test xml for this? Just set process name after unmarshal? prism/src/main/java/org/apache/falcon/resource/AbstractEntityManager.java <https://reviews.apache.org/r/30116/#comment118148> also include entity type. You can use entity.toShortString() prism/src/main/java/org/apache/falcon/resource/AbstractEntityManager.java <https://reviews.apache.org/r/30116/#comment118149> why release here? Won't finally take care of it? prism/src/main/java/org/apache/falcon/resource/AbstractEntityManager.java <https://reviews.apache.org/r/30116/#comment118150> not in submit, schedule right? prism/src/main/java/org/apache/falcon/resource/AbstractEntityManager.java <https://reviews.apache.org/r/30116/#comment118151> why catch Throwable? webapp/src/test/java/org/apache/falcon/resource/EntityManagerJerseyIT.java <https://reviews.apache.org/r/30116/#comment118153> Add Thread.join() - Shwetha GS On Feb. 11, 2015, 10:15 a.m., Suhas Vasu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/30116/ > ----------------------------------------------------------- > > (Updated Feb. 11, 2015, 10:15 a.m.) > > > Review request for Falcon. > > > Repository: falcon-git > > > Description > ------- > > In distributed mode, when parallel update command are issued for same entity, > they are not syncronized. > This leads to multiple coordinators being spawned in Oozie. > > We need to ensure that the update method is syncronized. > > > Diffs > ----- > > common/src/main/java/org/apache/falcon/entity/lock/MemoryLocks.java > PRE-CREATION > common/src/test/java/org/apache/falcon/entity/lock/MemoryLocksTest.java > PRE-CREATION > common/src/test/resources/config/process/process-duplicate-name.xml > PRE-CREATION > prism/src/main/java/org/apache/falcon/resource/AbstractEntityManager.java > 0d34ef3 > webapp/src/test/java/org/apache/falcon/cli/FalconCLIIT.java 118003f > webapp/src/test/java/org/apache/falcon/resource/EntityManagerJerseyIT.java > 40f8e04 > > Diff: https://reviews.apache.org/r/30116/diff/ > > > Testing > ------- > > UT's were added and were successful > > > Thanks, > > Suhas Vasu > >
