[GitHub] incubator-hivemall issue #74: [HIVEMALL-84-2] Fix docker

2017-04-25 Thread myui
Github user myui commented on the issue: https://github.com/apache/incubator-hivemall/pull/74 @amaya382 build of hivemall better to be optional in the document. Include ready-to-use jars on docker image. I think Docker file should invoke `bin/build.sh` to create jars before `l

[GitHub] incubator-hivemall issue #74: [HIVEMALL-84-2] Fix docker

2017-04-25 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/incubator-hivemall/pull/74 [![Coverage Status](https://coveralls.io/builds/11233773/badge)](https://coveralls.io/builds/11233773) Coverage remained the same at 37.962% when pulling **ede6c9d4922005ad

[GitHub] incubator-hivemall pull request #74: [HIVEMALL-84-2] Fix docker

2017-04-25 Thread amaya382
GitHub user amaya382 opened a pull request: https://github.com/apache/incubator-hivemall/pull/74 [HIVEMALL-84-2] Fix docker ## What changes were proposed in this pull request? Fix docker ## What type of PR is it? Bug Fix ## What is the Jira issue?

[GitHub] incubator-hivemall issue #68: [HIVEMALL-84] Add Docker Support

2017-04-25 Thread amaya382
Github user amaya382 commented on the issue: https://github.com/apache/incubator-hivemall/pull/68 @myui okay --- 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

[GitHub] incubator-hivemall issue #68: [HIVEMALL-84] Add Docker Support

2017-04-25 Thread myui
Github user myui commented on the issue: https://github.com/apache/incubator-hivemall/pull/68 link to jira issue 84. --- 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 w

[GitHub] incubator-hivemall issue #68: [HIVEMALL-84] Add Docker Support

2017-04-25 Thread myui
Github user myui commented on the issue: https://github.com/apache/incubator-hivemall/pull/68 @amaya382 thanks. could you send pr HIVEMALL-84-2? --- 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 hav

[GitHub] incubator-hivemall issue #68: [HIVEMALL-84] Add Docker Support

2017-04-25 Thread amaya382
Github user amaya382 commented on the issue: https://github.com/apache/incubator-hivemall/pull/68 @myui trivial errors on doc, * missing `.` in docker build section * we have to build hivemall when build image by docker-compose (by build arg `PREBUILD`) --- If your project is

[GitHub] incubator-hivemall issue #68: [HIVEMALL-84] Add Docker Support

2017-04-25 Thread myui
Github user myui commented on the issue: https://github.com/apache/incubator-hivemall/pull/68 @amaya382 merged w/ some modifications. Please check it. Great work! Tested only w/ Docker engine on Mac, not for docker compose. --- If your project is set up for it, you can reply

[GitHub] incubator-hivemall pull request #68: [HIVEMALL-84] Add Docker Support

2017-04-25 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-hivemall/pull/68 --- 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 feat

[GitHub] incubator-hivemall issue #71: [HIVEMALL-74] Implement pLSA

2017-04-25 Thread myui
Github user myui commented on the issue: https://github.com/apache/incubator-hivemall/pull/71 revising at https://github.com/myui/incubator-hivemall/tree/HIVEMALL-74 --- 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