Github user bhlx3lyx7 commented on the issue:
https://github.com/apache/incubator-griffin/pull/263
Hi @younglee1 , thanks for your work.
I've reviewed the modifications you've made, and have some suggestions:
1. In `griffin-doc/deploy/deploy-guide.md`, "For Griffin users, please
follow the instructions below to you can deploy Griffin in your environment"
confused me.
2. In `griffin-doc/docker/griffin-docker-guide.md`, "There are four
pre-built griffin environment images", actually, only the first one is the
griffin docker image, the next two are re-built based on the official images,
the last one is the official image of zookeeper, so I don't think we can say
they are four pre-built images, we'd better say it like "Here liststhe griffin
and dependent environment docker images".
Would you mind make some changes? We'll accept after that.
---
