Skip to content

[BUG] when Master Server scanCommand, in rare case, there maybe dirty data saved into ProcessInstance due to Transactional logic problem #1245

@Baoqi

Description

@Baoqi

Currently, in scanCommand, the exception is swallowed inside try{} catch{}, so, the transational is not working as expected.

https://github.com/apache/incubator-dolphinscheduler/blob/dev-db/dolphinscheduler-dao/src/main/java/org/apache/dolphinscheduler/dao/ProcessDao.java#L146

图片

If the lines (marked in red rectangle) throws exception. The processinstance will still be saved. but no thread will be created to handle this one.

I will provide PR for this.

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions