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.

Sign up for GitHub

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

Jump to bottom

[DRAFT] Add SimpleMessageBroker class to support a fully self contained pipeline. #238

Open
wants to merge 21 commits into
base: main
Choose a base branch
from
Open

[DRAFT] Add SimpleMessageBroker class to support a fully self contained pipeline. #238

wants to merge 21 commits into from

Conversation

drobison00
Copy link
Collaborator

Resolves #226

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

@drobison00
Checkpoint -- SimpleMessageBroker is working
ddb9a31
@drobison00
Checkpoint
ee78025
@drobison00
Checkpoint client and broker are working -- need to do some perf tuni…
4b7f5c0 
…ng then integrate.
@drobison00
Checkpoint -- all tests passing -- updating broker to deal with consi…
a8fd9c0 
…stency more correctly
@drobison00
Checkpoint -- order preserving updates and ordered_message_queue crea…
8f107f1 
…tion
@drobison00
Checkpoint -- All unit tests passing
3c29324
@drobison00
Start migrating source schema's to generic broker schema + client def…
20ec0c8 
…inition
@drobison00
Checkpoint -- Redis generic source/sink elements are working for Redi…
10cb53c 
…s -- moving to integrate simple client
@drobison00
Unit tests mostly working; mismatch on redis_task_source module name …
ae5a859 
…traces
@drobison00
Checkpoint
d661ede
@drobison00
Working system before moving rest_client to use ResponseSchemas.
8775a29
@drobison00
End to end is working (mostly) -- still need to handle otel_meter whe…
4501ec3 
…n Redis doesn't exist
@copy-pr-bot copy-pr-bot
Copy link

copy-pr-bot bot commented Nov 22, 2024

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[FEA]: Add new SimpleMessageBroker and supporting elements to NV-ingest
1 participant
@drobison00