ruanwenjun opened a new issue #5546:
URL: https://github.com/apache/dolphinscheduler/issues/5546


   **Describe the question**
   When we update a process definition, we need to perform multiple DML 
operations, it is needed to add transaction to ensure the data is consistent 
when an exception occurs.
   
https://github.com/apache/dolphinscheduler/blob/842c5400e605a8b8eb0d8fdc78701f10222063fd/dolphinscheduler-api/src/main/java/org/apache/dolphinscheduler/api/service/impl/ProcessDefinitionServiceImpl.java#L375
   
   
https://github.com/apache/dolphinscheduler/blob/842c5400e605a8b8eb0d8fdc78701f10222063fd/dolphinscheduler-service/src/main/java/org/apache/dolphinscheduler/service/process/ProcessService.java#L2228
   
   
   There are also some other method need to check, such as:
   
https://github.com/apache/dolphinscheduler/blob/842c5400e605a8b8eb0d8fdc78701f10222063fd/dolphinscheduler-service/src/main/java/org/apache/dolphinscheduler/service/process/ProcessService.java#L2367
   
   
https://github.com/apache/dolphinscheduler/blob/842c5400e605a8b8eb0d8fdc78701f10222063fd/dolphinscheduler-service/src/main/java/org/apache/dolphinscheduler/service/process/ProcessService.java#L2376
   
   **Which version of DolphinScheduler:**
    -[dev]
   
   **Describe alternatives you've considered**
   Add transaction, and it is recommended to split ProcessService into 
CommandService、ProcessDefinitionService、ProcessInstanceService、TaskService
   


-- 
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:
[email protected]


Reply via email to