Added PublishEventWithMetadata

[ItalyPaleAle]

Fixes #164

[berndverst]

LGTM

If you are looking for other contributions related to this:

• Would be cool if we had an integration test that also sets up Kafka and then sets the TTL (which is what you implemented this feature for @ItalyPaleAle) using the GO SDK
• Update the Go SDK Readme to show an example that sets the TTL: https://github.com/dapr/go-sdk/#pubsub
• Update the docs at https://docs.dapr.io/developing-applications/building-blocks/pubsub/pubsub-message-ttl/ (this would need to go into the 1.3 branch)

[berndverst]

@yaron2 can you please review this PR also and merge if it looks good to you?

[tanvigour]

Looks good to me. Thanks @ItalyPaleAle !

[ItalyPaleAle]

@yaron2 as I was talking to Bernd yesterday over Teams, while this PR does fix 164 perhaps there may be a better approach with what I proposed in 174. Can you take a look and let me know if I should scrap this PR? If you prefer the approach in 174, i should be able to make a new PR fairly quickly.

[yaron2]

@yaron2 as I was talking to Bernd yesterday over Teams, while this PR does fix 164 perhaps there may be a better approach with what I proposed in 174. Can you take a look and let me know if I should scrap this PR? If you prefer the approach in 174, i should be able to make a new PR fairly quickly.

Thanks for taking a look at this, I agree that this can be done in a different way. Please see my comment here: #174 (comment).

I'm closing this PR for now, we can re-open if needed.