Skip to content
This repository has been archived by the owner on Aug 16, 2023. It is now read-only.

Remove outbox transaction handler #ULMS-2507 #429

Open
wants to merge 102 commits into
base: setup-nats-consumer
Choose a base branch
from

Conversation

wakeful-kitten
Copy link
Contributor

No description provided.

@wakeful-kitten wakeful-kitten requested a review from a team as a code owner July 27, 2023 13:14
@wakeful-kitten wakeful-kitten requested a review from 0nkery July 27, 2023 15:23
Copy link
Contributor

@0nkery 0nkery left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Нужно, чтобы CI прошел успешно.
  • Не нужно обновлять БД, пока не первое сообщение не уйдет в очередь -- возможно, потребуется даже 4 сообщения. Иначе сообщения будут теряться и "транзакция" зависнет на полпути.

src/app/endpoint/group/update.rs Outdated Show resolved Hide resolved
src/app/stage/mod.rs Outdated Show resolved Hide resolved
src/app/stage/mod.rs Outdated Show resolved Hide resolved
src/app/stage/mod.rs Outdated Show resolved Hide resolved
src/app/endpoint/room.rs Show resolved Hide resolved
.github/workflows/check.yml Outdated Show resolved Hide resolved
src/app/endpoint/group/update.rs Outdated Show resolved Hide resolved
src/app/stage/nats_ids/sqlx.rs Outdated Show resolved Hide resolved
src/app/endpoint/group/update.rs Outdated Show resolved Hide resolved
src/app/stage/video_group/intent_event.rs Outdated Show resolved Hide resolved
src/app/error.rs Outdated Show resolved Hide resolved
src/app/stage/mod.rs Outdated Show resolved Hide resolved
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants