drcrallen commented on a change in pull request #7296: Remove mysql database 
connector from container image
URL: https://github.com/apache/incubator-druid/pull/7296#discussion_r266991954
 
 

 ##########
 File path: distribution/docker/README.md
 ##########
 @@ -24,3 +24,15 @@ From the root of the repo, run `docker build -t druid:tag 
-f distribution/docker
 ## Run
 
 Edit `environment` to suite. Run 'docker-compose -f 
distribution/docker/docker-compose.yml up`
+
+## mysql database connector
+
+This image contains solely the postgres metadata database connector. If you 
need
+the mysql metadata storage connector, consider adding these lines before the 
`addgroup`
 
 Review comment:
   I forget the details of the original conversation, but would it be possible 
to add a `Dockerfile.mysql` or similar that uses a FROM clause against the 
other docker container and just runs this command?
   
   That way the instructions can be something like "Build the dockerfile at 
`something/something/Dockerfile.mysql` and specify the upstream tag like this: 
`docker build -t sometag -f something/something/Dockerfile.mysql`" or similar?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org
For additional commands, e-mail: commits-h...@druid.apache.org

Reply via email to