[GitHub] zeppelin pull request #1404: ZEPPELIN-1411. UDF with pyspark not working - o...

2016-09-13 Thread zjffdu
GitHub user zjffdu reopened a pull request: https://github.com/apache/zeppelin/pull/1404 ZEPPELIN-1411. UDF with pyspark not working - object has no attribute 'parseDataType' ### What is this PR for? The root cause is that SQLContext's signature changes in spark 2.0. Spark

[GitHub] zeppelin pull request #1404: ZEPPELIN-1411. UDF with pyspark not working - o...

2016-09-13 Thread zjffdu
Github user zjffdu closed the pull request at: https://github.com/apache/zeppelin/pull/1404 --- 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 feature is e

[GitHub] zeppelin issue #1408: [ZEPPELIN-808] Changed Permissions on Notebooks do not...

2016-09-13 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/1408 @kavinkumarks this looks better and tested locally, works as expected. few things: * would be nice to update screenshot above * CI build fails, could you take a look into that or re

[GitHub] zeppelin issue #1393: [ZEPPELIN-1142] Zeppelin allows two users to simultane...

2016-09-13 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1393 Thanks @Leemoonsoo for reviewing this! Could you please merge this? - Kavin MailTo: kavin.ku...@imaginea.com --- If your project is set up for it, you can reply to this email and

[GitHub] zeppelin issue #1330: [ZEPPELIN-1144]Zeppelin home page should only list not...

2016-09-13 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/1330 @Leemoonsoo issues are created under [ZEPPELIN-1437](https://issues.apache.org/jira/browse/ZEPPELIN-1437) and [ZEPPELIN-1438](https://issues.apache.org/jira/browse/ZEPPELIN-1438) correspon

[jira] [Created] (ZEPPELIN-1438) Per user note list broadcast on note permission change

2016-09-13 Thread Khalid Huseynov (JIRA)
Khalid Huseynov created ZEPPELIN-1438: - Summary: Per user note list broadcast on note permission change Key: ZEPPELIN-1438 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1438 Project: Zeppelin

[jira] [Created] (ZEPPELIN-1437) Note list broadcast is not aware of user session

2016-09-13 Thread Khalid Huseynov (JIRA)
Khalid Huseynov created ZEPPELIN-1437: - Summary: Note list broadcast is not aware of user session Key: ZEPPELIN-1437 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1437 Project: Zeppelin

[GitHub] zeppelin issue #1423: ZEPPELIN-1425. sparkr.zip is not distributed to execut...

2016-09-13 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1423 Thanks for the explanation. LGTM and merge if there're no more discussions. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. I

[GitHub] zeppelin issue #1411: [ZEPPELIN-1417] Bugfix of "Connect to existing process...

2016-09-13 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1411 LGTM and merge if there're no more discussions --- 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

[GitHub] zeppelin issue #1423: ZEPPELIN-1425. sparkr.zip is not distributed to execut...

2016-09-13 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1423 @Leemoonsoo The `rpkg.zip` is for the case that user have R code inside of jars specified through --jars. The reason why I didn't include it here is that I have to duplicate lots of code from spa

[GitHub] zeppelin issue #1416: Rewrite of "Quick Start" page.

2016-09-13 Thread rfilmyer
Github user rfilmyer commented on the issue: https://github.com/apache/zeppelin/pull/1416 Alright. I'm probably going to make some more changes on this PR. (Probably 2 or 3 more pages). --- If your project is set up for it, you can reply to this email and have your reply appear on Gi

[GitHub] zeppelin issue #1418: [gh-pages][ZEPPELIN-990] Add header anchor to gh-pages...

2016-09-13 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1418 Merge if there're no further discussions. --- 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 feat

[GitHub] zeppelin issue #1416: Rewrite of "Quick Start" page.

2016-09-13 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1416 Thanks @rfilmyer for the great contribution! Like @AhyoungRyu explained, you can either add more commits here or create new PR depending on further contribution you want to make. LG

[jira] [Created] (ZEPPELIN-1436) Python Interpreter does not work on Windows

2016-09-13 Thread Deenar Toraskar (JIRA)
Deenar Toraskar created ZEPPELIN-1436: - Summary: Python Interpreter does not work on Windows Key: ZEPPELIN-1436 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1436 Project: Zeppelin

[GitHub] zeppelin issue #1426: ZEPPELIN-1430. Display appId and webui link in LivyInt...

2016-09-13 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1426 I think displaying result with additional information is quick and simple approach. However, i think there're some downsides like - can not leverage output streaming - when user out

[GitHub] zeppelin issue #1384: [ZEPPELIN-1387] Support table syntax in markdown inter...

2016-09-13 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1384 Thanks @1ambda. Looks great to me. Merge if there're no further discussions. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] zeppelin issue #1423: ZEPPELIN-1425. sparkr.zip is not distributed to execut...

2016-09-13 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1423 @zjffdu Do you think [rpkg.zip](https://github.com/apache/spark/blob/v1.6.2/core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala#L394) is the something also need to be taken care? --

[GitHub] zeppelin issue #1425: Improving Matplotlib Integration with Zeppelin

2016-09-13 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1425 Thanks @IshmeetKaur. I think i'll try to merge #1422 first because #1422 arrived earlier. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHu

[GitHub] zeppelin issue #1422: Allow matplotlib plot display in pyspark context

2016-09-13 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1422 Could you try indent code with 2 spaces like other part of codes do? --- 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 pro

[GitHub] zeppelin issue #1330: [ZEPPELIN-1144]Zeppelin home page should only list not...

2016-09-13 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1330 @mwkang @khalidhuseynov Could you handle or at least create issues for expected problems after merge this PR? I can at least think following two issues, - Note list broadcasting messag

[GitHub] zeppelin issue #1393: [ZEPPELIN-1142] Zeppelin allows two users to simultane...

2016-09-13 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1393 Thanks @kavinkumarks for the contribution. This changes not only resolving race condition described in ZEPPELIN-1142 but also changes user experience. i.e. Before, owner is not defi

[GitHub] zeppelin issue #1424: [ZEPPELIN-1426] User aware storage sync

2016-09-13 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1424 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 feature enabled and wishes so, or if the

[GitHub] zeppelin pull request #1413: [MINOR] Format Fix for InterpreterResultTest.ja...

2016-09-13 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1413 --- 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 feature is e

[GitHub] zeppelin issue #1404: ZEPPELIN-1411. UDF with pyspark not working - object h...

2016-09-13 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1404 Thanks @zjffdu. I think second ci test profile failure looks relevant. ``` ests run: 6, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 120.231 sec <<< FAILURE! - in org.apache.ze

[GitHub] zeppelin issue #1302: [ZEPPELIN-1306] Interpreter restarts on a note.

2016-09-13 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1302 @astroshim @corneadoug, Shell we have some unittests for `public void restart(String settingId, String noteId)` before we merge? Restarting only a interpreter instance/process for a note is new

[GitHub] zeppelin pull request #1427: [ZEPPELIN-1435] Move zeppelin-server-*.jar unde...

2016-09-13 Thread minahlee
GitHub user minahlee opened a pull request: https://github.com/apache/zeppelin/pull/1427 [ZEPPELIN-1435] Move zeppelin-server-*.jar under lib in distribution binary package ### What is this PR for? Move zeppelin-server-*.jar package under `lib` directory like other zeppelin mod

[jira] [Created] (ZEPPELIN-1435) Move zeppelin-server-*.jar under lib directory in distribution package

2016-09-13 Thread Mina Lee (JIRA)
Mina Lee created ZEPPELIN-1435: -- Summary: Move zeppelin-server-*.jar under lib directory in distribution package Key: ZEPPELIN-1435 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1435 Project: Zeppe

[GitHub] zeppelin pull request #1425: Improving Matplotlib Integration with Zeppelin

2016-09-13 Thread IshmeetKaur
Github user IshmeetKaur closed the pull request at: https://github.com/apache/zeppelin/pull/1425 --- 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 feature

[GitHub] zeppelin pull request #1425: Improving Matplotlib Integration with Zeppelin

2016-09-13 Thread IshmeetKaur
GitHub user IshmeetKaur reopened a pull request: https://github.com/apache/zeppelin/pull/1425 Improving Matplotlib Integration with Zeppelin ### What is this PR for? This PR improves the matplotlib integration with zeppelin. ### What type of PR is it? Improvement

[jira] [Created] (ZEPPELIN-1434) Long numbers rounding in Zeppelin-Web table

2016-09-13 Thread David Tzoor (JIRA)
David Tzoor created ZEPPELIN-1434: - Summary: Long numbers rounding in Zeppelin-Web table Key: ZEPPELIN-1434 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1434 Project: Zeppelin Issue Ty

[GitHub] zeppelin issue #1398: [MINOR] Doc fix for spark interpreter

2016-09-13 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1398 @zjffdu It's okay :) 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 feature enable

[jira] [Created] (ZEPPELIN-1433) What dependency to add to project (sbt or maven) get hold of ZeppelinContext?

2016-09-13 Thread sam (JIRA)
sam created ZEPPELIN-1433: - Summary: What dependency to add to project (sbt or maven) get hold of ZeppelinContext? Key: ZEPPELIN-1433 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1433 Project: Zeppelin

[GitHub] zeppelin pull request #1398: [MINOR] Doc fix for spark interpreter

2016-09-13 Thread zjffdu
GitHub user zjffdu reopened a pull request: https://github.com/apache/zeppelin/pull/1398 [MINOR] Doc fix for spark interpreter ### What is this PR for? Several changes on doc of spark interpreter. * %spark, %sql, %pyspark only works when spark is the default interpreter

[GitHub] zeppelin issue #1398: [MINOR] Doc fix for spark interpreter

2016-09-13 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1398 Sorry @AhyoungRyu I didn't notice you PR is based on my branch and thought you will merge to master directly. --- If your project is set up for it, you can reply to this email and have your repl

[GitHub] zeppelin issue #1398: [MINOR] Doc fix for spark interpreter

2016-09-13 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1398 @zjffdu Did you close this PR on purpose? --- 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 fea

[GitHub] zeppelin pull request #1398: [MINOR] Doc fix for spark interpreter

2016-09-13 Thread zjffdu
Github user zjffdu closed the pull request at: https://github.com/apache/zeppelin/pull/1398 --- 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 feature is e

[GitHub] zeppelin issue #1404: ZEPPELIN-1411. UDF with pyspark not working - object h...

2016-09-13 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1404 @Leemoonsoo , I updated the unit test. And also made a little change `AbstractTestRestApi`, the standalone way doesn't work for me. So I allow user to export `SPAKP_MASTER` to run it in other modes

[GitHub] zeppelin pull request #1412: [Zeppelin 1415] - Adding Zeppelin-Web Good Prac...

2016-09-13 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1412 --- 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 feature is e

[GitHub] zeppelin issue #1302: [ZEPPELIN-1306] Interpreter restarts on a note.

2016-09-13 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1302 Merging if there is no more discussions --- 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 featur

[GitHub] zeppelin issue #1358: [ZEPPELIN-1365] Error of Zeppelin Application in devel...

2016-09-13 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1358 @Leemoonsoo Done --- 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] zeppelin issue #1426: ZEPPELIN-1430. Display appId and webui link in LivyInt...

2016-09-13 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1426 Here is before and after of this PR that I observe, and I see these 2 changes - now it does not stream output, i.e. it waits for the entire paragraph execution to be finished, and

[GitHub] zeppelin issue #1424: [ZEPPELIN-1426] User aware storage sync

2016-09-13 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1424 Looks good to me. Will merge to master if there is no further discussion --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If you

[GitHub] zeppelin issue #1426: ZEPPELIN-1430. Display appId and webui link in LivyInt...

2016-09-13 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1426 @prabhjyotsingh @Leemoonsoo @felixcheung @bzz @minahlee Please help review. I changed the livy output display style from plain text to angular in order to display the link. And for now it i