----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52999/#review153454 -----------------------------------------------------------
Ship it! Ship It! - Sid Wagle On Oct. 20, 2016, 8:42 p.m., Vitalyi Brodetskyi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/52999/ > ----------------------------------------------------------- > > (Updated Oct. 20, 2016, 8:42 p.m.) > > > Review request for Ambari, Dmytro Sen, Sumit Mohanty, and Sid Wagle. > > > Bugs: AMBARI-18631 > https://issues.apache.org/jira/browse/AMBARI-18631 > > > Repository: ambari > > > Description > ------- > > Currently the database consistency check runs as a separate process prior > Ambari process. The database consistency checker load various modules needed > for performing the validations. (e.g. load stack definitions to be able to > compare service configs from stack with configs from db). > Once database consistency checker completed Ambari server is started. Ambari > server beside others loads the same modules as database consistency checker. > This double initialisation adds time to the ambari server startup time which > could be reduced if the database consistency check is moved into the ambari > server process. > Moreover the database consistency check may check at the beginning if the > database is empty and perform the checks only if there is data in the > database. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/checks/DatabaseConsistencyCheckHelper.java > 2d91eca > > ambari-server/src/main/java/org/apache/ambari/server/checks/DatabaseConsistencyChecker.java > 535d74f > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariServer.java > 4030ef9 > ambari-server/src/main/python/ambari_server_main.py 57ec58d > > ambari-server/src/test/java/org/apache/ambari/server/checks/DatabaseConsistencyCheckHelperTest.java > 4663310 > > ambari-server/src/test/java/org/apache/ambari/server/controller/AmbariServerTest.java > 54f6147 > > Diff: https://reviews.apache.org/r/52999/diff/ > > > Testing > ------- > > UT will be added and checked after patch draft review > > > Thanks, > > Vitalyi Brodetskyi > >
