Skip to content

feat(taskbroker): Passthrough mode#617

Open
untitaker wants to merge 1 commit intomainfrom
passthrough
Open

feat(taskbroker): Passthrough mode#617
untitaker wants to merge 1 commit intomainfrom
passthrough

Conversation

@untitaker
Copy link
Copy Markdown
Member

ref STREAM-882

Introduce passthrough mode, so that a broker can be used to spawn tasks
from any topic with any type of message format. This will make it easier
to migrate existing consumers to be tasks instead, without changing data
layout in prod. For more information refer to the ticket above.

ref STREAM-882

Introduce passthrough mode, so that a broker can be used to spawn tasks
from any topic with any type of message format. This will make it easier
to migrate existing consumers to be tasks instead, without changing data
layout in prod. For more information refer to the ticket above.
@linear-code
Copy link
Copy Markdown

linear-code Bot commented Apr 30, 2026

@untitaker untitaker marked this pull request as ready for review April 30, 2026 16:17
@untitaker untitaker requested a review from a team as a code owner April 30, 2026 16:17
Comment thread src/config.rs
/// Maps every application to its worker endpoint, both represented as strings.
pub worker_map: BTreeMap<String, String>,

/// Enable passthrough mode for consuming raw bytes from legacy topics.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
/// Enable passthrough mode for consuming raw bytes from legacy topics.
/// Enable passthrough mode for consuming raw bytes from raw topics.

Comment on lines +51 to +55
#[derive(Serialize)]
struct Params<'a> {
args: (&'a [u8],),
kwargs: HashMap<(), ()>,
}
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

For my own understanding, why define this struct here vs outside of the function?

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.

3 participants