[GitHub] zeppelin pull request #2831: ZEPPELIN-3281. Apply getRelativePath when it is...
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2831 ---
[GitHub] zeppelin pull request #2831: ZEPPELIN-3281. Apply getRelativePath when it is...
GitHub user zjffdu opened a pull request: https://github.com/apache/zeppelin/pull/2831 ZEPPELIN-3281. Apply getRelativePath when it is LocalConfigStorage ### What is this PR for? When it is LocalConfigStorage, we should use getRelativePath to get the config path. ### What type of PR is it? [Bug Fix] ### Todos * [ ] - Task ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-3281 ### How should this be tested? * Unit test added ### 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/zjffdu/zeppelin ZEPPELIN-3281 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/2831.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 #2831 commit 34385776b7946e2615b1e4d9feaa9efa338df820 Author: Jeff Zhang Date: 2018-03-02T10:57:10Z ZEPPELIN-3281. Apply getRelativePath when it is LocalConfigStorage ---
[GitHub] zeppelin pull request #2830: [HOTFIX] Fix branch-0.8 build
GitHub user zjffdu opened a pull request: https://github.com/apache/zeppelin/pull/2830 [HOTFIX] Fix branch-0.8 build ### What is this PR for? This is a hotfix PR for branch-0.8 build ### What type of PR is it? [Bug Fix] ### Todos * [ ] - Task ### What is the Jira issue? * No jira created ### How should this be tested? * CI 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/zjffdu/zeppelin Hotfix_0.8 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/2830.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 #2830 commit 59c2af4577d9bde3b503a251de1836f71c71e026 Author: Jeff Zhang Date: 2018-02-27T13:23:36Z [HOTFIX] Fix branch-0.8 build ---
[jira] [Created] (ZEPPELIN-3286) Run All Paragraphs stops if there is a disabled paragraph mid-run
Ruslan Dautkhanov created ZEPPELIN-3286: --- Summary: Run All Paragraphs stops if there is a disabled paragraph mid-run Key: ZEPPELIN-3286 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3286 Project: Zeppelin Issue Type: Bug Affects Versions: 0.7.3, 0.8.0, 0.9.0 Reporter: Ruslan Dautkhanov Run All Paragraphs runs fine until it reaches a disabled paragraph in the middle of a run. How it should be processed: disabled paragraphs should be just skipped. What happens: disabled paragraph doesn't run (as expected), but it doesn't run non-disabled paragraphs after that as well (this is the bug). Also it seems make Zeppelin not able to run any paragraphs at all (even manually).. it's in some stuck state. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[GitHub] zeppelin issue #2825: [ZEPPELIN-1967]: New pull request
Github user sanjaydasgupta commented on the issue: https://github.com/apache/zeppelin/pull/2825 Created from wrong branch in source repo. Need to correct branch and reissue. ---
[GitHub] zeppelin pull request #2825: [ZEPPELIN-1967]: New pull request
Github user sanjaydasgupta closed the pull request at: https://github.com/apache/zeppelin/pull/2825 ---
[GitHub] zeppelin pull request #2829: [ZEPPELIN-3284] z.getInterpreterContext().out()...
GitHub user Leemoonsoo reopened a pull request: https://github.com/apache/zeppelin/pull/2829 [ZEPPELIN-3284] z.getInterpreterContext().out().clear() broken in Python interpreter ### What is this PR for? ``` %python import time print("Hello") time.sleep(0.5) # in case of Ipython kernel, print may not immediately flushed and cleared. z.getInterpreterContext().out().clear() print("world") ``` Expected to print `world` only. This worked in zeppelin-0.7.x and it'll be nice keep this feature in the future version. ### What type of PR is it? Bug fix | Improvement ### What is the Jira issue? https://issues.apache.org/jira/browse/ZEPPELIN-3284 ### How should this be tested? * First time? Setup Travis CI as described on https://zeppelin.apache.org/contribution/contributions.html#continuous-integration * Strongly recommended: add automated unit tests for any new or changed behavior * Outline any manual 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 ZEPPELIN-3284 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/2829.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 #2829 commit 224a2df8e64d203a5270f1948e013c14f78b52cb Author: Lee moon soo Date: 2018-03-03T00:50:22Z make z.getInterpreterContext().out().clear() work in PythonInterpreter commit e0dcda61fa55d794b8de2f74a61454b46506f5ed Author: Lee moon soo Date: 2018-03-03T01:42:41Z make z.getInterpreterContext().out().clear() work in IPythonInterprete ---
[GitHub] zeppelin pull request #2829: [ZEPPELIN-3284] z.getInterpreterContext().out()...
Github user Leemoonsoo closed the pull request at: https://github.com/apache/zeppelin/pull/2829 ---
[GitHub] zeppelin pull request #2829: [ZEPPELIN-3284] z.getInterpreterContext().out()...
Github user Leemoonsoo closed the pull request at: https://github.com/apache/zeppelin/pull/2829 ---
[GitHub] zeppelin pull request #2829: [ZEPPELIN-3284] z.getInterpreterContext().out()...
GitHub user Leemoonsoo reopened a pull request: https://github.com/apache/zeppelin/pull/2829 [ZEPPELIN-3284] z.getInterpreterContext().out().clear() broken in Python interpreter ### What is this PR for? ``` %python import time print("Hello") time.sleep(0.5) # in case of Ipython kernel, print may not immediately flushed and cleared. z.getInterpreterContext().out().clear() print("world") ``` Expected to print `world` only. This worked in zeppelin-0.7.x and it'll be nice keep this feature in the future version. ### What type of PR is it? Bug fix | Improvement ### What is the Jira issue? https://issues.apache.org/jira/browse/ZEPPELIN-3284 ### How should this be tested? * First time? Setup Travis CI as described on https://zeppelin.apache.org/contribution/contributions.html#continuous-integration * Strongly recommended: add automated unit tests for any new or changed behavior * Outline any manual 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 ZEPPELIN-3284 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/2829.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 #2829 commit 224a2df8e64d203a5270f1948e013c14f78b52cb Author: Lee moon soo Date: 2018-03-03T00:50:22Z make z.getInterpreterContext().out().clear() work in PythonInterpreter commit e0dcda61fa55d794b8de2f74a61454b46506f5ed Author: Lee moon soo Date: 2018-03-03T01:42:41Z make z.getInterpreterContext().out().clear() work in IPythonInterprete ---
[GitHub] zeppelin issue #2827: [HOTFIX] fix checkstyle errors
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/2827 @zjffdu sure, no problem. ---
[GitHub] zeppelin pull request #2827: [HOTFIX] fix checkstyle errors
Github user prabhjyotsingh closed the pull request at: https://github.com/apache/zeppelin/pull/2827 ---
[GitHub] zeppelin issue #2827: [HOTFIX] fix checkstyle errors
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2827 Thanks @prabhjyotsingh , I have already started one hotfix PR for building issue of branch-0.8, there's more issues except code style, do you mind to close this PR and I will fix it in one PR. https://github.com/zjffdu/zeppelin/commit/0ef79d1807599d2fbc74e8eeb3e18d0997aa7b85 ---
[GitHub] zeppelin issue #2827: [HOTFIX] fix checkstyle errors
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2827 LGTM ---
[GitHub] zeppelin pull request #2829: [ZEPPELIN-3284] z.getInterpreterContext().out()...
GitHub user Leemoonsoo opened a pull request: https://github.com/apache/zeppelin/pull/2829 [ZEPPELIN-3284] z.getInterpreterContext().out().clear() broken in Python interpreter ### What is this PR for? ``` %python import time print("Hello") time.sleep(0.5) # in case of Ipython kernel, print may not immediately flushed and cleared. z.getInterpreterContext().out().clear() print("world") ``` Expected to print `world` only. This worked in zeppelin-0.7.x and it'll be nice keep this feature in the future version. ### What type of PR is it? Bug fix | Improvement ### What is the Jira issue? https://issues.apache.org/jira/browse/ZEPPELIN-3284 ### How should this be tested? * First time? Setup Travis CI as described on https://zeppelin.apache.org/contribution/contributions.html#continuous-integration * Strongly recommended: add automated unit tests for any new or changed behavior * Outline any manual 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 ZEPPELIN-3284 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/2829.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 #2829 commit 224a2df8e64d203a5270f1948e013c14f78b52cb Author: Lee moon soo Date: 2018-03-03T00:50:22Z make z.getInterpreterContext().out().clear() work in PythonInterpreter commit e0dcda61fa55d794b8de2f74a61454b46506f5ed Author: Lee moon soo Date: 2018-03-03T01:42:41Z make z.getInterpreterContext().out().clear() work in IPythonInterprete ---
[GitHub] zeppelin pull request #2828: [ZEPPELIN-3283] Remove redundant method Interpr...
GitHub user Leemoonsoo opened a pull request: https://github.com/apache/zeppelin/pull/2828 [ZEPPELIN-3283] Remove redundant method InterpreterOutput.getInterpreterResultMessages() ### What is this PR for? Remove Redundant method `InterpreterOutput.getInterpreterResultMessages()` ### What type of PR is it? Refactoring ### What is the Jira issue? https://issues.apache.org/jira/browse/ZEPPELIN-3283 ### 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-3283 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/2828.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 #2828 commit 82bda9c02b3d470d8bff4d86468717fc1564 Author: Lee moon soo Date: 2018-03-02T23:40:28Z Remove redundant method getInterpreterResultMessages() commit 12771a1454638e5c4aae428764d595d0c6c1c895 Author: Lee moon soo Date: 2018-03-02T23:47:35Z remove unused import ---
[GitHub] zeppelin issue #2763: [ZEPPELIN-3194][NEW-INTERPRETER] SAP Universe interpre...
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2763 Thanks @masyan for great contribution. Looks good to me. I'll merge this to master and branch-0.8 if no further comments. ---
[GitHub] zeppelin issue #2817: [ZEPPELIN-3264] Notebook Snapshot feature.
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2817 Interesting feature, thanks @SleepyThread for the contribution. ---
[jira] [Created] (ZEPPELIN-3285) Unnecessary margin before the title of note
Lee moon soo created ZEPPELIN-3285: -- Summary: Unnecessary margin before the title of note Key: ZEPPELIN-3285 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3285 Project: Zeppelin Issue Type: Bug Affects Versions: 0.8.0 Reporter: Lee moon soo Attachments: image-2018-03-02-15-59-39-827.png There's unnecessary margin before the title. !image-2018-03-02-15-59-39-827.png! -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Created] (ZEPPELIN-3284) z.getInterpreterContext().out().clear() broken in Python interpreter
Lee moon soo created ZEPPELIN-3284: -- Summary: z.getInterpreterContext().out().clear() broken in Python interpreter Key: ZEPPELIN-3284 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3284 Project: Zeppelin Issue Type: Bug Affects Versions: 0.8.0 Reporter: Lee moon soo Attachments: image-2018-03-02-15-54-38-942.png Zeppelin 0.7.x, %spark and %python interpreter able to clear output using z.getInterpreterContext().out().clear() !image-2018-03-02-15-54-38-942.png! However, master branch and branch-0.8, PythonInterpreter does not provide the same capability. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Created] (ZEPPELIN-3283) InterpreterOutput.toInterpreterResultMessage() and InterpreterOutput.getInterpreterResultMessages() are redundant
Lee moon soo created ZEPPELIN-3283: -- Summary: InterpreterOutput.toInterpreterResultMessage() and InterpreterOutput.getInterpreterResultMessages() are redundant Key: ZEPPELIN-3283 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3283 Project: Zeppelin Issue Type: Improvement Reporter: Lee moon soo org.apache.zeppelin.interpreter.InterpreterOutput.toInterpreterResultMessage() and org.apache.zeppelin.interpreter.InterpreterOutput.getInterpreterResultMessages() are redundant -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Created] (ZEPPELIN-3282) pyspark as a Default Interpreter
Ruslan Dautkhanov created ZEPPELIN-3282: --- Summary: pyspark as a Default Interpreter Key: ZEPPELIN-3282 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3282 Project: Zeppelin Issue Type: New Feature Components: front-end, zeppelin-server Affects Versions: 0.8.0 Reporter: Ruslan Dautkhanov Attachments: image-2018-03-02-13-26-39-913.png I can't choose PySpark as a default Interpreter when creating a new note.. is this a new bug or am I missing some configuration? !image-2018-03-02-13-26-39-913.png! -- This message was sent by Atlassian JIRA (v7.6.3#76005)
Unmodifiable interpreter properties
Hi fellow Zeppelin users. I would like to know if is there a way in zeppelin to set interpreter properties that can not be changed by the user from the graphic interface. An example use case in which this can be useful is if we want that zeppelin users can not kill jobs from the spark ui; for this you must modify the property "spark.ui.killEnabled" with value FALSE. The problem is that the user can change this property from the interpreter screen and enable the feature again. I am wondering if maybe is there some attribute in the properties registered in interpreter-setting.json that makes that property readonly (for end user) or something like that. Thank you in advance!.
[GitHub] zeppelin pull request #2827: [HOTFIX] fix checkstyle errors
GitHub user prabhjyotsingh opened a pull request: https://github.com/apache/zeppelin/pull/2827 [HOTFIX] fix checkstyle errors ### What is this PR for? There are some checkstyle errors in branch-0.8, this is to fix all those errors. ### What type of PR is it? [Hot Fix] ### How should this be tested? * CI should be green. ### Questions: * Does the licenses files need update? N/A * Is there breaking changes for older versions? N/A * Does this needs documentation? N/A You can merge this pull request into a Git repository by running: $ git pull https://github.com/prabhjyotsingh/zeppelin hotfix/branch-0.8_checkstyle_errors Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/2827.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 #2827 commit 1c94e5cd9d81ae18861b375056db74f5dfe36597 Author: Prabhjyot Singh Date: 2018-03-02T15:22:20Z fix checkstyle errors Change-Id: I0b24f9287f4a8d63d18d55863505370b70c3659f ---
[GitHub] zeppelin issue #2827: [HOTFIX] fix checkstyle errors
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/2827 @r-kamath @zjffdu Please help review this. ---
[GitHub] zeppelin issue #2816: [ZEPPELIN-3263] Do not use Firefox >=55 for integratio...
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/2816 I understand Selenium, Gecko, WebExtensions are constantly evolving. However, with this (https://github.com/apache/zeppelin/pull/2826/files), I'm able to make all the Selenium test case pass on FF-58. This PR is still work in progress. Also, there is stuff that will get fixed with FF-59. ---
[GitHub] zeppelin pull request #2826: [ZEPPELIN-3276] Upgrade Firefox version in Trav...
Github user prabhjyotsingh closed the pull request at: https://github.com/apache/zeppelin/pull/2826 ---
[GitHub] zeppelin pull request #2826: [ZEPPELIN-3276] Upgrade Firefox version in Trav...
GitHub user prabhjyotsingh opened a pull request: https://github.com/apache/zeppelin/pull/2826 [ZEPPELIN-3276] Upgrade Firefox version in Travis CI ### What is this PR for? Upgrade Firefox version in Travis CI ### What type of PR is it? [Improvement] ### What is the Jira issue? * [ZEPPELIN-3276](https://issues.apache.org/jira/browse/ZEPPELIN-3276) ### How should this be tested? * CI should be green. ### Questions: * Does the licenses files need update? N/A * Is there breaking changes for older versions? N/A * Does this needs documentation? N/A You can merge this pull request into a Git repository by running: $ git pull https://github.com/prabhjyotsingh/zeppelin ZEPPELIN-3276 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/2826.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 #2826 commit ffb03e10ad1d90dad9057bb008e4b930fda4bbce Author: Prabhjyot Singh Date: 2018-03-01T21:15:51Z upgrade FF to 58 ---
[jira] [Created] (ZEPPELIN-3281) Only apply getRelativePath when it is LocalConfigStorage
Jeff Zhang created ZEPPELIN-3281: Summary: Only apply getRelativePath when it is LocalConfigStorage Key: ZEPPELIN-3281 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3281 Project: Zeppelin Issue Type: Improvement Reporter: Jeff Zhang -- This message was sent by Atlassian JIRA (v7.6.3#76005)