[jira] [Created] (ZEPPELIN-3276) Upgrade Firefox version in Travis CI
Prabhjyot Singh created ZEPPELIN-3276: - Summary: Upgrade Firefox version in Travis CI Key: ZEPPELIN-3276 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3276 Project: Zeppelin Issue Type: Bug Components: CI-infra Affects Versions: 0.8.0, 0.9.0 Reporter: Prabhjyot Singh Assignee: Prabhjyot Singh Upgrade Firefox version to 58 -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Created] (ZEPPELIN-3277) NotebookServerTest.testMakeSureNoAngularObjectBroadcastToWebsocketWhoFireTheEvent fails
Jeff Zhang created ZEPPELIN-3277: Summary: NotebookServerTest.testMakeSureNoAngularObjectBroadcastToWebsocketWhoFireTheEvent fails Key: ZEPPELIN-3277 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3277 Project: Zeppelin Issue Type: Sub-task Reporter: Jeff Zhang {code:java} Failed tests: NotebookServerTest.testMakeSureNoAngularObjectBroadcastToWebsocketWhoFireTheEvent:157 notebookSocket.send(); Never wanted here: -> at org.apache.zeppelin.socket.NotebookServerTest.testMakeSureNoAngularObjectBroadcastToWebsocketWhoFireTheEvent(NotebookServerTest.java:157) But invoked here: -> at org.apache.zeppelin.socket.NotebookServer.broadcast(NotebookServer.java:487){code} -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[GitHub] zeppelin issue #2821: [ZEPPELIN-3271] Option for disabling scheduler
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/2821 Have pushed another commit that disables Cron/scheduler by default, and user needs this feature, will need to with config. ---
[GitHub] zeppelin issue #2821: [ZEPPELIN-3271] Option for disabling scheduler
Github user zjffdu2 commented on the issue: https://github.com/apache/zeppelin/pull/2821 @prabhjyotsingh Shouldn't it be turned on by default for backward compatibility ? ---
[GitHub] zeppelin issue #2821: [ZEPPELIN-3271] Option for disabling scheduler
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/2821 Yes, I've updated the doc (https://github.com/apache/zeppelin/pull/2821/files#diff-bed1be4fb1f7f6c8e3bd6db18d72dc44R54). and yes, I agree we should document this in our release notes as well. ---
[GitHub] zeppelin pull request #2763: [ZEPPELIN-3194][NEW-INTERPRETER] SAP Universe i...
GitHub user masyan reopened a pull request: https://github.com/apache/zeppelin/pull/2763 [ZEPPELIN-3194][NEW-INTERPRETER] SAP Universe interpreter ### What is this PR for? New interpreter, SAP BusinessObjects (Universes). See [documentation](https://github.com/masyan/zeppelin/blob/9178d1cc80253b9ae58b988443b619127f42d945/docs/interpreter/sap.md) ### What type of PR is it? [Feature] ### What is the Jira issue? https://issues.apache.org/jira/browse/ZEPPELIN-3194 ### How should this be tested? see screenshots ### Screenshots (if appropriate) Autocomplete + get results ![comletion_and_results](https://user-images.githubusercontent.com/705100/35770106-1fb3e9e6-0937-11e8-9ee4-df0c06a7b6c3.gif) Errors ![errors](https://user-images.githubusercontent.com/705100/35770108-23de9674-0937-11e8-89e1-a0d3380e88bb.gif) ### 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/masyan/zeppelin ZEPPELIN-3194 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/2763.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 #2763 commit f6b71f9c2a776c32972e14029f79638d364ccb9f Author: Maksim Reshetov Date: 2018-01-31T09:49:44Z [ZEPPELIN-3194] is null fix commit c1c09af5c014ca3c6410ed5dbeecc81738188b39 Author: Maksim Reshetov Date: 2018-01-31T11:01:59Z [ZEPPELIN-3194] docs fix + unit tests for util commit 1aeb7ad0f14c470253e668d32c9bac7757768e2d Author: Maksim Reshetov Date: 2018-01-31T11:11:39Z [ZEPPELIN-3194] pom license commit e00ace2e3efdc869c34d9e3bafe353f0eaa45f77 Author: Maksim Reshetov Date: 2018-01-31T15:32:30Z [ZEPPELIN-3194] completer tests commit 0784ba0303aaf50d438fe18729f219fa5df0e9b1 Author: Maksim Reshetov Date: 2018-02-01T14:30:14Z [ZEPPELIN-3194] docs commit ce97551b9611574ad3f68f49837a0acdcd218f1b Author: Maksim Reshetov Date: 2018-02-03T14:29:37Z Merge remote-tracking branch 'upstream/master' into ZEPPELIN-3194 commit 9178d1cc80253b9ae58b988443b619127f42d945 Author: Maksim Reshetov Date: 2018-02-03T14:39:16Z [ZEPPELIN-3194] docs commit 24336a85559873d37d04667e48cbcaeeea5d7cc1 Author: Maksim Reshetov Date: 2018-02-16T12:44:20Z [ZEPPELIN-3194] fix parser for result objects and conditions commit 1e25fd1eed29c443351419090c2a1abc7c3de40f Author: Maksim Reshetov Date: 2018-03-01T12:29:16Z [ZEPPELIN-3194] refactor parser os query. add tests ---
[GitHub] zeppelin issue #2817: [ZEPPELIN-3264] Notebook Snapshot feature.
Github user Tagar commented on the issue: https://github.com/apache/zeppelin/pull/2817 @SleepyThread - great feature! Building on @felixcheung's comment, it would be good if Snapshot feature would use org.apache.zeppelin.notebook.repo.GitNotebookRepo to store a snapshot/revision, so then you could compare revisions etc. ps. Looks like commits should be slashed? It shows unrelated commits in this PR. ---
[GitHub] zeppelin issue #2819: ZEPPELIN-3265. DevInterpreter doesn't work
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2819 @zjffdu this PR creates duplicate interpreters in interpreter.json. after upgrading to this commit, all installed interpreters are duplicated, e.g. %angular, %md, %spark, %python, ... . Is this the intended behavior? ---
[GitHub] zeppelin issue #2819: ZEPPELIN-3265. DevInterpreter doesn't work
Github user zjffdu2 commented on the issue: https://github.com/apache/zeppelin/pull/2819 oops, this should be a bug, let me fix it asap. ---
[jira] [Created] (ZEPPELIN-3278) Avoid duplicated interpreter setting
Jeff Zhang created ZEPPELIN-3278: Summary: Avoid duplicated interpreter setting Key: ZEPPELIN-3278 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3278 Project: Zeppelin Issue Type: Bug Reporter: Jeff Zhang -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[GitHub] zeppelin pull request #2824: ZEPPELIN-3278. Avoid duplicated interpreter set...
GitHub user zjffdu opened a pull request: https://github.com/apache/zeppelin/pull/2824 ZEPPELIN-3278. Avoid duplicated interpreter setting ### What is this PR for? Straightforward bugfix for ZEPPELIN-3278. Just don't create interpreter setting instance when registering from template. ### What type of PR is it? [Bug Fix] ### Todos * [ ] - Task ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-3278 ### 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 ZEPPELIN-3278 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/2824.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 #2824 commit c33cff652d404290a22f942f43cc78892596083c Author: Jeff Zhang Date: 2018-03-02T01:47:56Z [HOTFIX] Avoid duplicated interpreter setting ---
[GitHub] zeppelin issue #2824: ZEPPELIN-3278. Avoid duplicated interpreter setting
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2824 @weand Could you help try this PR ? Thanks ---
[jira] [Created] (ZEPPELIN-3279) [FlakyTest] NotebookTest.testPerSessionInterpreterCloseOnUnbindInterpreterSetting
Jeff Zhang created ZEPPELIN-3279: Summary: [FlakyTest] NotebookTest.testPerSessionInterpreterCloseOnUnbindInterpreterSetting Key: ZEPPELIN-3279 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3279 Project: Zeppelin Issue Type: Sub-task Reporter: Jeff Zhang {code:java} Failed tests: NotebookTest.testPerSessionInterpreterCloseOnUnbindInterpreterSetting:1095 Values should be different. Actual: 1306912768{code} -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[GitHub] zeppelin pull request #2821: [ZEPPELIN-3271] Option for disabling scheduler
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2821 ---
[jira] [Created] (ZEPPELIN-3280) Geode test is not covered in travis and it doesn't pass actually
Jeff Zhang created ZEPPELIN-3280: Summary: Geode test is not covered in travis and it doesn't pass actually Key: ZEPPELIN-3280 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3280 Project: Zeppelin Issue Type: Bug Reporter: Jeff Zhang -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[GitHub] zeppelin issue #2763: [ZEPPELIN-3194][NEW-INTERPRETER] SAP Universe interpre...
Github user masyan commented on the issue: https://github.com/apache/zeppelin/pull/2763 @Leemoonsoo I fixed the parser. CI red because of unrelated code. Ready to merge ---
[GitHub] zeppelin pull request #2823: ZEPPELIN-3273. SparkRInterpreter doesn't work i...
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2823 ---
[GitHub] zeppelin pull request #2824: ZEPPELIN-3278. Avoid duplicated interpreter set...
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2824 ---
[GitHub] zeppelin issue #2824: ZEPPELIN-3278. Avoid duplicated interpreter setting
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2824 @weand I have committed it for 0.8.0 release, feel free to open the ticket if it doesn't resolve your issue ---
[GitHub] zeppelin pull request #2825: [ZEPPELIN-1967]: New pull request
GitHub user sanjaydasgupta opened a pull request: https://github.com/apache/zeppelin/pull/2825 [ZEPPELIN-1967]: New pull request ### What is this PR for? The code in this PR enables embedding/interpolating Z variables into command strings passed to Spark's SQL and Shell interpreters. It implements the functionality described in issue [ZEPPELIN-1967](https://issues.apache.org/jira/browse/ZEPPELIN-1967) This PR resumes a fresh effort while taking into consideration all of the discussion in the context of the earlier [PR-2502](https://github.com/apache/zeppelin/pull/2502). The earlier PR-2502 was closed due to a corruption in my repo that could not be corrected. The code in this PR resolves all of the discussion and suggestions in the body of the earlier [PR-2502](https://github.com/apache/zeppelin/pull/2502). The following description is a summary of the current implementation: Patterns of the form `{var-name}` within commands will be interpolated only if a predefined object of the specified name exists in `z`. Additionally, **all** such patterns within the command line should also be translatable for any interpolation to occur. Partial translation of a command line (where only some of the patterns are translated, and others are not) is never performed. Patterns of the form `{{any-text}}` are translated into `{any-text}`. These translations are performed only when all occurrences of {, }, {{, and }} in any command string confirm to one of the two forms described above. A command containing { and/or } characters used in any other way (than `{var-name}` and `{{any-text}}` described above) is used as-is without any changes. No error is flagged in any case. This behavior is identical to the implementation of a similar feature in Jupyter's shell invocation using the ! magic command. At present only the SQL and Shell interpreters support object interpolation. ### What type of PR is it? [Improvement] ### Todos * [ ] - Task ### What is the Jira issue? https://issues.apache.org/jira/browse/ZEPPELIN-1967 ### How should this be tested? A new unit-test class ([ZeppCtxtVariableTest.java](https://github.com/apache/zeppelin/compare/master...sanjaydasgupta:master#diff-ad942153fbbad8ac1954b1063eabf920)) has been added. The attached screenshots also show tests of the functionality. ### Screenshots (if appropriate) ![figure-1](https://user-images.githubusercontent.com/477015/36888230-9104d3c4-1e1a-11e8-9399-f98b1246730b.png) ![figure-2](https://user-images.githubusercontent.com/477015/36888237-96843bbe-1e1a-11e8-80ac-4bdfe94ead22.png) ![figure-3](https://user-images.githubusercontent.com/477015/36888240-99fe9c3a-1e1a-11e8-9aa0-9faa32c0fa19.png) ![figure-4](https://user-images.githubusercontent.com/477015/36888246-9dd86f34-1e1a-11e8-8151-4ece9d8af8e8.png) ![figure-5](https://user-images.githubusercontent.com/477015/36888251-a1d21644-1e1a-11e8-8431-c5e59773f60c.png) ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation? Yes, detailed documentation has been added to the part describing ZeppelinContext variables. You can merge this pull request into a Git repository by running: $ git pull https://github.com/sanjaydasgupta/zeppelin master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/2825.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 #2825 commit a570cf45150303d9effe5a2b2fa09b8611c6312d Author: Sanjay Dasgupta Date: 2018-03-02T07:13:14Z ZEPPELIN-1967: New pull request ---