[GitHub] kafka pull request #1693: HOTFIX: start embedded kafka in KafkaStreamsTest t...

2016-08-02 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/kafka/pull/1693


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] kafka pull request #1693: HOTFIX: start embedded kafka in KafkaStreamsTest t...

2016-08-02 Thread dguy
GitHub user dguy opened a pull request:

https://github.com/apache/kafka/pull/1693

HOTFIX: start embedded kafka in KafkaStreamsTest to avoid hanging

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.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/dguy/kafka kafka-streams-test

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/kafka/pull/1693.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1693


commit 44b9178d80091d998b39d89e3c352737c76eee25
Author: Damian Guy 
Date:   2016-08-02T11:24:20Z

start an EmbeddedKafkaCluster in KafkaStreamsTest to avoid hanging on 
KafkaConsumser.poll




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---