This is an automated email from the ASF dual-hosted git repository.

pvary pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hive.git


The following commit(s) were added to refs/heads/master by this push:
     new ec88021  HIVE-23697 : Fix errors in the metastore upgrade script 
(Aasha Medhi reviewed by Peter Vary)
ec88021 is described below

commit ec88021b03b69ca6a1d4232f4c1c72f06d5eec19
Author: aasha <aasha.medhi2...@gmail.com>
AuthorDate: Fri Jun 19 01:39:47 2020 +0530

    HIVE-23697 : Fix errors in the metastore upgrade script (Aasha Medhi 
reviewed by Peter Vary)
    
    Closes #1113
---
 .../hadoop/hive/ql/parse/TestScheduledReplicationScenarios.java     | 1 +
 .../metastore-server/src/main/sql/mssql/hive-schema-4.0.0.mssql.sql | 4 ++--
 .../src/main/sql/mssql/upgrade-2.3.0-to-3.0.0.mssql.sql             | 2 ++
 .../src/main/sql/mssql/upgrade-3.2.0-to-4.0.0.mssql.sql             | 4 ++--
 .../metastore-server/src/main/sql/mysql/hive-schema-4.0.0.mysql.sql | 4 ++--
 .../src/main/sql/mysql/upgrade-3.2.0-to-4.0.0.mysql.sql             | 6 +++---
 .../src/main/sql/oracle/hive-schema-4.0.0.oracle.sql                | 2 +-
 .../src/main/sql/oracle/upgrade-3.2.0-to-4.0.0.oracle.sql           | 2 +-
 .../apache/hadoop/hive/metastore/dbinstall/rules/DatabaseRule.java  | 4 ++--
 .../org/apache/hadoop/hive/metastore/dbinstall/rules/Mysql.java     | 2 +-
 10 files changed, 17 insertions(+), 14 deletions(-)

diff --git 
a/itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/parse/TestScheduledReplicationScenarios.java
 
b/itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/parse/TestScheduledReplicationScenarios.java
index 2e1e5e0..017607e 100644
--- 
a/itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/parse/TestScheduledReplicationScenarios.java
+++ 
b/itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/parse/TestScheduledReplicationScenarios.java
@@ -180,6 +180,7 @@ public class TestScheduledReplicationScenarios extends 
BaseReplicationScenariosA
   }
 
   @Test
+  @Ignore("HIVE-23395")
   public void testExternalTablesReplLoadBootstrapIncr() throws Throwable {
     // Bootstrap
     String withClause = " WITH('" + 
HiveConf.ConfVars.REPL_EXTERNAL_TABLE_BASE_DIR.varname
diff --git 
a/standalone-metastore/metastore-server/src/main/sql/mssql/hive-schema-4.0.0.mssql.sql
 
b/standalone-metastore/metastore-server/src/main/sql/mssql/hive-schema-4.0.0.mssql.sql
index e5c4582..cba72cd 100644
--- 
a/standalone-metastore/metastore-server/src/main/sql/mssql/hive-schema-4.0.0.mssql.sql
+++ 
b/standalone-metastore/metastore-server/src/main/sql/mssql/hive-schema-4.0.0.mssql.sql
@@ -1320,7 +1320,7 @@ CREATE TABLE "SCHEDULED_EXECUTIONS" (
 CREATE INDEX IDX_SCHEDULED_EX_LAST_UPDATE ON "SCHEDULED_EXECUTIONS" 
("LAST_UPDATE_TIME");
 CREATE INDEX IDX_SCHEDULED_EX_SQ_ID ON "SCHEDULED_EXECUTIONS" 
("SCHEDULED_QUERY_ID");
 
---HIVE-23516
+-- HIVE-23516
 CREATE TABLE "REPLICATION_METRICS" (
   "RM_SCHEDULED_EXECUTION_ID" bigint PRIMARY KEY,
   "RM_POLICY" varchar(256) NOT NULL,
@@ -1329,7 +1329,7 @@ CREATE TABLE "REPLICATION_METRICS" (
   "RM_PROGRESS" varchar(max)
 );
 
---Create indexes for the replication metrics table
+-- Create indexes for the replication metrics table
 CREATE INDEX "POLICY_IDX" ON "REPLICATION_METRICS" ("RM_POLICY");
 CREATE INDEX "DUMP_IDX" ON "REPLICATION_METRICS" ("RM_DUMP_EXECUTION_ID");
 
diff --git 
a/standalone-metastore/metastore-server/src/main/sql/mssql/upgrade-2.3.0-to-3.0.0.mssql.sql
 
b/standalone-metastore/metastore-server/src/main/sql/mssql/upgrade-2.3.0-to-3.0.0.mssql.sql
index 53edaa2..e6de9a3 100644
--- 
a/standalone-metastore/metastore-server/src/main/sql/mssql/upgrade-2.3.0-to-3.0.0.mssql.sql
+++ 
b/standalone-metastore/metastore-server/src/main/sql/mssql/upgrade-2.3.0-to-3.0.0.mssql.sql
@@ -357,7 +357,9 @@ UPDATE COMPLETED_TXN_COMPONENTS SET CTC_WRITEID = CTC_TXNID;
 
 -- HIVE-23211: Fix metastore schema differences between init scripts, and 
upgrade scripts
 -- Not updating possible NULL values, since if NULLs existing in this table, 
the upgrade should fail
+DROP INDEX TC_TXNID_INDEX ON TXN_COMPONENTS;
 ALTER TABLE TXN_COMPONENTS ALTER COLUMN TC_TXNID bigint NOT NULL;
+CREATE INDEX TC_TXNID_INDEX ON TXN_COMPONENTS (TC_TXNID);
 ALTER TABLE COMPLETED_TXN_COMPONENTS ALTER COLUMN CTC_TXNID bigint NOT NULL;
 
 -- These lines need to be last.  Insert any changes above.
diff --git 
a/standalone-metastore/metastore-server/src/main/sql/mssql/upgrade-3.2.0-to-4.0.0.mssql.sql
 
b/standalone-metastore/metastore-server/src/main/sql/mssql/upgrade-3.2.0-to-4.0.0.mssql.sql
index 33f4df8..fa1250c 100644
--- 
a/standalone-metastore/metastore-server/src/main/sql/mssql/upgrade-3.2.0-to-4.0.0.mssql.sql
+++ 
b/standalone-metastore/metastore-server/src/main/sql/mssql/upgrade-3.2.0-to-4.0.0.mssql.sql
@@ -123,7 +123,7 @@ Exec sp_rename 'TXN_LOCK_TBL.NTXN_NEXT', 'TXN_LOCK', 
'COLUMN';
 ALTER TABLE TXN_COMPONENTS WITH CHECK ADD FOREIGN KEY(TC_TXNID) REFERENCES 
TXNS (TXN_ID);
 CREATE INDEX TC_TXNID_INDEX ON TXN_COMPONENTS (TC_TXNID);
 
---HIVE-23516
+-- HIVE-23516
 CREATE TABLE "REPLICATION_METRICS" (
   "RM_SCHEDULED_EXECUTION_ID" bigint PRIMARY KEY,
   "RM_POLICY" varchar(256) NOT NULL,
@@ -132,7 +132,7 @@ CREATE TABLE "REPLICATION_METRICS" (
   "RM_PROGRESS" varchar(max)
 );
 
---Create indexes for the replication metrics table
+-- Create indexes for the replication metrics table
 CREATE INDEX "POLICY_IDX" ON "REPLICATION_METRICS" ("RM_POLICY");
 CREATE INDEX "DUMP_IDX" ON "REPLICATION_METRICS" ("RM_DUMP_EXECUTION_ID");
 
diff --git 
a/standalone-metastore/metastore-server/src/main/sql/mysql/hive-schema-4.0.0.mysql.sql
 
b/standalone-metastore/metastore-server/src/main/sql/mysql/hive-schema-4.0.0.mysql.sql
index e5eb9a1..e036b31 100644
--- 
a/standalone-metastore/metastore-server/src/main/sql/mysql/hive-schema-4.0.0.mysql.sql
+++ 
b/standalone-metastore/metastore-server/src/main/sql/mysql/hive-schema-4.0.0.mysql.sql
@@ -1244,7 +1244,7 @@ CREATE INDEX IDX_SCHEDULED_EXECUTIONS_LAST_UPDATE_TIME ON 
SCHEDULED_EXECUTIONS (
 CREATE INDEX IDX_SCHEDULED_EXECUTIONS_SCHEDULED_QUERY_ID ON 
SCHEDULED_EXECUTIONS (SCHEDULED_QUERY_ID);
 CREATE UNIQUE INDEX UNIQUE_SCHEDULED_EXECUTIONS_ID ON SCHEDULED_EXECUTIONS 
(SCHEDULED_EXECUTION_ID);
 
---Create table replication metrics
+-- Create table replication metrics
 CREATE TABLE REPLICATION_METRICS (
   RM_SCHEDULED_EXECUTION_ID bigint NOT NULL,
   RM_POLICY varchar(256) NOT NULL,
@@ -1254,7 +1254,7 @@ CREATE TABLE REPLICATION_METRICS (
   PRIMARY KEY(RM_SCHEDULED_EXECUTION_ID)
 );
 
---Create indexes for the replication metrics table
+-- Create indexes for the replication metrics table
 CREATE INDEX POLICY_IDX ON REPLICATION_METRICS (RM_POLICY);
 CREATE INDEX DUMP_IDX ON REPLICATION_METRICS (RM_DUMP_EXECUTION_ID);
 
diff --git 
a/standalone-metastore/metastore-server/src/main/sql/mysql/upgrade-3.2.0-to-4.0.0.mysql.sql
 
b/standalone-metastore/metastore-server/src/main/sql/mysql/upgrade-3.2.0-to-4.0.0.mysql.sql
index f534beb..c1c888b 100644
--- 
a/standalone-metastore/metastore-server/src/main/sql/mysql/upgrade-3.2.0-to-4.0.0.mysql.sql
+++ 
b/standalone-metastore/metastore-server/src/main/sql/mysql/upgrade-3.2.0-to-4.0.0.mysql.sql
@@ -92,9 +92,9 @@ PREPARE stmt FROM @s;
 EXECUTE stmt;
 DEALLOCATE PREPARE stmt;
 RENAME TABLE NEXT_TXN_ID TO TXN_LOCK_TBL;
-ALTER TABLE TXN_LOCK_TBL RENAME COLUMN NTXN_NEXT TO TXN_LOCK;
+ALTER TABLE TXN_LOCK_TBL CHANGE NTXN_NEXT TXN_LOCK bigint;
 
---Create table replication metrics
+-- Create table replication metrics
 CREATE TABLE REPLICATION_METRICS (
   RM_SCHEDULED_EXECUTION_ID bigint NOT NULL,
   RM_POLICY varchar(256) NOT NULL,
@@ -104,7 +104,7 @@ CREATE TABLE REPLICATION_METRICS (
   PRIMARY KEY(RM_SCHEDULED_EXECUTION_ID)
 );
 
---Create indexes for the replication metrics table
+-- Create indexes for the replication metrics table
 CREATE INDEX POLICY_IDX ON REPLICATION_METRICS (RM_POLICY);
 CREATE INDEX DUMP_IDX ON REPLICATION_METRICS (RM_DUMP_EXECUTION_ID);
 
diff --git 
a/standalone-metastore/metastore-server/src/main/sql/oracle/hive-schema-4.0.0.oracle.sql
 
b/standalone-metastore/metastore-server/src/main/sql/oracle/hive-schema-4.0.0.oracle.sql
index aefef5e..c7bf0ba 100644
--- 
a/standalone-metastore/metastore-server/src/main/sql/oracle/hive-schema-4.0.0.oracle.sql
+++ 
b/standalone-metastore/metastore-server/src/main/sql/oracle/hive-schema-4.0.0.oracle.sql
@@ -101,7 +101,7 @@ CREATE TABLE DBS
     OWNER_NAME VARCHAR2(128) NULL,
     OWNER_TYPE VARCHAR2(10) NULL,
     CTLG_NAME VARCHAR2(256) DEFAULT 'hive' NOT NULL,
-    CREATE_TIME NUMBER (10)
+    CREATE_TIME NUMBER (10),
     DB_MANAGED_LOCATION_URI VARCHAR2(4000) NULL
 );
 
diff --git 
a/standalone-metastore/metastore-server/src/main/sql/oracle/upgrade-3.2.0-to-4.0.0.oracle.sql
 
b/standalone-metastore/metastore-server/src/main/sql/oracle/upgrade-3.2.0-to-4.0.0.oracle.sql
index 3a894b5..6146710 100644
--- 
a/standalone-metastore/metastore-server/src/main/sql/oracle/upgrade-3.2.0-to-4.0.0.oracle.sql
+++ 
b/standalone-metastore/metastore-server/src/main/sql/oracle/upgrade-3.2.0-to-4.0.0.oracle.sql
@@ -89,7 +89,7 @@ INSERT INTO TXNS (TXN_ID, TXN_STATE, TXN_STARTED, 
TXN_LAST_HEARTBEAT, TXN_USER,
 CREATE SEQUENCE TXNS_TXN_ID_SEQ INCREMENT BY 1 START WITH 1000000001 CACHE 20;
 ALTER TABLE TXNS MODIFY TXN_ID default TXNS_TXN_ID_SEQ.nextval;
 
-RENAME TABLE NEXT_TXN_ID TO TXN_LOCK_TBL;
+RENAME NEXT_TXN_ID TO TXN_LOCK_TBL;
 ALTER TABLE TXN_LOCK_TBL RENAME COLUMN NTXN_NEXT TO TXN_LOCK;
 
 --Create table replication metrics
diff --git 
a/standalone-metastore/metastore-server/src/test/java/org/apache/hadoop/hive/metastore/dbinstall/rules/DatabaseRule.java
 
b/standalone-metastore/metastore-server/src/test/java/org/apache/hadoop/hive/metastore/dbinstall/rules/DatabaseRule.java
index e06011f..4be7639 100644
--- 
a/standalone-metastore/metastore-server/src/test/java/org/apache/hadoop/hive/metastore/dbinstall/rules/DatabaseRule.java
+++ 
b/standalone-metastore/metastore-server/src/test/java/org/apache/hadoop/hive/metastore/dbinstall/rules/DatabaseRule.java
@@ -134,10 +134,10 @@ public abstract class DatabaseRule extends 
ExternalResource {
       return;
     }
     try {
-      if (runCmdAndPrintStreams(buildStopCmd(), 60) != 0) {
+      if (runCmdAndPrintStreams(buildStopCmd(), 600) != 0) {
         throw new RuntimeException("Unable to stop docker container");
       }
-      if (runCmdAndPrintStreams(buildRmCmd(), 15) != 0) {
+      if (runCmdAndPrintStreams(buildRmCmd(), 600) != 0) {
         throw new RuntimeException("Unable to remove docker container");
       }
     } catch (InterruptedException | IOException e) {
diff --git 
a/standalone-metastore/metastore-server/src/test/java/org/apache/hadoop/hive/metastore/dbinstall/rules/Mysql.java
 
b/standalone-metastore/metastore-server/src/test/java/org/apache/hadoop/hive/metastore/dbinstall/rules/Mysql.java
index dcd9ec5..228087f 100644
--- 
a/standalone-metastore/metastore-server/src/test/java/org/apache/hadoop/hive/metastore/dbinstall/rules/Mysql.java
+++ 
b/standalone-metastore/metastore-server/src/test/java/org/apache/hadoop/hive/metastore/dbinstall/rules/Mysql.java
@@ -54,7 +54,7 @@ public class Mysql extends DatabaseRule {
 
   @Override
   public String getJdbcUrl() {
-    return "jdbc:mysql://localhost:3306/" + HIVE_DB + 
"?sessionVariables=sql_mode=ANSI_QUOTES";
+    return "jdbc:mysql://localhost:3306/" + HIVE_DB;
   }
 
   @Override

Reply via email to