[GitHub] incubator-zeppelin pull request: ZEPPELIN-262 Use spark-submit to ...

2015-09-04 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/270#issuecomment-137903488 looks good! --- 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

[GitHub] incubator-zeppelin pull request: ZEPPELIN-277: Set the (TAB) key a...

2015-09-04 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/273#issuecomment-137896642 Thanks. I've tested this branch and the feature itself working well. However, depends on language, TAB is not ideal for some language like python or sc

[GitHub] incubator-zeppelin pull request: ZEPPELIN-279: move website w/ doc...

2015-09-04 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/282#issuecomment-137895116 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 fe

[GitHub] incubator-zeppelin pull request: ZEPPELIN-279: move website w/ doc...

2015-09-04 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/283#issuecomment-137895134 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 fe

[GitHub] incubator-zeppelin pull request: ZEPPELIN-262 Use spark-submit to ...

2015-09-04 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/270#issuecomment-137893230 @randerzander dependency jars downloaded from z.load() is being loaded after SparkContext is created by calling sc.addJar(). So i think it'll not be affe

[jira] [Created] (ZEPPELIN-280) 3 RemoteInterpreterServer processes keep running after `mvn test`

2015-09-04 Thread Alexander Bezzubov (JIRA)
Alexander Bezzubov created ZEPPELIN-280: --- Summary: 3 RemoteInterpreterServer processes keep running after `mvn test` Key: ZEPPELIN-280 URL: https://issues.apache.org/jira/browse/ZEPPELIN-280 Pro

[GitHub] incubator-zeppelin pull request: ZEPPELIN-278 fix build warnings a...

2015-09-04 Thread bzz
Github user bzz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/279#issuecomment-137891154 On CI `mvn package -Pspark-1.1 -Phadoop-2.3 -Ppyspark -B -pl 'zeppelin-interpreter,spark-dependencies,spark` ``` java.lang.reflect.InvocationTargetEx

[GitHub] incubator-zeppelin pull request: ZEPPELIN-278 fix build warnings a...

2015-09-04 Thread bzz
Github user bzz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/279#issuecomment-137878442 CI is *very unstable* - same command over the same branch passes on my machine ``` mvn package -Pspark-1.1 -Phadoop-2.3 -B -pl 'zeppelin-interpreter

[GitHub] incubator-zeppelin pull request: ZEPPELIN-278 fix build warnings a...

2015-09-04 Thread bzz
Github user bzz closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/279 --- 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 featur

[GitHub] incubator-zeppelin pull request: ZEPPELIN-278 fix build warnings a...

2015-09-04 Thread bzz
Github user bzz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/279#issuecomment-137878460 Giving it one more chance... --- 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

[GitHub] incubator-zeppelin pull request: ZEPPELIN-278 fix build warnings a...

2015-09-04 Thread bzz
GitHub user bzz reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/279 ZEPPELIN-278 fix build warnings and rat hang `mvn apache-rat:check` passes All sub-modules now use same version of: - maven-dependency-plugin - scala compiler You can me

[GitHub] incubator-zeppelin pull request: ZEPPELIN-279: cleanup gh-pages br...

2015-09-04 Thread bzz
GitHub user bzz opened a pull request: https://github.com/apache/incubator-zeppelin/pull/283 ZEPPELIN-279: cleanup gh-pages branch after move to master Details at the [ZEPPELIN-279](https://issues.apache.org/jira/browse/ZEPPELIN-279) You can merge this pull request into a Git repos

[GitHub] incubator-zeppelin pull request: ZEPPELIN-279: move website w/ doc...

2015-09-04 Thread bzz
GitHub user bzz opened a pull request: https://github.com/apache/incubator-zeppelin/pull/282 ZEPPELIN-279: move website w/ docs to master branch Deatilas at the [ZEPPELIN-279](https://issues.apache.org/jira/browse/ZEPPELIN-279) You can merge this pull request into a Git repository

Re: [DISCUSS] moving website with documentation from gh-pages to master branch

2015-09-04 Thread Felix Cheung
+1 On Thu, Sep 3, 2015 at 1:04 PM -0700, "Christian Tzolov" wrote: +1 On 3 September 2015 at 16:40, moon soo Lee wrote: > +1 > > On 2015년 9월 3일 (목) at 오전 7:25 IT CTO wrote: > > > +1 I just did the REST PR and had to open two PR for REST updates and Doc > > updates. > > > > > > On Thu, Se

[GitHub] incubator-zeppelin pull request: Fix zeppelin-server security test...

2015-09-04 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/281 --- 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: ZEPPELIN-76 skip Selenium tests o...

2015-09-04 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/280 --- 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: Zeppelin interpreter documentatio...

2015-09-04 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/271 --- 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: Zeppelin interpreter documentatio...

2015-09-04 Thread bzz
Github user bzz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/271#issuecomment-137867310 Looks good to me --- 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 th

[GitHub] incubator-zeppelin pull request: ZEPPELIN-262 Use spark-submit to ...

2015-09-04 Thread randerzander
Github user randerzander commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/270#issuecomment-137828701 @Leemoonsoo how does z.load work with spark-submit? Seems those dependency jars should be added automatically to spark-submit's --jars argument. --- If

[GitHub] incubator-zeppelin pull request: ZEPPELIN-76 skip Selenium tests o...

2015-09-04 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/280#issuecomment-137768184 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 fe

[GitHub] incubator-zeppelin pull request: ZEPPELIN 228 - Add unit tests for...

2015-09-04 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/250#issuecomment-137767317 Looks good to me! --- 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

[GitHub] incubator-zeppelin pull request: Zeppelin interpreter documentatio...

2015-09-04 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/271#issuecomment-137764603 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 fe

[GitHub] incubator-zeppelin pull request: ZEPPELIN-273 Spark 1.5 support

2015-09-04 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/269 --- 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 zeppelin-server security test...

2015-09-04 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/281#issuecomment-137759817 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 fe

[GitHub] incubator-zeppelin pull request: Fix zeppelin-server security test...

2015-09-04 Thread bzz
GitHub user bzz opened a pull request: https://github.com/apache/incubator-zeppelin/pull/281 Fix zeppelin-server security tests on case-sensitive hostname On os x, w/ case-sensitive hostname some test for zeppelin-server were failing ``` $hostname MacBookPro.local

[GitHub] incubator-zeppelin pull request: ZEPPELIN-277: Set the (TAB) key a...

2015-09-04 Thread tzolov
Github user tzolov commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/273#issuecomment-137695790 This oneliner is ready for reviewing. It works excellent with the interpreters i've been using and really improves the UX perception. But I wonder if there

[GitHub] incubator-zeppelin pull request: [Doc] Update installation instruc...

2015-09-04 Thread bzz
Github user bzz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/81#issuecomment-137667393 Ready to merge, after #280 --- 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 n

[GitHub] incubator-zeppelin pull request: ZEPPELIN-76 skip Selenium tests o...

2015-09-04 Thread bzz
Github user bzz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/280#issuecomment-137667231 Ready to merge. #81 can be merged right after --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If