Github user karol-brejna-i commented on a diff in the pull request:

    https://github.com/apache/incubator-gearpump/pull/162#discussion_r103432537
  
    --- Diff: integrationtest/docker/kafka/README.md ---
    @@ -0,0 +1,89 @@
    +Kafka in Docker
    +===
    +
    +**This is a fork spotify/kafka image. We decided to make it, because the 
project does not maintain proper tagging. 
    +For now our tests focus on Kafka 0.8.x, so here is a version that supports 
Kafka 0.8.x.
    +For latest version of kafka image go to the original project.**
    +   
    +This repository provides everything you need to run Kafka in Docker.
    +
    +For convenience also contains a packaged proxy that can be used to get 
data from
    +a legacy Kafka 7 cluster into a dockerized Kafka 8.
    +
    +Why?
    +---
    +The main hurdle of running Kafka in Docker is that it depends on Zookeeper.
    +Compared to other Kafka docker images, this one runs both Zookeeper and 
Kafka
    +in the same container. This means:
    +
    +* No dependency on an external Zookeeper host, or linking to another 
container
    +* Zookeeper and Kafka are configured to work together out of the box
    +
    +Run
    +---
    +
    +```bash
    +docker run -p 2181:2181 -p 9092:9092 --env ADVERTISED_HOST=`docker-machine 
ip \`docker-machine active\`` --env ADVERTISED_PORT=9092 grubykarol/kafka
    --- End diff --
    
    No, we use it without it. It's just example from original README.


---
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.
---

Reply via email to