[GitHub] zeppelin issue #1408: [ZEPPELIN-808] Changed Permissions on Notebooks do not...
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-trigger it? --- 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 #1393: [ZEPPELIN-1142] Zeppelin allows two users to simultane...
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 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 #1330: [ZEPPELIN-1144]Zeppelin home page should only list not...
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) correspondingly --- 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-1438) Per user note list broadcast on note permission change
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 Issue Type: Bug Reporter: Khalid Huseynov Fix For: 0.7.0 after https://github.com/apache/zeppelin/pull/1330 is merged, users will be able to receive their own list of notes, however that list is currently loaded automatically only once when starting Zeppelin (also possible by clicking reload notes). Thus, we need to address updating other users list of notes (workbench) on the fly when some note permissions are changed. For example if user1 gave read permissions to noteA for user2, noteA should appear immediately in user2's working directory of notes. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Created] (ZEPPELIN-1437) Note list broadcast is not aware of user session
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 Issue Type: Bug Reporter: Khalid Huseynov Fix For: 0.7.0 after https://github.com/apache/zeppelin/pull/1330 each user will receive his specific list, but if user refreshes his list to reload from storage, that list will be broadcasted to everyone since broadcast messages are still not aware of user sessions. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[GitHub] zeppelin issue #1423: ZEPPELIN-1425. sparkr.zip is not distributed to execut...
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. 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 #1411: [ZEPPELIN-1417] Bugfix of "Connect to existing process...
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 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 #1416: Rewrite of "Quick Start" page.
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 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 #1418: [gh-pages][ZEPPELIN-990] Add header anchor to gh-pages...
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 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 #1426: ZEPPELIN-1430. Display appId and webui link in LivyInt...
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 output want to use different zeppelin display system, like %table, %html, etc, it may be conflict with the feature this PR trying to provide. I think better approach would be [support multiple interpreter result]( https://issues.apache.org/jira/browse/ZEPPELIN-212) and display additional informations as a one of separate interpreter result with it's own result type. Although it'll going to be very large work, i think resolving ZEPPELIN-212 and build this feature on top of it, is right approach. --- 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 #1384: [ZEPPELIN-1387] Support table syntax in markdown inter...
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. 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 #1423: ZEPPELIN-1425. sparkr.zip is not distributed to execut...
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? --- 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 #1425: Improving Matplotlib Integration with Zeppelin
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 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 #1422: Allow matplotlib plot display in pyspark context
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 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 #1330: [ZEPPELIN-1144]Zeppelin home page should only list not...
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 message is not aware user session, so notebook list can display notes that supposed to not see when other user refresh the notebook list. - When a user changes the permission of a note, note list need to be broadcasted immediately so all connected users can have updated list of notebook. Otherwise, there's still some chances that a user able to see some note on notebook list that they can not access. --- 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 #1393: [ZEPPELIN-1142] Zeppelin allows two users to simultane...
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 defined when creating note. After, owner is set to current user when creating note. (when authentication is enabled) Especially with code changes such as https://github.com/apache/zeppelin/pull/1330, user will experience bigger behavior changes. But i think having current user as a owner when creating note is natural behavior that most people can think, even though it brings some UX changes. LGTM and 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 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 #1413: [MINOR] Format Fix for InterpreterResultTest.ja...
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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1404: ZEPPELIN-1411. UDF with pyspark not working - object h...
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.zeppelin.rest.ZeppelinSparkClusterTest pySparkTest(org.apache.zeppelin.rest.ZeppelinSparkClusterTest) Time elapsed: 3.842 sec <<< FAILURE! java.lang.AssertionError: expected: but was: at org.junit.Assert.fail(Assert.java:88) at org.junit.Assert.failNotEquals(Assert.java:743) at org.junit.Assert.assertEquals(Assert.java:118) at org.junit.Assert.assertEquals(Assert.java:144) at org.apache.zeppelin.rest.ZeppelinSparkClusterTest.pySparkTest(ZeppelinSparkClusterTest.java:150) ``` Could you check? --- 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 #1302: [ZEPPELIN-1306] Interpreter restarts on a note.
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 ability. So i think it make sense to have a unittest. Also, shell we have https://github.com/apache/zeppelin/blob/master/docs/rest-api/rest-interpreter.md#restart-an-interpreter updated, too? --- 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 #1427: [ZEPPELIN-1435] Move zeppelin-server-*.jar unde...
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 modules such as `zeppelin-engine`, `zeppelin-interpreter` ### What type of PR is it? Refactoring ### What is the Jira issue? [ZEPPELIN-1435](https://issues.apache.org/jira/browse/ZEPPELIN-1435) ### How should this be tested? Build with: ``` mvn clean package -DskipTests -Pbuild-distr -pl '!alluxio,!angular,!cassandra,!elasticsearch,!file,!flink,!hbase,!ignite,!jdbc,!kylin,!lens,!livy,!postgresql,!python,!shell,!bigquery' ``` and check if `zeppelin-server-0.7.0-SNAPSHOT.jar` file is placed under `zeppelin-distribution/target/zeppelin-0.7.0-SNAPSHOT/zeppelin-0.7.0-SNAPSHOT/lib` not `zeppelin-distribution/target/zeppelin-0.7.0-SNAPSHOT/zeppelin-0.7.0-SNAPSHOT` ### 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/minahlee/zeppelin ZEPPELIN-1435 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/1427.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 #1427 commit 03a018f7dbb6f43ab72a71735e0e1203e716f73a Author: Mina LeeDate: 2016-09-13T13:42:03Z Move zeppelin-server-*.jar under lib in distribution binary package --- 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-1435) Move zeppelin-server-*.jar under lib directory in distribution package
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: Zeppelin Issue Type: Improvement Reporter: Mina Lee Assignee: Mina Lee Fix For: 0.7.0 This issue has brought up by [~jonathak] in comment on https://issues.apache.org/jira/browse/ZEPPELIN-1250. Every zeppelin-*.jar packages are placed under lib except zeppelin-server-*.jar. It makes more sense to keep all zeppelin-*.jar packages in same place. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[GitHub] zeppelin pull request #1425: Improving Matplotlib Integration with Zeppelin
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 is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin pull request #1425: Improving Matplotlib Integration with Zeppelin
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 ### What is the Jira issue? https://issues.apache.org/jira/browse/ZEPPELIN-1428 ### How should this be tested? show() functionality in the PyZeppelinContext class should work with matplotlib. ### 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/IshmeetKaur/zeppelin master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/1425.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 #1425 commit 1e3a5088011c0cca4844404d5edc87d18f34b517 Author: Ishmeet KaurDate: 2016-09-06T14:44:16Z Show function modification pyspark commit 561860a2321409c596397e2eb3366ff7c1820e51 Author: Ishmeet Kaur Date: 2016-09-07T15:52:35Z show function modification pyspark commit 6491ff2839e2e1a94fbbbd30727bf821cdb69805 Author: Ishmeet Kaur Date: 2016-09-07T19:52:44Z show function modification commit 645e6af18dd754a0ebcbe5ff040fd0a9d2346de7 Author: Ishmeet Kaur Date: 2016-09-08T14:04:01Z Merge branch 'master' of https://github.com/apache/zeppelin into modify_show commit 207d6dd0dbe79f66b34088ff96db1fa8835ddb61 Author: Ishmeet Kaur Date: 2016-09-08T14:31:59Z Revert "show function modification pyspark" This reverts commit 561860a2321409c596397e2eb3366ff7c1820e51. commit d42d49b0b820aa24024677b7910b40e7025efebd Author: Ishmeet Kaur Date: 2016-09-08T14:42:58Z show function modification commit 3224184c51886ea286bd41dc9c9c34b09d9bffe9 Author: Ishmeet Kaur Date: 2016-09-08T15:45:59Z combining show and show_plot commit 0faf2aa2b3b979f352a7a1f8f449d47a24651a89 Author: Ishmeet Kaur Date: 2016-09-09T08:06:55Z dataframe pandas dependency commit 1d3583a72918c905a783d32145b4bea3826a637b Author: Ishmeet Kaur Date: 2016-09-10T09:22:10Z pandas dataframe dependency resolution commit cd3e111383b261b1def068e8ec9fbacf73132fe9 Author: Ishmeet Kaur Date: 2016-09-10T15:50:13Z Dataframe pandas module resolution commit 3ee2555885ed5ea872cd6ef52d43ab44d0107418 Author: ishmeetkaur Date: 2016-09-11T22:04:39Z dataframe function removed commit 4ff14c1fe089c76f64ad3092ead819bebb80 Author: ishmeetkaur Date: 2016-09-11T22:14:37Z Merge remote-tracking branch 'upstream/master' into modify_show commit 3e152fc675b31c00b957974d50c2710134bea490 Author: ishmeetkaur Date: 2016-09-12T13:10:47Z Merge remote-tracking branch 'origin/master' into modify_show commit 9dc5dc01662e5c89b02a93cb5b4483c236e374d9 Author: ishmeetkaur Date: 2016-09-12T13:24:35Z removing whitespace commit d991fd511c69134ffaa1251f802aababa42d66f5 Author: IshmeetKaur Date: 2016-09-12T14:54:23Z Merge pull request #1 from IshmeetKaur/modify_show Modify show --- 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-1434) Long numbers rounding in Zeppelin-Web table
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 Type: Bug Reporter: David Tzoor Priority: Critical zeppelin-web is using {{Handsontable}} package to generate the table view within the notebooks' paragraphs. In the [paragraph controller|https://github.com/apache/zeppelin/blob/master/zeppelin-web/src/app/notebook/paragraph/paragraph.controller.js], within the Handsontable object instantiation, the {{cells}} function checks if the given value is a number (using {{!isNan(value)}} method). If the value is indeed a number, it will render as follows: {code:javascript} else if (!isNaN(value)) { cellProperties.format = '0,0.[0]'; td.style.textAlign = 'left'; Handsontable.renderers.NumericRenderer.apply(this, arguments); } {code} Since javascript supports numbers between {{-(2^53^ - 1)}} and {{(2^53^ - 1)}}, any number not within this range, will be rounded (up or down) and wrong numbers will be shown. I suggest to add another condition to check whether the number is a safe number: {code:javascript} Number.isSafeInteger(parseInt(value)) {code} In that case, just render the value as a string (no formatting). The second issue is with very small floats. The current rendering will only show up to 5 decimal places. There should be another condition to check if the value is a float, and if so use parseFloat or Number(value) to render it with the e notation: {code:javascript} function isFloat(a) { return a.indexOf('.') !== -1; } ... if (isFloat(value)) { parseFloat(value); } {code} -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[GitHub] zeppelin issue #1398: [MINOR] Doc fix for spark interpreter
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 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-1433) What dependency to add to project (sbt or maven) get hold of ZeppelinContext?
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 Issue Type: Improvement Reporter: sam Zeppelin has an object `ZeppelinContext`, which can then be used to share state between languages and bind variables to angular and thus create cool user interfaces inside Zeppelin notebooks. We have written numerous convenience methods to create things like drop down menus, buttons, UI stuff, from scala. These methods call `ZeppelinContext`. We wish to add these methods to an sbt project, so that we can package them in a jar, but it seems the Zeppelin project provides no artifact that contains `ZeppelinContext` (we have tried many). Rather there only seems to exist two work arounds: 1. Build all of Zeppelin and add the resulting jar as an unmanaged jar (not nice). 2. Use duck typing (also really not nice). Is there a lesser known resolver / artifact id to get hold of this type?? If there is no such publically accessible module please could such a module be made available? -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[GitHub] zeppelin pull request #1398: [MINOR] Doc fix for spark interpreter
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 group of note. So I update the doc to use the full interpreter name. * Add SparkSession for 2.0 * Also add comments inline with other changes to explain the reason. ### What type of PR is it? [Documentation] ### Todos * [ ] - Task ### What is the Jira issue? * No jira created. ### 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/zjffdu/zeppelin spark_doc_fix Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/1398.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 #1398 commit 40d4b11e29130fd8256af5c7848a1c7e7013c793 Author: Jeff ZhangDate: 2016-09-02T09:30:43Z [MINOR] Doc fix for spark interpreter --- 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 #1398: [MINOR] Doc fix for spark interpreter
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 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 #1398: [MINOR] Doc fix for spark interpreter
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 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 #1398: [MINOR] Doc fix for spark interpreter
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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1404: ZEPPELIN-1411. UDF with pyspark not working - object h...
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. --- 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 #1412: [Zeppelin 1415] - Adding Zeppelin-Web Good Prac...
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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1302: [ZEPPELIN-1306] Interpreter restarts on a note.
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 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 #1426: ZEPPELIN-1430. Display appId and webui link in LivyInt...
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 returns result. - and I see empty line between "1" and "2" where I wrote "Thread.sleep()", not a big deal, but it wasn't earlier. Before: ![master](https://cloud.githubusercontent.com/assets/674497/18464749/8cb27b7e-79b0-11e6-91d8-cfbdf8a8af1a.gif) After: ![with-1430](https://cloud.githubusercontent.com/assets/674497/18464750/8cb63746-79b0-11e6-8e41-3fd58586c541.gif) --- 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 #1424: [ZEPPELIN-1426] User aware storage sync
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 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 #1330: [ZEPPELIN-1144]Zeppelin home page should only list not...
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1330 Thank you @mwkang @khalidhuseynov ! 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 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 #1426: ZEPPELIN-1430. Display appId and webui link in ...
GitHub user zjffdu opened a pull request: https://github.com/apache/zeppelin/pull/1426 ZEPPELIN-1430. Display appId and webui link in LivyInterpreter's output ### What is this PR for? For now, it is hard to figure out what the yarn application of the livy session represent, it would be better to display the appId and webui link in the output of LivyInterpreter for diagnosing purpose. It can also be applied to the native SparkInterpreter, but could be done in another ticket. ### What type of PR is it? [Feature] ### Todos * [ ] - Task ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-1430 ### How should this be tested? Tested manually ### Screenshots (if appropriate) ![image](https://cloud.githubusercontent.com/assets/164491/1846/e4eab580-79bb-11e6-8c8d-393ab6805638.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/zjffdu/zeppelin ZEPPELIN-1430 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/1426.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 #1426 commit 6fe78b0dd1222d5d1b87b8a2b192aa94df989a90 Author: Jeff ZhangDate: 2016-09-13T06:07:02Z ZEPPELIN-1430. Display appId and webui link in LivyInterpreter's output --- 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 #1330: [ZEPPELIN-1144]Zeppelin home page should only list not...
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/1330 I think it doesn't solve all problems, but definitely in the roadmap. 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. ---