----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38001/#review97589 -----------------------------------------------------------
I would expect that you have done fairly extensive functional testing of this patch prior to submitting it. Please specify the functional testing that was done. Also, please include the output from the unit tests. - John Speidel On Sept. 1, 2015, 11:53 a.m., Andrew Onischuk wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/38001/ > ----------------------------------------------------------- > > (Updated Sept. 1, 2015, 11:53 a.m.) > > > Review request for Ambari, John Speidel and Robert Nettleton. > > > Bugs: AMBARI-12538 > https://issues.apache.org/jira/browse/AMBARI-12538 > > > Repository: ambari > > > Description > ------- > > Attempting to access the Ambari Dashboard results in an HTTP 500 Error after > changing cluster name and restarting the Ambari server > > Refer to <https://issues.apache.org/jira/browse/AMBARI-12538> for more info. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ExportBlueprintRequest.java > 8b13826 > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ProvisionClusterRequest.java > 0150141 > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ScaleClusterRequest.java > a6995ed > > ambari-server/src/main/java/org/apache/ambari/server/orm/entities/TopologyRequestEntity.java > 8535ed8 > > ambari-server/src/main/java/org/apache/ambari/server/topology/AmbariContext.java > 40717cc > > ambari-server/src/main/java/org/apache/ambari/server/topology/ClusterTopology.java > 9a9929b > > ambari-server/src/main/java/org/apache/ambari/server/topology/ClusterTopologyImpl.java > 9dbd197 > > ambari-server/src/main/java/org/apache/ambari/server/topology/HostRequest.java > cc82a63 > > ambari-server/src/main/java/org/apache/ambari/server/topology/LogicalRequest.java > b7f95cf > > ambari-server/src/main/java/org/apache/ambari/server/topology/PersistedState.java > dbf6735 > > ambari-server/src/main/java/org/apache/ambari/server/topology/PersistedStateImpl.java > 894bc6d > > ambari-server/src/main/java/org/apache/ambari/server/topology/TopologyManager.java > 5f3bb9d > > ambari-server/src/main/java/org/apache/ambari/server/topology/TopologyRequest.java > d102c21 > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog212.java > 02df181 > ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql 9828eeb > ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql 22f7493 > ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql 9fc1239 > ambari-server/src/main/resources/Ambari-DDL-Postgres-EMBEDDED-CREATE.sql > 0c966ca > ambari-server/src/main/resources/Ambari-DDL-SQLAnywhere-CREATE.sql 9f21160 > ambari-server/src/main/resources/Ambari-DDL-SQLServer-CREATE.sql 58760ff > > ambari-server/src/test/java/org/apache/ambari/server/topology/ClusterTopologyImplTest.java > bbea96c > > ambari-server/src/test/java/org/apache/ambari/server/topology/TopologyManagerTest.java > 64dfb28 > > ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog212Test.java > 7a394ca > > Diff: https://reviews.apache.org/r/38001/diff/ > > > Testing > ------- > > mvn clean test > > > Thanks, > > Andrew Onischuk > >
