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
Kafka: Add message key and other metadata as metadata in consumer #3289
Kafka: Add message key and other metadata as metadata in consumer #3289
Conversation
37ab449
to
ba601df
Compare
66722b7
to
0b87d21
Compare
Signed-off-by: Patrick Assuied <patrick.assuied@elationhealth.com>
0b87d21
to
dc695c0
Compare
@dapr/maintainers-components-contrib, @berndverst, @yaron2, First contribution. Waiting for approval... What's the typical SLA? |
@@ -28,6 +28,11 @@ import ( | |||
|
|||
const ( | |||
key = "partitionKey" | |||
keyMetadataKey = "__key" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please note I chose the __
prefix convention to reduce chances of collision with other headers... The __
prefix is a common convention in kafka connect. Though I could make the prefix more specific (like __kafka.timestamp
for instance
…pr#3289) Signed-off-by: Patrick Assuied <patrick.assuied@elationhealth.com>
Description
Pass Message Key and other message metadata as metadata in Kafka.pubsub consumer.
Currently, only message value is passed to endpoint.
Issue reference
Fixes #2213
Checklist