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

feat: Add reply consumer to taskworker prototypes #79465

Draft
wants to merge 1 commit into
base: hackweek-kafkatasks
Choose a base branch
from

Conversation

markstory
Copy link
Member

I wanted to see how using kafka to consume task results would behave in comparison grpc results. In this arrangement, consumers would consume from both a 'work' and 'result' topic. workers would receive tasks via grpc 'webhooks', execute the task and publish results over kafka.

This allows consumers to not have a grpc interface which I think will make them easier to operate, and give us a centralized place to control ratelimiting and throughput to workers.

There are several code-crimes that will need to be resolved before we could take this approach any further. Most important is that Arroyo does not support consuming from multiple topics in the way I have been thinking about this.

I wanted to see how using kafka to consume task results would behave in
comparison grpc results. In this arrangement, consumers would consume
from both a 'work' and 'result' topic. workers would receive tasks via
grpc 'webhooks', execute the task and publish results over kafka.

This allows consumers to not have a grpc interface which I think will
make them easier to operate, and give us a centralized place to control
ratelimiting and throughput to workers.

There are several code-crimes that will need to
be resolved before we could take this approach any further. Most
important is that Arroyo does not support consuming from multiple topics
in the way I have been thinking about this.
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Oct 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Scope: Backend Automatically applied to PRs that change backend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant