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

[GEN-2347]: add Kafka destination #2428

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

BenElferink
Copy link
Contributor

This pull request introduces support for Kafka as a new destination type in the configuration system. The changes include adding Kafka-specific configuration options, updating utility functions, and modifying documentation to reflect the new destination.

Key changes include:

Kafka Configuration:

  • Added Kafka configuration constants and Kafka struct in common/config/kafka.go to define Kafka-specific settings and methods.
  • Updated common/config/root.go to include Kafka in the list of available configers.

Utility Functions:

  • Added errorMissingKey function in common/config/utils.go to handle missing configuration keys.

Destination Types:

  • Added KafkaDestinationType to the list of destination types in common/dests.go.

Documentation and Metadata:

  • Added Kafka destination metadata in destinations/data/kafka.yaml to define Kafka's configuration fields and display properties.
  • Updated documentation in docs/adding-new-dest.mdx and docs/backends-overview.mdx to include Kafka as a supported destination. [1] [2] [3] [4]

@BenElferink BenElferink added the enhancement New feature or request label Feb 11, 2025
Copy link

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

Successfully merging this pull request may close these issues.

1 participant