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

feat: Create topic with additional configuration if present #601

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

robertcoltheart
Copy link
Contributor

Description

Passes additional configuration options, if present, to the topic creation method.

Fixes #600

How Has This Been Tested?

Please describe the tests that you ran to verify your changes.

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have added tests to cover my changes
  • I have made corresponding changes to the documentation

Disclaimer

By sending us your contributions, you are agreeing that your contribution is made subject to the terms of our Contributor Ownership Statement

@robertcoltheart robertcoltheart changed the title Create topic with additional configuration if present feat: Create topic with additional configuration if present Nov 6, 2024
@robertcoltheart robertcoltheart force-pushed the feature/topic-creation-configs branch from 8832d81 to fb3b005 Compare November 6, 2024 20:02
@robertcoltheart robertcoltheart force-pushed the feature/topic-creation-configs branch from fb3b005 to f34011b Compare November 6, 2024 21:16
@@ -3,12 +3,12 @@
init_broker:
@echo command | date
@echo Initializing Kafka broker
docker-compose -f docker-compose.yml up -d
docker compose -f docker-compose.yml up -d
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed the makefile, since it was failing to build due to using the wrong command under latest Ubuntu

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

Successfully merging this pull request may close these issues.

[Feature Request]: Create topic with additional configuration
1 participant