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
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions