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

kerwin pushed a commit to branch 2.0.1-release
in repository https://gitbox.apache.org/repos/asf/dolphinscheduler.git


The following commit(s) were added to refs/heads/2.0.1-release by this push:
     new 967ac26  The sql has a syntax error, and because it is a function, the 
excepti… (#7741)
967ac26 is described below

commit 967ac268fc574ed099cee03992410b0def36e3e0
Author: uh001 <[email protected]>
AuthorDate: Fri Dec 31 16:57:31 2021 +0800

    The sql has a syntax error, and because it is a function, the excepti… 
(#7741)
    
    * The sql has a syntax error, and because it is a function, the exception 
information will be hidden
---
 .../sql/upgrade/2.0.0_schema/mysql/dolphinscheduler_ddl.sql    | 10 ++++++++--
 .../upgrade/2.0.0_schema/postgresql/dolphinscheduler_ddl.sql   | 10 +++++++---
 2 files changed, 15 insertions(+), 5 deletions(-)

diff --git 
a/dolphinscheduler-dao/src/main/resources/sql/upgrade/2.0.0_schema/mysql/dolphinscheduler_ddl.sql
 
b/dolphinscheduler-dao/src/main/resources/sql/upgrade/2.0.0_schema/mysql/dolphinscheduler_ddl.sql
index 7e1e605..ba12846 100644
--- 
a/dolphinscheduler-dao/src/main/resources/sql/upgrade/2.0.0_schema/mysql/dolphinscheduler_ddl.sql
+++ 
b/dolphinscheduler-dao/src/main/resources/sql/upgrade/2.0.0_schema/mysql/dolphinscheduler_ddl.sql
@@ -147,7 +147,10 @@ BEGIN
                      AND TABLE_SCHEMA=(SELECT DATABASE())
                      AND COLUMN_NAME ='code')
     THEN
-           alter table t_ds_project add `code` bigint(20) NOT NULL COMMENT 
'encoding' AFTER `name`;
+           alter table t_ds_project add `code` bigint(20) COMMENT 'encoding' 
AFTER `name`;
+           -- update default value for not null
+           UPDATE t_ds_project SET code = id;
+           alter table t_ds_project modify `code` bigint(20) NOT NULL;
     END IF;
 END;
 
@@ -399,7 +402,10 @@ alter table t_ds_schedules add timezone_id varchar(40) 
DEFAULT NULL COMMENT 'tim
 alter table t_ds_schedules add environment_code bigint(20) DEFAULT '-1' 
COMMENT 'environment code' AFTER worker_group;
 
 -- t_ds_process_definition
-alter table t_ds_process_definition add `code` bigint(20) NOT NULL COMMENT 
'encoding' AFTER `id`;
+alter table t_ds_process_definition add `code` bigint(20) COMMENT 'encoding' 
AFTER `id`;
+-- update default value for not null
+UPDATE t_ds_process_definition SET code = id;
+alter table t_ds_process_definition modify `code` bigint(20) NOT NULL;
 alter table t_ds_process_definition change project_id project_code bigint(20) 
NOT NULL COMMENT 'project code' AFTER `description`;
 alter table t_ds_process_definition add `warning_group_id` int(11) DEFAULT 
NULL COMMENT 'alert group id' AFTER `locations`;
 alter table t_ds_process_definition add UNIQUE KEY `process_unique` 
(`name`,`project_code`) USING BTREE;
diff --git 
a/dolphinscheduler-dao/src/main/resources/sql/upgrade/2.0.0_schema/postgresql/dolphinscheduler_ddl.sql
 
b/dolphinscheduler-dao/src/main/resources/sql/upgrade/2.0.0_schema/postgresql/dolphinscheduler_ddl.sql
index e5024d9..7e72c7b 100644
--- 
a/dolphinscheduler-dao/src/main/resources/sql/upgrade/2.0.0_schema/postgresql/dolphinscheduler_ddl.sql
+++ 
b/dolphinscheduler-dao/src/main/resources/sql/upgrade/2.0.0_schema/postgresql/dolphinscheduler_ddl.sql
@@ -45,10 +45,10 @@ BEGIN
        EXECUTE 'ALTER TABLE ' || quote_ident(v_schema) 
||'.t_ds_process_definition ALTER COLUMN project_code TYPE bigint';
 
        --- add columns
-    EXECUTE 'ALTER TABLE ' || quote_ident(v_schema) ||'.t_ds_user ADD COLUMN 
IF NOT EXISTS "state" int DEFAULT 1';
+        EXECUTE 'ALTER TABLE ' || quote_ident(v_schema) ||'.t_ds_user ADD 
COLUMN IF NOT EXISTS "state" int DEFAULT 1';
        EXECUTE 'ALTER TABLE ' || quote_ident(v_schema) ||'.t_ds_alertgroup ADD 
COLUMN IF NOT EXISTS "alert_instance_ids" varchar(255) DEFAULT NULL';
        EXECUTE 'ALTER TABLE ' || quote_ident(v_schema) ||'.t_ds_alertgroup ADD 
COLUMN IF NOT EXISTS "create_user_id" int4 DEFAULT NULL';
-       EXECUTE 'ALTER TABLE ' || quote_ident(v_schema) ||'.t_ds_project ADD 
COLUMN IF NOT EXISTS "code" bigint NOT NULL';
+       EXECUTE 'ALTER TABLE ' || quote_ident(v_schema) ||'.t_ds_project ADD 
COLUMN IF NOT EXISTS "code" bigint';
        EXECUTE 'ALTER TABLE ' || quote_ident(v_schema) ||'.t_ds_command ADD 
COLUMN IF NOT EXISTS "environment_code" bigint DEFAULT -1';
        EXECUTE 'ALTER TABLE ' || quote_ident(v_schema) ||'.t_ds_command ADD 
COLUMN IF NOT EXISTS "dry_run" int DEFAULT 0';
        EXECUTE 'ALTER TABLE ' || quote_ident(v_schema) ||'.t_ds_command ADD 
COLUMN IF NOT EXISTS "process_definition_version" int DEFAULT 0';
@@ -74,6 +74,10 @@ BEGIN
        EXECUTE 'ALTER TABLE ' || quote_ident(v_schema) 
||'.t_ds_process_definition ADD COLUMN IF NOT EXISTS "code" bigint';
        EXECUTE 'ALTER TABLE ' || quote_ident(v_schema) 
||'.t_ds_process_definition ADD COLUMN IF NOT EXISTS "warning_group_id" int';
 
+        --update default value for not null
+        EXECUTE 'UPDATE ' || quote_ident(v_schema) ||'.t_ds_process_definition 
SET code = id';
+        EXECUTE 'UPDATE ' || quote_ident(v_schema) ||'.t_ds_project SET code = 
id';
+
        ---drop columns
        EXECUTE 'ALTER TABLE ' || quote_ident(v_schema) ||'.t_ds_tenant DROP 
COLUMN IF EXISTS "tenant_name"';
        EXECUTE 'ALTER TABLE ' || quote_ident(v_schema) 
||'.t_ds_process_instance DROP COLUMN IF EXISTS "process_instance_json"';
@@ -92,7 +96,7 @@ BEGIN
        EXECUTE 'ALTER TABLE ' || quote_ident(v_schema) 
||'."t_ds_process_definition" ALTER COLUMN "code" SET NOT NULL';
        EXECUTE 'ALTER TABLE ' || quote_ident(v_schema) 
||'."t_ds_process_definition" ALTER COLUMN "project_code" SET NOT NULL';
        EXECUTE 'ALTER TABLE ' || quote_ident(v_schema) 
||'."t_ds_process_definition" ADD CONSTRAINT "process_unique" UNIQUE 
("name","project_code")';
-       EXECUTE 'ALTER TABLE ' || quote_ident(v_schema) 
||'."t_ds_process_definition" ALTER COLUMN "description" SET NOT NULL';
+        EXECUTE 'ALTER TABLE ' || quote_ident(v_schema) ||'."t_ds_project" 
ALTER COLUMN "code" SET NOT NULL';
 
        --- drop index
        EXECUTE 'DROP INDEX IF EXISTS "process_instance_index"';

Reply via email to