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

Kafka Backend group ID configuration make easier #391

Merged
merged 3 commits into from
Nov 11, 2021

Conversation

purbon
Copy link
Collaborator

@purbon purbon commented Nov 11, 2021

No description provided.

@purbon purbon force-pushed the fix/group_id_kafka-backend branch from 77a213a to f2fd8f8 Compare November 11, 2021 11:21
@purbon purbon merged commit ed0b734 into master Nov 11, 2021
purbon added a commit that referenced this pull request Nov 13, 2021
* use julie.instance.id  in case group.id is not defined for the kafka backend

* make default variable as consumer group id for kafka backend consumer, by default named julieops, if not it can be configured via property file or env var

* ammend kafka tests, not to use same group.id, so consumption is done back from beginiing
@purbon purbon deleted the fix/group_id_kafka-backend branch December 9, 2021 10:30
purbon added a commit that referenced this pull request Dec 9, 2021
* use julie.instance.id  in case group.id is not defined for the kafka backend

* make default variable as consumer group id for kafka backend consumer, by default named julieops, if not it can be configured via property file or env var

* ammend kafka tests, not to use same group.id, so consumption is done back from beginiing
purbon added a commit that referenced this pull request Dec 9, 2021
* use julie.instance.id  in case group.id is not defined for the kafka backend

* make default variable as consumer group id for kafka backend consumer, by default named julieops, if not it can be configured via property file or env var

* ammend kafka tests, not to use same group.id, so consumption is done back from beginiing
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant