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

2018-02-05 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2750 Thanks @felixcheung , These examples seems to fetch the download url dynamically, I don't think maven download plugin support this. If you don't mind, I will leave it as follow up PR to improve.

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

2018-02-05 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/2750 well... that's not my understand and other implementations are pulling from the download script and then to the referred mirror, eg.

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

2018-02-05 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2750 @felixcheung IIUC, these are for release download page, for external project, we could use the dist link. ---

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

2018-02-05 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/2750 Ah sorry. Please see http://www.apache.org/dev/release-publishing.html#distribution And specifically http://www.apache.org/dev/release-download-pages.html#links

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

2018-02-05 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2750 Fixed. Now using apache dist url ---

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

2018-02-05 Thread HorizonNet
GitHub user HorizonNet opened 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 of

[GitHub] zeppelin issue #2737: ZEPPELIN-3181. Enable impersonation support for kerber...

2018-02-05 Thread hayssams
Github user hayssams commented on the issue: https://github.com/apache/zeppelin/pull/2737 I suspect a problem with this PR if the Kerberos ticket duration is shorter than the interpreter lifetime. That is why I previously opened a PR that explicitly do a ticket renewal at an

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

2018-02-05 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/2750 Hmm we cant fix to one or two mirrors either, i think they tend to disappear - please see how other projects handles this ---

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

2018-02-05 Thread HorizonNet
Github user HorizonNet commented on the issue: https://github.com/apache/zeppelin/pull/2765 I just reran the last failing build. All builds are now green. ---

[GitHub] zeppelin issue #2762: [ZEPPELIN-3204] FIX: cursor in paragraph editor jumps

2018-02-05 Thread Savalek
Github user Savalek commented on the issue: https://github.com/apache/zeppelin/pull/2762 @timovwb sorry about this. I change the task number. ---

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

2018-02-05 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2757 It sounds a little redundant to ask user to both implement these version related method and `isRevisionSupported`. How about creating a sub-interface of `NotebookRepo` which support version