[GitHub] zeppelin issue #2709: ZEPPELIN-3111. Refactor SparkInterpreter

2018-01-24 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2709 @Leemoonsoo @felixcheung @prabhjyotsingh @jongyoul Could you help review it ? Thanks ---

[GitHub] zeppelin issue #2746: ZEPEPELIN-3190. Should not use singleton for FileSyste...

2018-01-24 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2746 @prabhjyotsingh Please help review it, Thanks ---

[GitHub] zeppelin pull request #2741: [ZEPPELIN-3187] Remove doubles of settings in z...

2018-01-24 Thread mebelousov
Github user mebelousov closed the pull request at: https://github.com/apache/zeppelin/pull/2741 ---

[GitHub] zeppelin pull request #2741: [ZEPPELIN-3187] Remove doubles of settings in z...

2018-01-24 Thread mebelousov
GitHub user mebelousov reopened a pull request: https://github.com/apache/zeppelin/pull/2741 [ZEPPELIN-3187] Remove doubles of settings in zeppelin-site.xml.template ### What is this PR for? Remove doubles in zeppelin-site.xml.template ### What type of PR is it? [Bug

[GitHub] zeppelin pull request #2738: [ZEPPELIN-3182] Support saving notebooks to Goo...

2018-01-24 Thread karth295
Github user karth295 closed the pull request at: https://github.com/apache/zeppelin/pull/2738 ---

[GitHub] zeppelin pull request #2738: [ZEPPELIN-3182] Support saving notebooks to Goo...

2018-01-24 Thread karth295
GitHub user karth295 reopened a pull request: https://github.com/apache/zeppelin/pull/2738 [ZEPPELIN-3182] Support saving notebooks to Google Cloud Storage ### What is this PR for? Support saving notebooks to Google Cloud Storage, similar to implementations for S3 and Azure

[GitHub] zeppelin issue #2738: [ZEPPELIN-3182] Support saving notebooks to Google Clo...

2018-01-24 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2738 CI still fails, could you retrigger it again ? Just close and reopen this PR. ---

[GitHub] zeppelin issue #2740: [ZEPPELIN-3185] Add profiles for adding dependencies f...

2018-01-24 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/2740 I believe `zeppelin-jdbc` depends only on `zeppelin-interpreter`, and not on `zeppelin-zengine`, so it should be safe. ---

[GitHub] zeppelin issue #2740: [ZEPPELIN-3185] Add profiles for adding dependencies f...

2018-01-24 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/2740 I guess it could make sense if people want to use jdbc alone (and that is Hadoop 2 set of jdbc) Would it confict if zengine or something is hadoop2 and then the interpreter is hadoop3

[GitHub] zeppelin pull request #2746: ZEPEPELIN-3190. Should not use singleton for Fi...

2018-01-24 Thread zjffdu
GitHub user zjffdu opened a pull request: https://github.com/apache/zeppelin/pull/2746 ZEPEPELIN-3190. Should not use singleton for FileSystemStorage ### What is this PR for? For now, `FileSystemNotebookRepo`, `FileSystemConfigStorage`, `FileSystemRecoveryStorage` use `FileSyste

[GitHub] zeppelin pull request #:

2018-01-24 Thread zjffdu
Github user zjffdu commented on the pull request: https://github.com/apache/zeppelin/commit/fbb35f92ada1361148c63ad42bf6a5a231328b1a#commitcomment-27088037 Thanks @Evaneaston , just created [ZEPPELIN-3190](https://issues.apache.org/jira/browse/ZEPPELIN-3190) and will fix it soon. -

[jira] [Created] (ZEPPELIN-3190) Should not use singleton for FileSystemStorage

2018-01-24 Thread Jeff Zhang (JIRA)
Jeff Zhang created ZEPPELIN-3190: Summary: Should not use singleton for FileSystemStorage Key: ZEPPELIN-3190 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3190 Project: Zeppelin Issue T

[GitHub] zeppelin pull request #:

2018-01-24 Thread zjffdu
Github user zjffdu commented on the pull request: https://github.com/apache/zeppelin/commit/fbb35f92ada1361148c63ad42bf6a5a231328b1a#commitcomment-27086076 Thanks @Evaneaston for trying this PR, let me verify that. ---

[GitHub] zeppelin pull request #:

2018-01-24 Thread Evaneaston
Github user Evaneaston commented on the pull request: https://github.com/apache/zeppelin/commit/fbb35f92ada1361148c63ad42bf6a5a231328b1a#commitcomment-27082601 So after pulling this in, I'm having trouble getting zeppelin to start. I've got an AWS EMR cluster that I've installed a bu

[GitHub] zeppelin issue #2742: [WIP] [ZEPPELIN-3168] Interpreter Settings Authorizati...

2018-01-24 Thread necosta
Github user necosta commented on the issue: https://github.com/apache/zeppelin/pull/2742 Q: If user create new interpreter property, is it readonly by default or not ? A: Editable by default ---

[GitHub] zeppelin pull request #2745: [ZEPPELIN-3189] NPE on paragraph run via API

2018-01-24 Thread necosta
GitHub user necosta opened a pull request: https://github.com/apache/zeppelin/pull/2745 [ZEPPELIN-3189] NPE on paragraph run via API ### What is this PR for? Bugfix over NPE when running REST API command to run paragraph synchronously. ### What type of PR is it?

[jira] [Created] (ZEPPELIN-3189) Cannot run paragraph via API

2018-01-24 Thread Nelson Costa (JIRA)
Nelson Costa created ZEPPELIN-3189: -- Summary: Cannot run paragraph via API Key: ZEPPELIN-3189 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3189 Project: Zeppelin Issue Type: Bug

[GitHub] zeppelin issue #2740: [ZEPPELIN-3185] Add profiles for adding dependencies f...

2018-01-24 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/2740 Thanks for the review will merge this if no more discussion. ---

[GitHub] zeppelin pull request #2744: ZEPPELIN-3184. Use hadoop-azure to replace azur...

2018-01-24 Thread zjffdu
GitHub user zjffdu opened a pull request: https://github.com/apache/zeppelin/pull/2744 ZEPPELIN-3184. Use hadoop-azure to replace azure-storage ### What is this PR for? So that user can use azure storage via hadoop filesystem. ### What type of PR is it? [ Impr

[GitHub] zeppelin issue #2740: [ZEPPELIN-3185] Add profiles for adding dependencies f...

2018-01-24 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2740 LGTM ---