Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[AB#43722] internal media messages are now stored to inbox directly #292

Merged
merged 1 commit into from
Apr 25, 2024

Conversation

AxTrusov
Copy link
Member

@AxTrusov AxTrusov commented Apr 24, 2024

Pull Request

Prerequisites

  • The PR is targeting the right branch (dev for features and master for
    releases)
  • potential release notes to the PR description added
  • potential testing notes to the PR description added
  • appropriate labels for the PR applied

Description

With the integration of transactional inbox/outbox pattern, some messages that are only circulating within a service started taking long round-trips like this: source handler -> outbox -> RMQ -> inbox -> destination handler
This change makes sure that all internal messages are written directly to inbox, so it now looks like this: source handler -> inbox -> destination handler

Testing notes

  • Make sure the following media service operations work as before:
    • Publishing and unpublishing of entities (both single and bulk)
    • Bulk deletion of entities
    • Operations on snapshots (both single and bulk)
    • ingest with images, videos, and localizations

@AxTrusov AxTrusov requested a review from Zehelein April 24, 2024 12:38
@AxTrusov AxTrusov merged commit e9798a3 into dev Apr 25, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants