----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61542/#review182532 -----------------------------------------------------------
Ship it! Ship It! - Alejandro Fernandez On Aug. 9, 2017, 9:41 p.m., Sid Wagle wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/61542/ > ----------------------------------------------------------- > > (Updated Aug. 9, 2017, 9:41 p.m.) > > > Review request for Ambari, Alejandro Fernandez, Dmytro Sen, Jonathan Hurley, > and Sumit Mohanty. > > > Bugs: AMBARI-21695 > https://issues.apache.org/jira/browse/AMBARI-21695 > > > Repository: ambari > > > Description > ------- > > org.apache.ambari.server.metadata.CachedRoleCommandOrderProvider#getRoleCommandOrder(org.apache.ambari.server.state.Cluster) > > This method caches the RCO for the stack which is never cleared. Since the BI > stack does not have Accumulo service the RCO is not honored resulting in > ACCUMULO service not starting after the express upgrade when ACCUMULO is > added. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/metadata/CachedRoleCommandOrderProvider.java > 49abc66 > > ambari-server/src/main/java/org/apache/ambari/server/serveraction/upgrades/FinalizeUpgradeAction.java > d531460 > ambari-server/src/main/resources/scripts/configs.py 40a8324 > > > Diff: https://reviews.apache.org/r/61542/diff/1/ > > > Testing > ------- > > UpgradeActionTest unit tests are green. > > > Thanks, > > Sid Wagle > >