----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47026/#review132026 -----------------------------------------------------------
Fix it, then Ship it! ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog240.java (line 198) <https://reviews.apache.org/r/47026/#comment196067> I don't think we need the word "ambari" in an Ambari database table. ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog240.java (line 229) <https://reviews.apache.org/r/47026/#comment196069> Use {} in log statements ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog240.java (lines 279 - 280) <https://reviews.apache.org/r/47026/#comment196068> Create an enum for cluster_type for 'LOCAL_AMBARI' - Jonathan Hurley On May 5, 2016, 12:29 p.m., Gaurav Nagar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47026/ > ----------------------------------------------------------- > > (Updated May 5, 2016, 12:29 p.m.) > > > Review request for Ambari, DIPAYAN BHOWMICK, Jonathan Hurley, Nitiraj > Rathore, Pallav Kulshreshtha, Rohit Choudhary, and Ashwin Rajeev. > > > Bugs: AMBARI-16274 > https://issues.apache.org/jira/browse/AMBARI-16274 > > > Repository: ambari > > > Description > ------- > > Added tables for Remote Cluster to create scripts. > Added table changes to UpgradeCatalog240 > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog240.java > f85a4c7 > ambari-server/src/main/resources/Ambari-DDL-Derby-CREATE.sql f5336bc > ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql fca3be3 > ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql ce0bd84 > ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql 7fb8c31 > ambari-server/src/main/resources/Ambari-DDL-Postgres-EMBEDDED-CREATE.sql > 0f3a2c2 > ambari-server/src/main/resources/Ambari-DDL-SQLAnywhere-CREATE.sql b89389c > ambari-server/src/main/resources/Ambari-DDL-SQLServer-CREATE.sql 1107c4d > > ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog240Test.java > 7c85ba7 > > Diff: https://reviews.apache.org/r/47026/diff/ > > > Testing > ------- > > Manual Testing. > > > Thanks, > > Gaurav Nagar > >