qiuyanjun888 commented on code in PR #18349:
URL: 
https://github.com/apache/dolphinscheduler/pull/18349#discussion_r3418285601


##########
dolphinscheduler-dao/src/main/resources/sql/dolphinscheduler_mysql.sql:
##########
@@ -1149,7 +1149,8 @@ DROP TABLE IF EXISTS 
`t_ds_environment_worker_group_relation`;
 CREATE TABLE `t_ds_environment_worker_group_relation` (
   `id` bigint(11) NOT NULL AUTO_INCREMENT COMMENT 'id',
   `environment_code` bigint(20) NOT NULL COMMENT 'environment code',
-  `worker_group` varchar(255) NOT NULL COMMENT 'worker group id',
+  `worker_group_id` int(11) DEFAULT NULL COMMENT 'worker group id',
+  `worker_group` varchar(255) NOT NULL COMMENT 'worker group name',

Review Comment:
   Addressed in 05ef5125df. The base schema now defines worker_group_id as NOT 
NULL. For upgrades, DDL adds it nullable first, DML backfills it from 
t_ds_worker_group, and then the column is changed to NOT NULL for MySQL and 
PostgreSQL.
   



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to