Skip to content
This repository has been archived by the owner on Sep 4, 2024. It is now read-only.

Add messageKey option to Kafka transport #315

Merged
merged 1 commit into from
Apr 18, 2024

Conversation

dolfinus
Copy link
Contributor

@dolfinus dolfinus commented Apr 10, 2024

Update documentation for KafkaTransport according to changed proposed in OpenLineage/OpenLineage#2559.

Probably should not be merged before OpenLineage 1.13.0 release.

Copy link

netlify bot commented Apr 10, 2024

Deploy Preview for mellifluous-eclair-326dcd ready!

Name Link
🔨 Latest commit f01ec86
🔍 Latest deploy log https://app.netlify.com/sites/mellifluous-eclair-326dcd/deploys/66210dca6baec8000808eb80
😎 Deploy Preview https://deploy-preview-315--mellifluous-eclair-326dcd.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@dolfinus dolfinus force-pushed the feature/kafka-message-key branch 2 times, most recently from f6a995b to a7e07b2 Compare April 10, 2024 15:54
@dolfinus dolfinus marked this pull request as ready for review April 15, 2024 07:57
@dolfinus
Copy link
Contributor Author

Added a note that these options were added since 1.13.0

@dolfinus dolfinus force-pushed the feature/kafka-message-key branch 3 times, most recently from 10b3955 to 04f737b Compare April 17, 2024 15:40
@dolfinus
Copy link
Contributor Author

dolfinus commented Apr 18, 2024

Rebased to main. Also included addresses changes from OpenLineage/OpenLineage#2615, although it is not yet merged.

Signed-off-by: Мартынов Максим Сергеевич <[email protected]>
@mobuchowski mobuchowski merged commit b5f1bdd into OpenLineage:main Apr 18, 2024
6 checks passed
@dolfinus dolfinus deleted the feature/kafka-message-key branch April 18, 2024 17:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants