Skip to content

[SPIKE] JustSaying message ingestion and retry #4997

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

Draft
wants to merge 34 commits into
base: master
Choose a base branch
from

Conversation

ngallegos
Copy link
Contributor

@ngallegos ngallegos commented Jun 3, 2025

Results from https://github.com/Particular/Vision/issues/605

Completed:

  • Added incoming and outgoing message translation from/to JustSaying message format
  • Added multiple error queue support and discoverability

Started, but not complete:

  • JustSaying IMessageMonitor implementation to report failure information to ServiceControl
  • JustSaying middleware implementation to acknowledge retries to ServiceControl
  • ServiceControl api for receiving failure and retry acknowledgments
  • ServiceControl support for storing and reconciling failure information with messages in the error queue

Note

The message translation was done in the SQS Transport and surfaced here through git submodules. If you want to run the spike yourself, you'll need to run the following commands in the repository root after pulling the source code:

> git submodule init
> git submodule update --recursive

ngallegos and others added 30 commits May 5, 2025 13:18
Report files can be large given that we are capturing daily data.
Partial broken implementation
…thub:Particular/ServiceControl into tf-exp-1579-just-saying-collaboration
@ngallegos ngallegos marked this pull request as draft June 3, 2025 18:06
@ngallegos ngallegos changed the title [Tf exp 1579 just saying collaboration [SPIKE] JustSaying message ingestion and retry spike Jun 3, 2025
@ngallegos ngallegos changed the title [SPIKE] JustSaying message ingestion and retry spike [SPIKE] JustSaying message ingestion and retry Jun 3, 2025
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.

5 participants