----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32940/#review79564 -----------------------------------------------------------
Ship it! The patch looks fine to me. I'm not really familiar with the ambari-web javascript code, so it may not be a bad idea to ask somebody with Ambari UI experience just to double-check it. - Robert Nettleton On April 8, 2015, 12:22 a.m., Billie Rinaldi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/32940/ > ----------------------------------------------------------- > > (Updated April 8, 2015, 12:22 a.m.) > > > Review request for Ambari, John Speidel and Robert Nettleton. > > > Bugs: AMBARI-10382 > https://issues.apache.org/jira/browse/AMBARI-10382 > > > Repository: ambari > > > Description > ------- > > AMBARI-10382 Add Accumulo to BlueprintConfigurationProcessor and fix other HA > name service substitutions > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessor.java > 04027af > > ambari-server/src/test/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessorTest.java > d8794ac > > ambari-web/app/controllers/main/admin/highAvailability/nameNode/step3_controller.js > e06893e > ambari-web/app/controllers/wizard/step7_controller.js b5d84dc > > Diff: https://reviews.apache.org/r/32940/diff/ > > > Testing > ------- > > > Thanks, > > Billie Rinaldi > >
