Skip to content

[change] Make batch import atomic #551

@nemesifier

Description

@nemesifier

I was reviewing the batch CSV import code due to #550 and it seems to me the create operation is not atomic.

What if the import operation fails in the middle?

I believe we would end up with half users created.

I think it's better to make everything run in a transaction and if anything fails rollback.

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type

    Projects

    Status

    To do

    Status

    To do (general)

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions