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

HOTFIX: start embedded kafka in KafkaStreamsTest to avoid hanging #1693

Closed
wants to merge 1 commit into from

Conversation

dguy
Copy link
Contributor

@dguy dguy commented Aug 2, 2016

The KafkaStreamsTest can occasionally hang if the test doesn't run fast enough. This is due to there being no brokers available on the broker.urls provided to the StreamsConfig. The KafkaConsumer does a poll and blocks causing the test to never complete.

@ijuma
Copy link
Contributor

ijuma commented Aug 2, 2016

LGTM

asfgit pushed a commit that referenced this pull request Aug 2, 2016
The KafkaStreamsTest can occasionally hang if the test doesn't run fast enough. This is due to there being no brokers available on the broker.urls provided to the StreamsConfig. The KafkaConsumer does a poll and blocks causing the test to never complete.

Author: Damian Guy <damian.guy@gmail.com>

Reviewers: Ismael Juma <ismael@juma.me.uk>

Closes #1693 from dguy/kafka-streams-test
@asfgit asfgit closed this in 3bb38d3 Aug 2, 2016
@guozhangwang
Copy link
Contributor

Good catch, thanks @dguy !

@dguy dguy deleted the kafka-streams-test branch August 3, 2016 08:25
efeg pushed a commit to efeg/kafka that referenced this pull request May 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants