Return-Path: X-Original-To: apmail-ambari-dev-archive@www.apache.org Delivered-To: apmail-ambari-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id D130417AC1 for ; Fri, 7 Nov 2014 00:07:36 +0000 (UTC) Received: (qmail 15658 invoked by uid 500); 7 Nov 2014 00:07:36 -0000 Delivered-To: apmail-ambari-dev-archive@ambari.apache.org Received: (qmail 15619 invoked by uid 500); 7 Nov 2014 00:07:36 -0000 Mailing-List: contact dev-help@ambari.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@ambari.apache.org Delivered-To: mailing list dev@ambari.apache.org Received: (qmail 15601 invoked by uid 99); 7 Nov 2014 00:07:36 -0000 Received: from reviews-vm.apache.org (HELO reviews.apache.org) (140.211.11.40) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 07 Nov 2014 00:07:36 +0000 Received: from reviews.apache.org (localhost [127.0.0.1]) by reviews.apache.org (Postfix) with ESMTP id E36241DFCDB; Fri, 7 Nov 2014 00:07:39 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============4346176673811350719==" MIME-Version: 1.0 Subject: Re: Review Request 27607: Rolling Upgrade Bootstrap - Create new tables and populate during the installation of a new cluster From: "Jonathan Hurley" To: "Nate Cole" , "Dmitro Lisnichenko" , "Yurii Shylov" , "Sid Wagle" , "Jonathan Hurley" Cc: "Ambari" , "Alejandro Fernandez" Date: Fri, 07 Nov 2014 00:07:39 -0000 Message-ID: <20141107000739.12275.46403@reviews.apache.org> X-ReviewBoard-URL: https://reviews.apache.org Auto-Submitted: auto-generated Sender: "Jonathan Hurley" X-ReviewGroup: Ambari X-ReviewRequest-URL: https://reviews.apache.org/r/27607/ X-Sender: "Jonathan Hurley" References: <20141106220349.10455.66626@reviews.apache.org> In-Reply-To: <20141106220349.10455.66626@reviews.apache.org> Reply-To: "Jonathan Hurley" X-ReviewRequest-Repository: ambari --===============4346176673811350719== MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27607/#review60277 ----------------------------------------------------------- Ship it! Any chance you feel like changing all of the SQL files to be readable for all of the other tables? :) ambari-server/src/main/java/org/apache/ambari/server/orm/entities/ClusterEntity.java Formatting is off here. - Jonathan Hurley On Nov. 6, 2014, 5:03 p.m., Alejandro Fernandez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/27607/ > ----------------------------------------------------------- > > (Updated Nov. 6, 2014, 5:03 p.m.) > > > Review request for Ambari, Dmitro Lisnichenko, Jonathan Hurley, Nate Cole, Sid Wagle, and Yurii Shylov. > > > Bugs: AMBARI-8148 > https://issues.apache.org/jira/browse/AMBARI-8148 > > > Repository: ambari > > > Description > ------- > > 1. Create new tables and populate during the installation of a new cluster. > > *cluster_version* table > Column | Type | Keys | Example > ----------|------------|------|--------------------------------------------- > id |BIGINT |PK |1 > cluster_id|BIGINT |FK |2 > stack |VARCHAR(255)| |HDP-2.2 > version |VARCHAR(255)| |HDP-2-2.0.1-885 > state |VARCHAR(32) | |INSTALLED, CURRENT, UPGRADING, UPGRADE_FAILED > start_time|DATETIME | |2014-10-31 14:00:00 > end_time |DATETIME | |2014-10-31 15:00:00 > user_name |VARCHAR(32) | |admin > > *hostcomponentstate* table > Add column upgrade_state VARCHAR(32) ("NONE" is the default, "UPGRADING", "UPGRADE_FAILED") > > *host_version* table > Column |Type |Keys |Example > ---------|------------|-------------|---------------------------------- > id |BIGINT |PK |1 > host_name|VARCHAR(255)|FK hosts.name|c6401.ambari.apache.org > stack |VARCHAR(255)| |HDP-2.2 > version |VARCHAR(255)| |2.2.0.1-885 > state |VARCHAR(32) | |INSTALLED, CURRENT, NOT_INSTALLED > > > Diffs > ----- > > ambari-server/src/main/java/org/apache/ambari/server/orm/entities/ClusterEntity.java b030ea1 > ambari-server/src/main/java/org/apache/ambari/server/orm/entities/ClusterVersionEntity.java PRE-CREATION > ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostComponentStateEntity.java 7dce204 > ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostEntity.java a71fddb > ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostVersionEntity.java PRE-CREATION > ambari-server/src/main/java/org/apache/ambari/server/state/ClusterVersionState.java PRE-CREATION > ambari-server/src/main/java/org/apache/ambari/server/state/HostVersionState.java PRE-CREATION > ambari-server/src/main/java/org/apache/ambari/server/state/ServiceComponentHost.java 3718556 > ambari-server/src/main/java/org/apache/ambari/server/state/svccomphost/ServiceComponentHostImpl.java bc97936 > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog200.java 14c385d > ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql 9054b3f > ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql abe8b39 > ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql 0bea067 > ambari-server/src/main/resources/Ambari-DDL-Postgres-EMBEDDED-CREATE.sql ebc8f5c > ambari-server/src/main/resources/META-INF/persistence.xml 2d8fedb > ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog200Test.java 16b4d44 > ambari-server/src/test/python/TestAmbariServer.py c3651eb > > Diff: https://reviews.apache.org/r/27607/diff/ > > > Testing > ------- > > Ran unit tests > > ambari-server unit tests > ---------------------------------------------------------------------- > Total run:684 > Total errors:0 > Total failures:0 > OK > > > Thanks, > > Alejandro Fernandez > > --===============4346176673811350719==--