calvinjiang commented on pull request #6160:
URL: https://github.com/apache/dolphinscheduler/pull/6160#issuecomment-916748544


   > Firstly, I think bigint should not default to null. Secondly, the code 
inserted into the task table should be improved first. If you merge this 
directly, environment_ Code does not have a default value. If a user directly 
creates a workflow, an error will be reported
   > 
![image](https://user-images.githubusercontent.com/42576980/132822035-c2580b18-a916-402d-a2da-dd4f09f4fc91.png)
   
   First of all  I found that  the default value of many fields in the 
dolphinscheduler_mysql.sql  was set to be null. So this may be a problem.
   
![image](https://user-images.githubusercontent.com/4928204/132826329-db0cab55-5686-42cf-b711-c484b634be1f.png)
   
   Secondly this PR is for the issue #6159.
   
   Finally When you create a workflow ,there will an error to be reported. I 
checked out code , and found that this problem was caused by the commit 
b661ec96750e8254f88be3861f125052e0ab30f9. So I will fix this new problem in the 
next PR.
   
   
![image](https://user-images.githubusercontent.com/4928204/132828581-5857fd79-f989-40de-aca1-687c23951b3c.png)
   
   
   
![image](https://user-images.githubusercontent.com/4928204/132828903-11ee331e-1ebd-45ed-99f7-9833fa2b41a4.png)
   
   


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