[GitHub] zeppelin issue #1041: Get note revision of note - git repo
Github user onkarshedge commented on the issue: https://github.com/apache/zeppelin/pull/1041 @khalidhuseynov will this be merged soon ? Currently I have extended the `Rev` class for my Ipfs Revision. If this is merged I won't have to extend. ```java private class IpfsRevision extends NotebookRepoVersioned.Rev { private String commitMessage; IpfsRevision(String name, int time, String message) { super(name, time); commitMessage = message; } ``` --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin pull request #1042: [WIP] bin/install-interpreter.sh for netinst pa...
GitHub user Leemoonsoo opened a pull request: https://github.com/apache/zeppelin/pull/1042 [WIP] bin/install-interpreter.sh for netinst package ### What is this PR for? This is proof of the concept implementation of bin/install-interpreter.sh for netinst package which suggested in the [discussion](http://apache-zeppelin-users-incubating-mailing-list.75479.x6.nabble.com/Ask-opinion-regarding-0-6-0-release-package-tp3298p3314.html). Some usages will be ``` # download all interpreters provided by Apache Zeppelin project bin/install-interpreter.sh --all # download an interpreter with name (for example markdown interpreter) bin/install-interpreter.sh --name md # download an (3rd party) interpreter with specific maven artifact name bin/install-interpreter.sh --name md -t org.apache.zeppelin:zeppelin-markdown:0.6.0-SNAPSHOT ``` If it looks fine, i'll continue the work (refactor code, and add test) ### What type of PR is it? Feature ### Todos * [x] - working implementation * [ ] - refactor * [ ] - add test ### What is the Jira issue? * Open an issue on Jira https://issues.apache.org/jira/browse/ZEPPELIN/ * Put link here, and add [ZEPPELIN-*Jira number*] in PR title, eg. [ZEPPELIN-533] ### How should this be tested? Outline the steps to test the PR here. ### Screenshots (if appropriate) ### Questions: * Does the licenses files need update? * Is there breaking changes for older versions? * Does this needs documentation? You can merge this pull request into a Git repository by running: $ git pull https://github.com/Leemoonsoo/zeppelin netinst Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/1042.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 #1042 commit efdef3d293472930776405dbeeb8e092f26981ae Author: Lee moon sooDate: 2016-06-18T03:32:59Z Initial implementation of install-interpreter.sh --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #458: [WIP] Fix notebook clone bug
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/458 Duplicated with #933 --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin pull request #458: [WIP] Fix notebook clone bug
Github user minahlee closed the pull request at: https://github.com/apache/zeppelin/pull/458 --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #966: [ZEPPELIN-958] Support syntax highlight for python and ...
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/966 That was only to map the list of interpreter to the language though. To figure out which interpreter is active in the interpreter right now we have regex to match "%something" at the beginning which won't work for the default interpreter at all. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin pull request #1041: Get note revision of note - git repo
GitHub user khalidhuseynov opened a pull request: https://github.com/apache/zeppelin/pull/1041 Get note revision of note - git repo ### What is this PR for? This PR implements the backend of git storage layer for `get(noteId, revision)`, so that you can get the note of certain revision. It doesn't provide frontend api yet, which is future work. ### What type of PR is it? Improvement ### Todos * [ ] - implement `get` of GitNotebookRepo * [ ] - add tests ### What is the Jira issue? N/A ### How should this be tested? two tests for `get` function should pass ### Screenshots (if appropriate) ### Questions: * Does the licenses files need update? no * Is there breaking changes for older versions? no * Does this needs documentation? no You can merge this pull request into a Git repository by running: $ git pull https://github.com/khalidhuseynov/incubator-zeppelin feat/git-get-note-revision-of-note Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/1041.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 #1041 commit c91dff08c255efd69b9fde5068c36b7a3ad74dd2 Author: Khalid HuseynovDate: 2016-06-16T23:53:58Z change field name: revId -> id commit 01d16463048fb65551e716af2da540d109347f1f Author: Khalid Huseynov Date: 2016-06-16T23:55:07Z basic impl of get specific version of note commit d71b4036069422a70bdd2242b7bf3755182d7bd6 Author: Khalid Huseynov Date: 2016-06-17T00:23:32Z add edge case fail test --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #747: [ZEPPELIN-605][WIP] Add support for Scala 2.11
Github user lresende commented on the issue: https://github.com/apache/zeppelin/pull/747 Ok, looks like the hanging I was seeing was mostly a local issue in my Spark conf and is now fixed. I have Zeppelin and Spark built with Scala 2.11, and the tutorial notebook works well, I can also update the interpreter properties and that works well as well. Export is working, but Importing as mentioned by @chbrown just don't do anything. I will try with master and see if this is an issue specific to the scala 2.11 branch or not. Btw, I am not sure if the CI build these days is helping or not, as it's very hard to trust that failures are related to changes. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin pull request #1040: [ZEPPELIN-1031] Update Elasticsearch to release...
GitHub user lresende opened a pull request: https://github.com/apache/zeppelin/pull/1040 [ZEPPELIN-1031] Update Elasticsearch to release 2.3.3 ### What is this PR for? Update ElasticSearch interpreter to use ElasticSearch 2.3.3 ### What type of PR is it? [Improvement] ### What is the Jira issue? [ZEPPELIN-1031](https://issues.apache.org/jira/browse/ZEPPELIN-1031) You can merge this pull request into a Git repository by running: $ git pull https://github.com/lresende/incubator-zeppelin elasticsearch Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/1040.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 #1040 commit 67cc8539235488e02f05ddc0b782fa0f4addc0a9 Author: Luciano ResendeDate: 2016-06-17T17:58:09Z [ZEPPELIN-1031] Update Elasticsearch to release 2.3.3 --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin pull request #1039: [ZEPPELIN-1032] Update Apache Flink to 1.0.3 re...
GitHub user lresende opened a pull request: https://github.com/apache/zeppelin/pull/1039 [ZEPPELIN-1032] Update Apache Flink to 1.0.3 release ### What is this PR for? Update Flink interpreter to use most recent Apache Flink 1.0.3 release ### What type of PR is it? [Improvement] ### What is the Jira issue? [ZEPPELIN-1032](https://issues.apache.org/jira/browse/ZEPPELIN-1032) You can merge this pull request into a Git repository by running: $ git pull https://github.com/lresende/incubator-zeppelin flink Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/1039.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 #1039 commit 0d55fd774dc29636752bbd5e7b1812e991651a71 Author: Luciano ResendeDate: 2016-06-17T17:59:38Z [ZEPPELIN-1032] Update Apache Flink to 1.0.3 release --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #928: [ZEPPELIN-116] Add Mahout Support for Spark Interpreter
Github user rawkintrevo commented on the issue: https://github.com/apache/zeppelin/pull/928 Lot's of connection test failures when building Zeppelin-Server? --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin pull request #1028: [Zeppelin-1022] Apply new mechanism to LivyInte...
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1028 --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[jira] [Created] (ZEPPELIN-1032) Update Apache Flink to 1.0.3 release
Luciano Resende created ZEPPELIN-1032: - Summary: Update Apache Flink to 1.0.3 release Key: ZEPPELIN-1032 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1032 Project: Zeppelin Issue Type: Bug Reporter: Luciano Resende Update to latest Flink release -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[GitHub] zeppelin issue #928: [ZEPPELIN-116][WIP] Add Mahout Support for Spark Interp...
Github user rawkintrevo commented on the issue: https://github.com/apache/zeppelin/pull/928 @bzz and @Leemoonsoo A big part of the refactor was introducing no new dependencies- instead loading from maven or MAHOUT_HOME at interpretter start up via dependency resolver. This was done to get around version mismatches. Here is output from `mvn dependency:tree` I'm not super familiar, so I'm not 100% sure there are nothing new, but I am like 97% sure. ``` [INFO] --- maven-dependency-plugin:2.8:tree (default-cli) @ zeppelin-mahout --- [INFO] org.apache.zeppelin:zeppelin-mahout:jar:0.6.0-SNAPSHOT [INFO] +- org.slf4j:slf4j-api:jar:1.7.10:compile [INFO] +- org.slf4j:slf4j-log4j12:jar:1.7.10:compile [INFO] | \- log4j:log4j:jar:1.2.17:compile [INFO] +- org.apache.zeppelin:zeppelin-interpreter:jar:0.6.0-SNAPSHOT:provided [INFO] +- org.apache.zeppelin:zeppelin-spark:jar:0.6.0-SNAPSHOT:compile [INFO] +- org.apache.zeppelin:zeppelin-spark-dependencies:jar:0.6.0-SNAPSHOT:compile [INFO] +- com.google.code.gson:gson:jar:2.2:compile [INFO] +- org.datanucleus:datanucleus-core:jar:3.2.10:compile [INFO] +- org.datanucleus:datanucleus-api-jdo:jar:3.2.6:compile [INFO] +- org.datanucleus:datanucleus-rdbms:jar:3.2.9:compile [INFO] +- org.scala-lang:scala-library:jar:2.10.4:compile [INFO] +- org.scala-lang:scala-compiler:jar:2.10.4:compile [INFO] +- org.scala-lang:scala-reflect:jar:2.10.4:compile [INFO] \- junit:junit:jar:4.11:test [INFO]\- org.hamcrest:hamcrest-core:jar:1.3:test ``` --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin pull request #1037: fix typo the description of interpreter menu.
GitHub user astroshim opened a pull request: https://github.com/apache/zeppelin/pull/1037 fix typo the description of interpreter menu. ### What is this PR for? This PR is for fixing typo of the interpreter menu description. ### What type of PR is it? Bug Fix ### Screenshots (if appropriate) - before ![image](https://cloud.githubusercontent.com/assets/3348133/16156875/1dba142e-34f1-11e6-937b-c3d2c75a6c42.png) - after ![image](https://cloud.githubusercontent.com/assets/3348133/16156901/365979a2-34f1-11e6-91a7-4c99b78a1d86.png) ### Questions: * Does the licenses files need update? no * Is there breaking changes for older versions? no * Does this needs documentation? no You can merge this pull request into a Git repository by running: $ git pull https://github.com/astroshim/zeppelin feat/typoInterpreters Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/1037.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 #1037 commit 64f1f608a38f42eadbee202f037a4d3c5ac6189d Author: astroshimDate: 2016-06-17T16:04:24Z fix typo description of interpreter menu. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1025: ZEPPELIN-964 ] Jobmanagement - (2) basic front-end
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1025 @cloverhearts Thanks for the quick response. I understand. :+1: --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1025: ZEPPELIN-964 ] Jobmanagement - (2) basic front-end
Github user cloverhearts commented on the issue: https://github.com/apache/zeppelin/pull/1025 @astroshim check to sub-task section. https://issues.apache.org/jira/browse/ZEPPELIN-531 Thank you. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1034: [ZEPPELIN-1009] [HOTFIX] Fix Selenium test error
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1034 I'm getting different error ... ``` Results : Tests in error: ZeppelinIT.testAngularDisplay:151->AbstractZeppelinIT.waitForParagraph:84->AbstractZeppelinIT.pollingWait:110 û Timeout ``` --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[jira] [Created] (ZEPPELIN-1030) Add example settings for Hive, Phoenix, Tajo
Jongyoul Lee created ZEPPELIN-1030: -- Summary: Add example settings for Hive, Phoenix, Tajo Key: ZEPPELIN-1030 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1030 Project: Zeppelin Issue Type: Sub-task Components: zeppelin-interpreter Reporter: Jongyoul Lee -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[GitHub] zeppelin issue #1028: [Zeppelin-1022] Apply new mechanism to LivyInterpreter
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1028 Will merge this if no more 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 your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1027: [ZEPPELIN-1020] Remove suggestion list on click
Github user ravicodder commented on the issue: https://github.com/apache/zeppelin/pull/1027 @suvam97 Thanks for the contribution!! 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 feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1005: ZEPPELIN-995 Change scheduler for JDBC interpreter to ...
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1005 Merging it into master and branch-0.6 --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #806: [ZEPPELIN-707]Automatically adds %.* of previous paragr...
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/806 @mwkang I have one more question. See below. https://cloud.githubusercontent.com/assets/3612566/16153092/d3b8e618-34df-11e6-9649-3540c11f86cd.png;> Can you fix the case of wrong interpreter name? Or do you have any idea to deal with? --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1036: [MINOR] Add .Rhistory in .gitignore
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1036 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 feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1032: ZEPPELIN-1027: Python - add basic matplotlib example n...
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1032 Good one, we should have it. 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 feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1036: [MINOR] Add .Rhistory in .gitignore
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1036 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 feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[jira] [Created] (ZEPPELIN-1029) Configuration REST API
Kévin Giroux created ZEPPELIN-1029: -- Summary: Configuration REST API Key: ZEPPELIN-1029 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1029 Project: Zeppelin Issue Type: Improvement Components: zeppelin-server Affects Versions: 0.5.6 Environment: Zeppelin Server Reporter: Kévin Giroux Fix For: 0.6.1 The Zeppelin Server provide a lot of REST API for creating, cloning, editing, getting and deleting notebook and paragraph, However, it is still impossible to change the configuration, as for example, the type of a graphic, or data that need to be displayed. It would be very usefull for user that doesn't have access to websocket or can't manage modification easily. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[GitHub] zeppelin issue #1027: [ZEPPELIN-1020] Remove suggestion list on click
Github user suvam97 commented on the issue: https://github.com/apache/zeppelin/pull/1027 @prabhjyotsingh @r-kamath Thanks for the review! @prabhjyotsingh done required changes. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1027: [ZEPPELIN-1020] Remove suggestion list on click
Github user r-kamath commented on the issue: https://github.com/apache/zeppelin/pull/1027 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 feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1027: [ZEPPELIN-1020] Remove suggestion list on click
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1027 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 feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1034: [ZEPPELIN-1009] [HOTFIX] Fix Selenium test error
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1034 Looks great 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 this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1028: [Zeppelin-1022] Apply new mechanism to LivyInterpreter
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1028 @bzz I was referring to https://github.com/apache/zeppelin/pull/1034, ZEPPELIN-1009. ``` Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 104.524 sec - in org.apache.zeppelin.integration.ZeppelinIT Results : Tests in error: ParagraphActionsIT.testRemoveButton:153 û NoSuchElement Unable to locate eleme... ParagraphActionsIT.testCreateNewButton:86 û NoSuchElement Unable to locate ele... ``` @jongyoul Thank you for testing it out. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #806: [ZEPPELIN-707]Automatically adds %.* of previous paragr...
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/806 @mwkang Those tests are enough. I've tested it and LGTM. Thanks for contribution. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin pull request #986: [Zeppelin 946] Permissions not honoring group
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/986 --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin pull request #1035: [MINOR] Changed guava version to 19.0 for selen...
GitHub user jongyoul opened a pull request: https://github.com/apache/zeppelin/pull/1035 [MINOR] Changed guava version to 19.0 for selenium test ### What is this PR for? Enable selenium test with SafariDriver. ### What type of PR is it? [Bug Fix] ### Todos * [x] - Upgrade guava version ### What is the Jira issue? N/A ### How should this be tested? 1. Install selenium driver for Safari 1. Make sure that you don't have firefox installed 1. Start Zeppelin without this patch 1. run a selenium test. `TEST_SELENIUM="true" mvn package -DfailIfNoTests=false -pl 'zeppelin-interpreter,zeppelin-zengine,zeppelin-server' -Dtest=ParagraphActionsIT` 1. Got error 1. Repeat after patch 1. Success ### Screenshots (if appropriate) Got error like: ``` java.lang.NoSuchMethodError: com.google.common.util.concurrent.MoreExecutors.directExecutor()Ljava/util/concurrent/Executor; at org.openqa.selenium.safari.WebSocketConnection.send(WebSocketConnection.java:139) at org.openqa.selenium.safari.SafariDriverCommandExecutor.execute(SafariDriverCommandExecutor.java:188) at org.openqa.selenium.remote.RemoteWebDriver.execute(RemoteWebDriver.java:620) at org.openqa.selenium.remote.RemoteWebDriver.startSession(RemoteWebDriver.java:247) at org.openqa.selenium.remote.RemoteWebDriver.(RemoteWebDriver.java:129) at org.openqa.selenium.remote.RemoteWebDriver.(RemoteWebDriver.java:142) at org.openqa.selenium.safari.SafariDriver.(SafariDriver.java:62) at org.openqa.selenium.safari.SafariDriver.(SafariDriver.java:42) at org.apache.zeppelin.WebDriverManager.getWebDriver(WebDriverManager.java:103) at org.apache.zeppelin.integration.ParagraphActionsIT.startUp(ParagraphActionsIT.java:51) ``` ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation? No You can merge this pull request into a Git repository by running: $ git pull https://github.com/jongyoul/zeppelin minor-upgrade-guava-for-selenium-with-safaridriver Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/1035.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 #1035 commit 6f932789d84fb313d2865067d946dc51e9fd3d1d Author: Jongyoul LeeDate: 2016-06-17T08:33:49Z Changed guava version to 19.0 --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin pull request #746: [ZEPPELIN-699] Add new synchronous paragraph run...
GitHub user doanduyhai reopened a pull request: https://github.com/apache/zeppelin/pull/746 [ZEPPELIN-699] Add new synchronous paragraph run REST API ### What is this PR for? Right now, when calling the REST API `http://:/api/notebook/job//` Zeppelin always returns **OK** as shown by this source code: https://github.com/apache/incubator-zeppelin/blob/master/zeppelin-server/src/main/java/org/apache/zeppelin/rest/NotebookRestApi.java#L477 This ticket will update the behavior so that Zeppelin also return the result of the paragraph execution ### What type of PR is it? [Improvement] ### Todos * [ ] - Code Review * [ ] - Simple Test ### Is there a relevant Jira issue? **[ZEPPELIN-699]** ### How should this be tested? * `git fetch origin pull/746/head:ParagraphExecutionRESTAPI` * `git checkout ParagraphExecutionRESTAPI` * `mvn clean package -DskipTests` * `bin/zeppelin-daemon.sh restart` * Create a new note * In the first paragraph, put the following code ```scala %sh echo "Current time = "`date +"%T" ``` * Retrieve the current note id in the URL * Retrieve the current paragraph id * Use a REST Client like **[POSTman]** to create a HTTP POST query `http://:/api/notebook/run//` * You should receive something similar as follow for answer ``` { "status": "OK", "body": { "code": "SUCCESS", "type": "TEXT", "msg": "Current time = 16:14:18\n" } } ``` ### Screenshots (if appropriate) ![zeppelin_synchronous_rest_api](https://cloud.githubusercontent.com/assets/1532977/15748069/b4a26a46-28dd-11e6-8f51-aa13ddba3f1c.gif) API Documentation update **Existing asynchronous API** ![image](https://cloud.githubusercontent.com/assets/1532977/15773274/5b508cae-2976-11e6-9e52-14d8b7e7828e.png) **New synchronous API** ![image](https://cloud.githubusercontent.com/assets/1532977/15773309/84965a94-2976-11e6-9719-81d8b555c3c4.png) ### Questions: * Does the licenses files need update? --> **No** * Is there breaking changes for older versions? --> **No** * Does this needs documentation? --> **Yes** [ZEPPELIN-699]: https://issues.apache.org/jira/browse/ZEPPELIN-699 [POSTman]: https://www.getpostman.com/ You can merge this pull request into a Git repository by running: $ git pull https://github.com/doanduyhai/incubator-zeppelin ZEPPELIN-699 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/746.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 #746 commit 090d9de360b2130d2fee135949ef31a35642934f Author: DuyHai DOANDate: 2016-06-02T14:15:46Z [ZEPPELIN-699] Add new synchronous paragraph run REST API commit faf9542feee910cb4d584b4b49927136a3f839f7 Author: DuyHai DOAN Date: 2016-06-02T14:24:38Z [ZEPPELIN-699] Update Notebook REST API documentation commit 0106cf5735912467ee36aaf37edefd617548863e Author: DuyHai DOAN Date: 2016-06-03T08:20:17Z [ZEPPELIN-699] Fix doc after code review --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin pull request #746: [ZEPPELIN-699] Add new synchronous paragraph run...
Github user doanduyhai closed the pull request at: https://github.com/apache/zeppelin/pull/746 --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #998: Authenticated user aware notebook storage layer
Github user vgmartinez commented on the issue: https://github.com/apache/zeppelin/pull/998 LGTM...I have tested and works well for 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 this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1024: Add force layout visualization in paragraphs for SQL q...
Github user davidtzoor commented on the issue: https://github.com/apache/zeppelin/pull/1024 Sorry for the mess. I will create a new branch with the commits and reopen a new pull request. Thank you for your help --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1019: [ZEPPELIN-1015] Cron job fails to run a paragraph when...
Github user jmrr commented on the issue: https://github.com/apache/zeppelin/pull/1019 LGTM too, thanks @Leemoonsoo I've also tried with same interpreter and also works --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1013: [ZEPPELIN-1002] Move configuration menus under dropdow...
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/1013 I also don't feel the necessity of showing full user name since logged user would know his own username. Let me remove username under dropdown then. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1013: [ZEPPELIN-1002] Move configuration menus under dropdow...
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1013 @minahlee Actually, the long name would normally be truncated by CSS instead of javascript, so it wouldn't be good. (Since it was done elsewhere, no need to fix it here) Is it necessary to show long name though? Maybe this can be handled in a different PR, so that we can think of other way of doing it, and not block this one. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #998: Authenticated user aware notebook storage layer
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/998 rebased from master --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1028: [Zeppelin-1022] Apply new mechanism to LivyInterpreter
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1028 I've tested it. 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 feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1034: [ZEPPELIN-1009] [HOTFIX] Fix Selenium test error
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1034 @Leemoonsoo Previous errors are gone, nice find --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1033: Python: fix for 'run all' paragraphs
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/1033 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 feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin pull request #1034: [ZEPPELIN-1009] [HOTFIX] Fix Selenium test erro...
GitHub user Leemoonsoo reopened a pull request: https://github.com/apache/zeppelin/pull/1034 [ZEPPELIN-1009] [HOTFIX] Fix Selenium test error ### What is this PR for? Fix selenium test error described in [ZEPPELIN-1009](https://issues.apache.org/jira/browse/ZEPPELIN-1009) ### What type of PR is it? Hot Fix ### What is the Jira issue? https://issues.apache.org/jira/browse/ZEPPELIN-1009 ### How should this be tested? Outline the steps to test the PR here. ### Questions: * Does the licenses files need update? no * Is there breaking changes for older versions? no * Does this needs documentation? no You can merge this pull request into a Git repository by running: $ git pull https://github.com/Leemoonsoo/zeppelin ZEPPELIN-1009 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/1034.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 #1034 commit 13dee87619c11ab8770ad81ea2693a41840d4192 Author: Lee moon sooDate: 2016-06-17T06:06:57Z fix --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin pull request #1034: [ZEPPELIN-1009] [HOTFIX] Fix Selenium test erro...
Github user Leemoonsoo closed the pull request at: https://github.com/apache/zeppelin/pull/1034 --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1032: ZEPPELIN-1027: Python - add basic matplotlib example n...
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1032 Yeah ð for @minahlee. And it 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 this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1032: ZEPPELIN-1027: Python - add basic matplotlib example n...
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1032 Makes perfect sense 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 this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin pull request #1033: Python: fix for 'run all' paragraphs
GitHub user bzz opened a pull request: https://github.com/apache/zeppelin/pull/1033 Python: fix for 'run all' paragraphs ### What is this PR for? Switch to FIFO scheduler as in current implementation `.interpret()` is not thread-safe and so in parallel one 'Run All' fails some paragraphs with NPE in logs ### What type of PR is it? Bug Fix | Improvement ### How should this be tested? 'Run All' passes without NPE in logs i.e on this [Zeppelin notebook for python](https://www.zeppelinhub.com/viewer/notebooks/aHR0cHM6Ly9yYXcuZ2l0aHVidXNlcmNvbnRlbnQuY29tL2J6ei9pbmN1YmF0b3ItemVwcGVsaW4vMTkyZjU3YjZjMGZkMjc4NzgwZDI3NDAzMGY1YmJlOTZlZThkNzdiYi9ub3RlYm9vay8yQlFBMzVDSlovbm90ZS5qc29u) ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation? No You can merge this pull request into a Git repository by running: $ git pull https://github.com/bzz/incubator-zeppelin fix/python-run-all Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/1033.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 #1033 commit 72e9d62406b8525ac81deef7a04165be03441e02 Author: Alexander BezzubovDate: 2016-06-17T04:59:50Z Python: switch to FIFO scheduler In current implementation `.interpret()` is not thread-safe and so in parallel one (esp on empty or graph paragraphs) 'Run All' fails with NPE in logs --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1032: ZEPPELIN-1027: Python - add basic matplotlib example n...
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1032 I'm planning to add at least one more tutorial notebook of using python, something similar to what we have for Apache Spark but with pandas and matplotlib etc. Do you guys think advanced one should be incorporated in here - :confused:, or shall we have a separate one ð , say title "Zeppelin Tutorial: Python - pandas, matplotlib". --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---