----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37138/#review94486 -----------------------------------------------------------
common/src/main/java/org/apache/falcon/entity/store/ConfigurationStore.java (line 174) <https://reviews.apache.org/r/37138/#comment149112> onReload being called twice? Once here and once later in the for loop. common/src/main/java/org/apache/falcon/entity/store/ConfigurationStore.java (line 191) <https://reviews.apache.org/r/37138/#comment149113> Shouldn't this check be made before invoking onReload, as a fail-fast mechanism? - Pallavi Rao On Aug. 6, 2015, 8:14 p.m., sandeep samudrala wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/37138/ > ----------------------------------------------------------- > > (Updated Aug. 6, 2015, 8:14 p.m.) > > > Review request for Falcon, Ajay Yadava and Srikanth Sundarrajan. > > > Bugs: FALCON-1368 > https://issues.apache.org/jira/browse/FALCON-1368 > > > Repository: falcon-git > > > Description > ------- > > I have made changes to make configuration store loading of entities parallel > for each entity type in the respective order ( CLUSTER, FEED, PROCESS) > > > Diffs > ----- > > common/src/main/java/org/apache/falcon/entity/store/ConfigurationStore.java > 7b53ebb > > Diff: https://reviews.apache.org/r/37138/diff/ > > > Testing > ------- > > This is a sample of how it looks and there is a good dip in the restart time > of falcon server. > > 2015-08-06 00:59:04,687 INFO - [pool-5-thread-95:] ~ Restored configuration > FEED/ipfeed34 > 2015-08-06 00:59:04,690 INFO - [pool-5-thread-94:] ~ Restored configuration > FEED/ipfeed18 > 2015-08-06 00:59:04,694 INFO - [pool-5-thread-93:] ~ Restored configuration > FEED/ipfeed10 > > > Thanks, > > sandeep samudrala > >
