[GitHub] zeppelin issue #2719: Spark impersonation

2018-02-06 Thread hayssams
Github user hayssams commented on the issue: https://github.com/apache/zeppelin/pull/2719 @zjffdu my bas. Ticket renewal is not required. Thanks for your clarification. ---

[GitHub] zeppelin issue #2764: ZEPPELIN-3146. Fixed Checkstyle errors in alluxio modu...

2018-02-06 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2764 LGTM ---

[GitHub] zeppelin issue #2767: ZEPPELIN-3145. Fixed Checkstyle errors and warnings in...

2018-02-06 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2767 Thanks @HorizonNet , LGTM ---

[GitHub] zeppelin pull request #2769: ZEPPELIN-3171. Restart of interpreter in note a...

2018-02-06 Thread zjffdu
GitHub user zjffdu opened a pull request: https://github.com/apache/zeppelin/pull/2769 ZEPPELIN-3171. Restart of interpreter in note also aborts running interpreter in another note ### What is this PR for? The root cause is that in isolated mode interpreters will

[GitHub] zeppelin issue #2750: ZEPPELIN-3108. Support Spark 2.3

2018-02-06 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2750 These download are for spark interpreter integration test. So spark distribution is needed, only spark artifact is not sufficient. ---

[GitHub] zeppelin issue #2750: ZEPPELIN-3108. Support Spark 2.3

2018-02-06 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/2750 I don't feel strongly about it, but fetching the url from the download script (.cgi url) is actually fairly simple to do, but I admit may not be what the maven download plugin support.

[GitHub] zeppelin pull request #2759: ZEPPELIN-3202. Added missing test dependencies ...

2018-02-06 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2759 ---

[GitHub] zeppelin issue #2587: [ZEPPELIN-2909]. Support shared SparkContext across la...

2018-02-06 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/2587 This looks good, with one minor comment, this was one long pending feature missing in Livy a big +1 for adding this functionality. ---

[jira] [Created] (ZEPPELIN-3206) Move spark integration test to zeppelin-integration

2018-02-06 Thread Jeff Zhang (JIRA)
Jeff Zhang created ZEPPELIN-3206: Summary: Move spark integration test to zeppelin-integration Key: ZEPPELIN-3206 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3206 Project: Zeppelin

[GitHub] zeppelin issue #2587: [ZEPPELIN-2909]. Support shared SparkContext across la...

2018-02-06 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2587 @prabhjyotsingh Could you help review it ? Thanks ---

[jira] [Created] (ZEPPELIN-3205) restarting interpreter setting in a notebook abort running jobs of other notebooks

2018-02-06 Thread dungnguyen (JIRA)
dungnguyen created ZEPPELIN-3205: Summary: restarting interpreter setting in a notebook abort running jobs of other notebooks Key: ZEPPELIN-3205 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3205

[GitHub] zeppelin pull request #2764: ZEPPELIN-3146. Fixed Checkstyle errors in allux...

2018-02-06 Thread HorizonNet
GitHub user HorizonNet reopened a pull request: https://github.com/apache/zeppelin/pull/2764 ZEPPELIN-3146. Fixed Checkstyle errors in alluxio module ### What is this PR for? Fix the Checkstyle errors and warning in the **alluxio** module. ### What type of PR is it?

[GitHub] zeppelin pull request #2759: ZEPPELIN-3202. Added missing test dependencies ...

2018-02-06 Thread HorizonNet
Github user HorizonNet closed the pull request at: https://github.com/apache/zeppelin/pull/2759 ---

[GitHub] zeppelin pull request #2765: ZEPPELIN-3164. Fixed Checkstyle errors and warn...

2018-02-06 Thread HorizonNet
Github user HorizonNet closed the pull request at: https://github.com/apache/zeppelin/pull/2765 ---

[GitHub] zeppelin pull request #2765: ZEPPELIN-3164. Fixed Checkstyle errors and warn...

2018-02-06 Thread HorizonNet
GitHub user HorizonNet reopened a pull request: https://github.com/apache/zeppelin/pull/2765 ZEPPELIN-3164. Fixed Checkstyle errors and warnings in the scalding module ### What is this PR for? Fixed the errors and warnings from Checkstyle in the **scalding** module. ###

[GitHub] zeppelin pull request #2764: ZEPPELIN-3146. Fixed Checkstyle errors in allux...

2018-02-06 Thread HorizonNet
Github user HorizonNet closed the pull request at: https://github.com/apache/zeppelin/pull/2764 ---

[GitHub] zeppelin pull request #2759: ZEPPELIN-3202. Added missing test dependencies ...

2018-02-06 Thread HorizonNet
GitHub user HorizonNet reopened a pull request: https://github.com/apache/zeppelin/pull/2759 ZEPPELIN-3202. Added missing test dependencies in the scio module ### What is this PR for? Added missing test dependencies for the **scio** module to prevent test failures when

[GitHub] zeppelin pull request #2766: ZEPPELIN-3147. Added Checkstyle to the angular ...

2018-02-06 Thread HorizonNet
Github user HorizonNet closed the pull request at: https://github.com/apache/zeppelin/pull/2766 ---

[GitHub] zeppelin pull request #2767: ZEPPELIN-3145. Fixed Checkstyle errors and warn...

2018-02-06 Thread HorizonNet
Github user HorizonNet closed the pull request at: https://github.com/apache/zeppelin/pull/2767 ---

[GitHub] zeppelin pull request #2766: ZEPPELIN-3147. Added Checkstyle to the angular ...

2018-02-06 Thread HorizonNet
GitHub user HorizonNet reopened a pull request: https://github.com/apache/zeppelin/pull/2766 ZEPPELIN-3147. Added Checkstyle to the angular module ### What is this PR for? Added Checkstyle to the **angular** module. Did not have to fix any errors or warnings. ### What

[GitHub] zeppelin pull request #2767: ZEPPELIN-3145. Fixed Checkstyle errors and warn...

2018-02-06 Thread HorizonNet
GitHub user HorizonNet reopened a pull request: https://github.com/apache/zeppelin/pull/2767 ZEPPELIN-3145. Fixed Checkstyle errors and warnings in the shell module ### What is this PR for? Fixed all Checkstyle errors and warnings in the **shell** module. ### What type

[GitHub] zeppelin issue #2768: ZEPPELIN-3142. Fixed Checkstyle errors and warnings in...

2018-02-06 Thread HorizonNet
Github user HorizonNet commented on the issue: https://github.com/apache/zeppelin/pull/2768 @masyan Thanks for the tip. ---

[GitHub] zeppelin pull request #2768: ZEPPELIN-3142. Fixed Checkstyle errors and warn...

2018-02-06 Thread HorizonNet
GitHub user HorizonNet reopened a pull request: https://github.com/apache/zeppelin/pull/2768 ZEPPELIN-3142. Fixed Checkstyle errors and warnings in the livy module ### What is this PR for? Fixed the Checkstyle errors and warnings in the **livy** module. ### What type of

[GitHub] zeppelin pull request #2768: ZEPPELIN-3142. Fixed Checkstyle errors and warn...

2018-02-06 Thread HorizonNet
Github user HorizonNet closed the pull request at: https://github.com/apache/zeppelin/pull/2768 ---

[GitHub] zeppelin issue #2768: ZEPPELIN-3142. Fixed Checkstyle errors and warnings in...

2018-02-06 Thread masyan
Github user masyan commented on the issue: https://github.com/apache/zeppelin/pull/2768 you can reopen PR (for update jenkins status) ---

[GitHub] zeppelin issue #2768: ZEPPELIN-3142. Fixed Checkstyle errors and warnings in...

2018-02-06 Thread HorizonNet
Github user HorizonNet commented on the issue: https://github.com/apache/zeppelin/pull/2768 Did a rerun of the failing builds. They are now green. ---

[GitHub] zeppelin pull request #2768: ZEPPELIN-3142. Fixed Checkstyle errors and warn...

2018-02-06 Thread HorizonNet
GitHub user HorizonNet opened a pull request: https://github.com/apache/zeppelin/pull/2768 ZEPPELIN-3142. Fixed Checkstyle errors and warnings in the livy module ### What is this PR for? Fixed the Checkstyle errors and warnings in the **livy** module. ### What type of

[GitHub] zeppelin pull request #:

2018-02-06 Thread Savalek
Github user Savalek commented on the pull request: https://github.com/apache/zeppelin/commit/4bf0f3a910e01107c00f28e4d138e52c5013f132#commitcomment-27371992 In zeppelin-web/src/app/notebook/notebook-actionBar.html: In zeppelin-web/src/app/notebook/notebook-actionBar.html on line

[GitHub] zeppelin pull request #:

2018-02-06 Thread jhonderson
Github user jhonderson commented on the pull request: https://github.com/apache/zeppelin/commit/4bf0f3a910e01107c00f28e4d138e52c5013f132#commitcomment-27371810 In zeppelin-web/src/app/notebook/notebook-actionBar.html: In zeppelin-web/src/app/notebook/notebook-actionBar.html on

[GitHub] zeppelin issue #2719: Spark impersonation

2018-02-06 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2719 Thanks @hayssams for the contribution. Several comments 1. Have you verified the current mechanism of kerberos support ? 2. This PR seems only work with yarn client mode, but not yarn

[GitHub] zeppelin issue #2757: [ZEPPELIN-3198] UI should not show Version/GIT Control...

2018-02-06 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/2757 @zjffdu have made the suggested changes, waiting for CI, does this looks good? ---

[GitHub] zeppelin issue #2742: [ZEPPELIN-3168] Interpreter Settings Authorization

2018-02-06 Thread necosta
Github user necosta commented on the issue: https://github.com/apache/zeppelin/pull/2742 A few more changes: - Exposed interpreter property `Description` - Exposed interpreter property `Readonly` - Added interpreter option `disallowCustomInterpreter` (Prevents users from

[GitHub] zeppelin issue #2767: ZEPPELIN-3145. Fixed Checkstyle errors and warnings in...

2018-02-06 Thread HorizonNet
Github user HorizonNet commented on the issue: https://github.com/apache/zeppelin/pull/2767 Did a reran of the Travis build. Alls builds are green now. ---