[GitHub] incubator-zeppelin pull request: Fix interpreter.sh classpath

2016-05-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/889 --- 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 fea

[GitHub] incubator-zeppelin pull request: Fix interpreter.sh classpath

2016-05-18 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/889#issuecomment-220165691 Every jar under interpreter/spark/ supposed to be included in the classpath by [interpreter.sh#67](https://github.com/apache/incubator-zeppelin/blob/maste

[GitHub] incubator-zeppelin pull request: Fix interpreter.sh classpath

2016-05-12 Thread elbamos
Github user elbamos commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/889#issuecomment-218920877 It takes out a change I'd out there because the r interpreter jar wasn't getting loaded in time in some circumstances. I need to test this. I will try to do

[GitHub] incubator-zeppelin pull request: Fix interpreter.sh classpath

2016-05-12 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/889#issuecomment-218909569 LGTM --- 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 f

[GitHub] incubator-zeppelin pull request: Fix interpreter.sh classpath

2016-05-12 Thread Leemoonsoo
GitHub user Leemoonsoo opened a pull request: https://github.com/apache/incubator-zeppelin/pull/889 Fix interpreter.sh classpath ### What is this PR for? This PR apply fix #769 again, which is reverted by #208. Also removing unnecessary code from interpreter.sh ### W