----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32981/#review79377 -----------------------------------------------------------
ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariServer.java <https://reviews.apache.org/r/32981/#comment128653> This will make the start process slower. Can we instead do this on-demand? And move it to its own class, especially if we decide to add more features to this checker. ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariServer.java <https://reviews.apache.org/r/32981/#comment128654> Great idea for this checker :-) version <https://reviews.apache.org/r/32981/#comment128652> Why does this need to change as opposed to getting value from pom file? - Alejandro Fernandez On April 8, 2015, 4:55 p.m., Dmytro Sen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/32981/ > ----------------------------------------------------------- > > (Updated April 8, 2015, 4:55 p.m.) > > > Review request for Ambari, Alejandro Fernandez and Sid Wagle. > > > Bugs: AMBARI-10405 > https://issues.apache.org/jira/browse/AMBARI-10405 > > > Repository: ambari > > > Description > ------- > > We have a situation where service components that have been installed and > previously working have disappeared from the API response. > /api/v1/clusters/cluster_name/services/YARN/components does NOT return > NODEMANAGER > /api/v1/clusters/cluster_name/services/STORM/components does NOT return > SUPERVISOR > Note that Ambari Server shows an ERROR when loading these service components, > but the server continues the startup sequence. > Ambari Web is even usable and you can login, but the YARN and STORM service > summary panels show invalid info (and other operations fail, since the server > thinks these service components don't even exist). > Ambari DB consistency should be check during ambari-server start > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariServer.java > 7ea96be > version 6a4bfb9 > > Diff: https://reviews.apache.org/r/32981/diff/ > > > Testing > ------- > > manual tests on corrupted DB's > > > Thanks, > > Dmytro Sen > >
