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

[Proposal] Ability to pass per entry metadata to bulk publish. #1198

Open
mapaille opened this issue Nov 30, 2023 · 5 comments
Open

[Proposal] Ability to pass per entry metadata to bulk publish. #1198

mapaille opened this issue Nov 30, 2023 · 5 comments
Labels

Comments

@mapaille
Copy link

In https://github.com/dapr/dotnet-sdk/blob/master/src/Dapr.Client/DaprClientGrpc.cs#L214, the entry metadata is always empty which prevents us to set the sessionId when publishing events to a session enabled topic.

An option would be to add an overload which takes a list of entries as parameter.

@halspang
Copy link
Contributor

@mapaille - Are you asking for an override that sets the same metadata per event? Or an override that lets you specify metadata per event?

@mapaille
Copy link
Author

mapaille commented Dec 1, 2023

@halspang I would like an override that lets me specify metadata per event because each event in a batch can have its own session id.

@halspang
Copy link
Contributor

halspang commented Dec 1, 2023

Makes sense to me, I think this could be a valuable addition.

@halspang halspang added the kind/enhancement New feature or request label Dec 1, 2023
@mapaille
Copy link
Author

mapaille commented Dec 4, 2023

@halspang Kafka seems to have similar feature using the key metadata. https://developer.confluent.io/courses/apache-kafka/partitions/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants