----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61542/#review182550 -----------------------------------------------------------
Ship it! Ship It! - Jonathan Hurley On Aug. 9, 2017, 7:26 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, 7:26 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/2/ > > > Testing > ------- > > UpgradeActionTest unit tests are green. > > > File Attachments > ---------------- > > AMBARI-21695-1.patch > > https://reviews.apache.org/media/uploaded/files/2017/08/09/eadc72ba-a0ce-4b99-9f57-f5230a7d165b__AMBARI-21695-1.patch > AMBARI-21695-2.patch > > https://reviews.apache.org/media/uploaded/files/2017/08/09/addb12dc-661d-480c-9526-c5c13a7a5566__AMBARI-21695-2.patch > > > Thanks, > > Sid Wagle > >