You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@dolphinscheduler.apache.org by GitBox <gi...@apache.org> on 2021/02/24 10:38:24 UTC

[GitHub] [incubator-dolphinscheduler] lgcareer commented on a change in pull request #4806: [Improvement][SQL] Place the upgraded schema to the correct version directory and check schema errors

lgcareer commented on a change in pull request #4806:
URL: https://github.com/apache/incubator-dolphinscheduler/pull/4806#discussion_r581847775



##########
File path: sql/dolphinscheduler_mysql.sql
##########
@@ -703,7 +703,7 @@ CREATE TABLE `t_ds_task_instance` (
   `alert_flag` tinyint(4) DEFAULT NULL COMMENT 'whether alert',
   `retry_times` int(4) DEFAULT '0' COMMENT 'task retry times',
   `pid` int(4) DEFAULT NULL COMMENT 'pid of task',
-  `app_link` text COMMENT 'yarn app id',
+  `app_link` varchar(255) DEFAULT NULL COMMENT 'yarn app id',

Review comment:
       Hi,the type of `app_link` should be text,it was updated  when released the 1.3.0 version




----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org