After commit callback #882

New issue

Open

opened 2021-01-16 07:25:10 +00:00 by KitaitiMakoto · 2 comments

KitaitiMakoto commented

2021-01-16 07:25:10 +00:00

Owner

Think you

begin transaction
insert a post
publishes PostPublished event to the actor channel
some error occurs
transaction rollbacked
but you can find the post by search

We should publish PostPublished only after transaction commited.

Solusion might include sending patches to diesel.

Think you 1. begin transaction 2. insert a post 3. publishes `PostPublished` event to the actor channel 4. some error occurs 5. transaction rollbacked 6. but you can find the post by search We should publish `PostPublished` only after transaction commited. Solusion might include sending patches to diesel.

KitaitiMakoto commented

2021-01-16 12:57:58 +00:00

Author

Owner

Currently, we don't use transactions. There's no problem.

KitaitiMakoto commented

2021-02-15 15:06:52 +00:00

Author

Owner

It might be good to move database-related code from request handlers to models.

KitaitiMakoto added the

A: Backend

label 2021-03-27 11:21:59 +00:00

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: Plume/Plume#882

No description provided.

Rows
Columns