[GitHub] zeppelin pull request #2752: [ZEPPELIN-3195] Remove the limit on the number ...
Github user mebelousov closed the pull request at: https://github.com/apache/zeppelin/pull/2752 ---
[GitHub] zeppelin pull request #2783: ZEPPELIN-3150. Fixed Checkstyle errors and warn...
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2783 ---
[GitHub] zeppelin pull request #2738: [ZEPPELIN-3182] Support saving notebooks to Goo...
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2738 ---
[GitHub] zeppelin pull request #2762: [ZEPPELIN-3204] FIX: cursor in paragraph editor...
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2762 ---
[GitHub] zeppelin issue #2783: ZEPPELIN-3150. Fixed Checkstyle errors and warnings in...
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2783 LGTM ---
[GitHub] zeppelin pull request #2787: ZEPPELIN-3153. Fixed Checkstyle errors and warn...
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2787 ---
[GitHub] zeppelin issue #2786: ZEPPELIN-3152. Fixed Checkstyle errors and warnings in...
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2786 LGTM ---
[GitHub] zeppelin pull request #2792: [ZEPPELIN-3228] Currently interpreter dependenc...
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2792 ---
[GitHub] zeppelin issue #2792: [ZEPPELIN-3228] Currently interpreter dependencies are...
Github user jhonderson commented on the issue: https://github.com/apache/zeppelin/pull/2792 @zjffdu by the way, here is my travis: https://travis-ci.org/jhonderson/zeppelin/builds/340723065 ---
[GitHub] zeppelin pull request #2792: [ZEPPELIN-3228] Currently interpreter dependenc...
Github user jhonderson closed the pull request at: https://github.com/apache/zeppelin/pull/2792 ---
[GitHub] zeppelin issue #2792: [ZEPPELIN-3228] Currently interpreter dependencies are...
Github user jhonderson commented on the issue: https://github.com/apache/zeppelin/pull/2792 Re-triggering build. ---
[GitHub] zeppelin pull request #2792: [ZEPPELIN-3228] Currently interpreter dependenc...
GitHub user jhonderson reopened a pull request: https://github.com/apache/zeppelin/pull/2792 [ZEPPELIN-3228] Currently interpreter dependencies are not downloaded on zeppelin start - regression issue Currently interpreter dependencies are not downloaded on zeppelin start. This was solved in [ZEPPELIN-3228], but it is happening again. ### What is this PR for? When zeppelin is started/restarted, server should try and download interpreter dependencies. ### What type of PR is it? [Bug Fix] ### What is the Jira issue? * [ZEPPELIN-3228] ### How should this be tested? * Put a dependency (say "org.apache.commons:commons-csv:1.1") in any of the interpreter. * From command line delete local-repo directory * Restart zeppelin server Expectation is local-repo should be recreated with all the dependencies that were mentioned in any of the interpreters. ### 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/jhonderson/zeppelin ZEPPELIN-1143 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/2792.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 #2792 commit 3fcefb28a37c055c235a9ba995815415e18266d6 Author: Cardenas, Jhon Date: 2018-02-12T22:47:10Z [ZEPPELIN-1143] When zeppelin starts it does the interpreter dependencies loading. ---
[GitHub] zeppelin issue #2792: [ZEPPELIN-3228] Currently interpreter dependencies are...
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2792 @jhonderson Can you retrigger the build (via close and reopen), I can not access your jenkins job ---
[GitHub] zeppelin issue #2794: ZEPPELIN-3225: Add a bunch of missing annotations to A...
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2794 @monsieurp welcome any contribution including document ---
[GitHub] zeppelin pull request #2797: ZEPPELIN-3158. Fixed Checkstyle errors and warn...
GitHub user HorizonNet opened a pull request: https://github.com/apache/zeppelin/pull/2797 ZEPPELIN-3158. Fixed Checkstyle errors and warnings in ignite module ### What is this PR for? Fixed all Checkstyle errors and warnings in the ignite module. ### What type of PR is it? Improvement ### Todos * [ ] - Task ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-3158 ### 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/ultratendency/zeppelin ZEPPELIN-3158 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/2797.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 #2797 commit a4bb15b88ae8f9d5e0d824a54f62d9891142afe8 Author: Jan Hentschel Date: 2018-02-13T13:59:53Z ZEPPELIN-3158. Fixed Checkstyle errors and warnings in ignite module ---
[jira] [Created] (ZEPPELIN-3229) Improve MS Outlook support to by decoding percent encoded in URLs
Andrew X created ZEPPELIN-3229: -- Summary: Improve MS Outlook support to by decoding percent encoded in URLs Key: ZEPPELIN-3229 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3229 Project: Zeppelin Issue Type: Improvement Components: Core Affects Versions: 0.7.3, 0.8.0 Environment: MS Outlook (any version), A browser with access to Zeppelin. Reporter: Andrew X Zeppelin notebook links, when clicked on from MS Outlook (any version), gets encoded such that the hash '#' is replaced with '%23' thereby resulting in error 404 in zeppelin. Can supported be added for this? Actual browser error below when link with ../#//notebook/2D7HE4Y1C clicked on from Outlook: h2. HTTP ERROR 404 Problem accessing /%23/notebook/2D7HE4Y1C. Reason: Not Found -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[GitHub] zeppelin pull request #2796: Test
Github user miandreu closed the pull request at: https://github.com/apache/zeppelin/pull/2796 ---
[GitHub] zeppelin issue #2796: Test
Github user miandreu commented on the issue: https://github.com/apache/zeppelin/pull/2796 sorry, I was wrong I was just doing some tests ---
[GitHub] zeppelin pull request #2796: Test
GitHub user miandreu opened a pull request: https://github.com/apache/zeppelin/pull/2796 Test ### What is this PR for? A few sentences describing the overall goals of the pull request's commits. First time? Check out the contributing guide - https://zeppelin.apache.org/contribution/contributions.html ### What type of PR is it? [Bug Fix | Improvement | Feature | Documentation | Hot Fix | Refactoring] ### Todos * [ ] - Task ### 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? * 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/miandreu/zeppelin master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/2796.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 #2796 commit 2622d3f0994448813a4d4bfd032a0795515a01b7 Author: Miquel Angel Andreu <31543113+miandreu@...> Date: 2018-02-11T18:44:56Z Merge pull request #1 from apache/master Bringing fork up to date commit 78c85881475dde974c147e5d2832249be955af6b Author: Miquel Angel Andreu <31543113+miandreu@...> Date: 2018-02-13T17:27:19Z Added CI test for Cassandra and Spark 1.3 Allow Cassandra-spark-1.1 profile to run Cassandra tests Apache license for cassandra.cql file Add extra params to InterpreterContext Removed wrong syntax test ---
[GitHub] zeppelin issue #2771: ZEPPELIN-3209. Preserve thread context classloader whe...
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2771 Thanks @sancyx for the contribution. It looks good, but i'd like to see if it passes CI. Could you try setup CI and trigger build? ``` Looks like travis-ci is not configured for your fork. Please setup by swich on 'zeppelin' repository at https://travis-ci.org/profile and travis-ci. And then make sure 'Build branch updates' option is enabled in the settings https://travis-ci.org/banzaicloud/zeppelin/settings. To trigger CI after setup, you will need ammend your last commit with git commit --amend git push your-remote HEAD --force See http://zeppelin.apache.org/contribution/contributions.html#continuous-integration. Build step 'Execute shell' marked build as failure Putting comment on the pull request Finished: FAILURE ``` Let me know if you need any help on CI. ---
[GitHub] zeppelin pull request #2795: ZEPPELIN-3154. Fixed Checkstyle errors and warn...
GitHub user HorizonNet opened a pull request: https://github.com/apache/zeppelin/pull/2795 ZEPPELIN-3154. Fixed Checkstyle errors and warnings in flink module ### What is this PR for? Fixed the Checkstyle errors and warnings in the flink module. ### What type of PR is it? Improvement ### Todos * [ ] - Task ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-3154 ### 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/ultratendency/zeppelin ZEPPELIN-3154 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/2795.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 #2795 ---
[GitHub] zeppelin issue #2794: ZEPPELIN-3225: Add a bunch of missing annotations to A...
Github user monsieurp commented on the issue: https://github.com/apache/zeppelin/pull/2794 Hey @mebelousov! We also use Active Directory at work. We don't go through the AD connector though but instead rely on the LDAP connector. By the way, there are many improvements that are to be made to this class. Would the Zeppelin contributors accept a PR to document the class for instance? Thanks! ---
[GitHub] zeppelin issue #2752: [ZEPPELIN-3195] Remove the limit on the number of run ...
Github user mebelousov commented on the issue: https://github.com/apache/zeppelin/pull/2752 @zjffdu @felixcheung What will we do with this bug? ---
[GitHub] zeppelin issue #2794: ZEPPELIN-3225: Add a bunch of missing annotations to A...
Github user mebelousov commented on the issue: https://github.com/apache/zeppelin/pull/2794 Our team uses AD. I have tested. Seems that it works good. ---
[GitHub] zeppelin issue #2792: [ZEPPELIN-3228] Currently interpreter dependencies are...
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2792 Thanks @jhonderson , will merge it to branch-0.8 if CI pass ---
[GitHub] zeppelin issue #2792: [ZEPPELIN-3228] Currently interpreter dependencies are...
Github user jhonderson commented on the issue: https://github.com/apache/zeppelin/pull/2792 @zjffdu i have created the ticket for this issue. If this is approved, could you please merge this solution to branch-0.8? ---
[jira] [Created] (ZEPPELIN-3228) Interpreter dependencies are not downloaded on zeppelin start - regression issue
Jhon Cardenas created ZEPPELIN-3228: --- Summary: Interpreter dependencies are not downloaded on zeppelin start - regression issue Key: ZEPPELIN-3228 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3228 Project: Zeppelin Issue Type: Bug Components: zeppelin-interpreter Affects Versions: 0.8.0 Reporter: Jhon Cardenas Fix For: 0.8.0 Same issue that ZEPPELIN-1143. It seems to have been caused by code refactorings. When zeppelin is started/restarted, server should try and download dependencies. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[GitHub] zeppelin pull request #2794: Add a bunch of missing annotations to ActiveDir...
GitHub user monsieurp opened a pull request: https://github.com/apache/zeppelin/pull/2794 Add a bunch of missing annotations to ActiveDirectoryGroupRealm.java. ### What is this PR for? Add a bunch of missing annotations to ActiveDirectoryGroupRealm.java. Those annotations are missing and generate a lot of warnings in my IDE (NetBeans). ### What type of PR is it? Improvement. ### Todos * [ ] - Task ### What is the Jira issue? https://issues.apache.org/jira/browse/ZEPPELIN-3225 ### How should this be tested? There's not much to test. ### 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/monsieurp/zeppelin zep-3225 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/2794.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 #2794 commit 7c24a141522cf555aecabf78adb999e61d107a88 Author: Patrice Clement Date: 2018-02-13T11:18:06Z Add a bunch of missing annotations to ActiveDirectoryGroupRealm.java. ---
