#882 After commit callback

Open
opened 2 months ago by KitaitiMakoto · 2 comments

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.

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 2 months ago
Owner

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

Currently, we don't use transactions. There's no problem.
KitaitiMakoto commented 1 month ago
Owner

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

It might be good to move database-related code from request handlers to models.
KitaitiMakoto added the
A: Backend
label 2 weeks ago
Sign in to join this conversation.
No Milestone
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.