[incubator-superset] branch master updated: Docker Demo, Port Change (#6289)

[ma...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

maximebeauchemin pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-superset.git


The following commit(s) were added to refs/heads/master by this push:
     new 70327ca  Docker Demo, Port Change (#6289)
70327ca is described below

commit 70327ca42c805f673ac9fb82934d8563aeb68aa8
Author: Irfan Faizullabhoy <fa...@usc.edu>
AuthorDate: Wed Nov 7 21:28:04 2018 -0800

    Docker Demo, Port Change (#6289)
    
    If you follow the documentation, the current docker instructions return an error when trying to connect via localhost. This seems to be due to the fact that it's running on port 8080, but forwarding port 8088. I suppose the cleaner solution is to do this, as opposed to changing both the compose file and the instructions... Hope this helps!!
---
 contrib/docker/docker-init.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/contrib/docker/docker-init.sh b/contrib/docker/docker-init.sh
index 6ba5ca7..f50046b 100644
--- a/contrib/docker/docker-init.sh
+++ b/contrib/docker/docker-init.sh
@@ -21,4 +21,4 @@ cd superset/assets && npm run build && cd ../../
 superset worker &
 
 # Start the dev web server
-flask run -p 8080 --with-threads --reload --debugger --host=0.0.0.0
+flask run -p 8088 --with-threads --reload --debugger --host=0.0.0.0