[GitHub] incubator-quickstep pull request #336: Fixed the bug that Executor / Cli doe...
GitHub user zuyu opened a pull request: https://github.com/apache/incubator-quickstep/pull/336 Fixed the bug that Executor / Cli does not create directory for StorageManager. Assigned to @jianqiao. @yuanchenl Please test this PR using Docker. Thanks! You can merge this pull request into a Git repository by running: $ git pull https://github.com/zuyu/incubator-quickstep dist-directory Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-quickstep/pull/336.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 #336 commit 9fdb0db4bfd616433d02aeeebe1e162e6040a7e7 Author: Zuyu Zhang Date: 2018-03-01T03:03:46Z Fixed the bug that Executor / Cli does not create directory for StorageManager. ---
[GitHub] incubator-quickstep issue #335: Dockerfile Support for Distributed Deploymen...
Github user yuanchenl commented on the issue: https://github.com/apache/incubator-quickstep/pull/335 Rebased ---
[GitHub] incubator-quickstep issue #335: Dockerfile Support for Distributed Deploymen...
Github user zuyu commented on the issue: https://github.com/apache/incubator-quickstep/pull/335 Before merging this PR, please rebase this branch with the master. ---
[GitHub] incubator-quickstep issue #335: Dockerfile Support for Distributed Deploymen...
Github user yuanchenl commented on the issue: https://github.com/apache/incubator-quickstep/pull/335 The repeated installations are removed. CMake version is changed to 3.10.2 (stable). The goal of this PR is to simply get the distributed quickstep correctly built in a docker image. Since different container instances have different roles (conductor, cli, executor) and different configurations, I am currently running the image using pseudoTTY (docker run -ti) and manually specify the role and the conductor address for quickstep_distributed_cli_shell inside the tty. Do you want me to change the dockerfile so that we can specify those information during building phase and run the image without the stdin attached? ---
