DarkAssassinator commented on issue #12873: URL: https://github.com/apache/dolphinscheduler/issues/12873#issuecomment-1312631004
> > now TaskInstance and TaskDefinition contain environment and cpuQuota and memoryMax, but these shoud be just task params, not task fields. > > Currently, the json of our task params is huge, which already affects query performance. If we add more content to it, it is bound to aggravate the problem. I think we should split this json into different tables. @caishunfeng @DarkAssassinator +1 i think that this is a good idea, we can add a table named `t_ds_task_params`, then drop the `task_params` column in `t_ds_task_instance`, add `task_params_code` column? WDYT. cc @SbloodyS @caishunfeng -- 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]
