Re: ConfigStorage - Credentials persistence

2018-10-04 Thread Jhon Anderson Cardenas Diaz
Great, that would also avoid to duplicate code between the implementations of NotebookStorage and ConfigStorage. El jue., 4 oct. 2018 a las 19:00, Jeff Zhang () escribió: > Right, this is the future plan. I plan to unify both config stuff and > notebookrepo. > So that all the configuration files

[GitHub] zeppelin issue #3201: [ZEPPELIN-3805] Don't distribution shade jar to lib/in...

2018-10-04 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/3201 It is not necessary to publish zeppelin-interpreter-api.jar to lib/interpreter. zeppelin interpreter api jar is already included to all interpreter jars under lib folder. ---

Re: ConfigStorage - Credentials persistence

2018-10-04 Thread Jeff Zhang
Right, this is the future plan. I plan to unify both config stuff and notebookrepo. So that all the configuration files and notebook files will be stored in the same storage layer. Jhon Anderson Cardenas Diaz 于2018年10月5日周五 上午6:21写道: > Also maybe the *helium configuration* should be included in t

Re: ConfigStorage - Credentials persistence

2018-10-04 Thread Jhon Anderson Cardenas Diaz
Also maybe the *helium configuration* should be included in the ConfigStorage component, in order to be persisted like others configurations. El jue., 4 oct. 2018 a las 16:44, Jhon Anderson Cardenas Diaz (< jhonderson2...@gmail.com>) escribió: > Hi! > > Currently in the ConfigStorage component th

ConfigStorage - Credentials persistence

2018-10-04 Thread Jhon Anderson Cardenas Diaz
Hi! Currently in the ConfigStorage component there are methods to persist and retrieve the credentials of the zeppelin users: public abstract String loadCredentials() throws IOException; public abstract void saveCredentials(String credentials) throws IOException; But those methods are not being

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

2018-10-04 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2709 Thanks @alonshoham Feel free to create a PR for that issue ---

[GitHub] zeppelin issue #3201: [ZEPPELIN-3805] Don't distribution shade jar to lib/in...

2018-10-04 Thread liuxunorg
Github user liuxunorg commented on the issue: https://github.com/apache/zeppelin/pull/3201 @zjffdu , Because the interpreter process requires the zeppelin-interpreter-api.jar file, So in the `zeppelin-distribution/src/assemble/distribution.xml` only include ```

[jira] [Created] (ZEPPELIN-3808) Link to Interpeter Installation doc is incorrect

2018-10-04 Thread Eric Pugh (JIRA)
Eric Pugh created ZEPPELIN-3808: --- Summary: Link to Interpeter Installation doc is incorrect Key: ZEPPELIN-3808 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3808 Project: Zeppelin Issue T

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

2018-10-04 Thread alonshoham
Github user alonshoham commented on the issue: https://github.com/apache/zeppelin/pull/2709 Your commit looks OK. Notice that I mentioned another problem in the ticket with interpreter.cmd On Wed, Oct 3, 2018 at 9:55 AM Jeff Zhang wrote: > @alonshoham

[GitHub] zeppelin issue #3137: [ZEPPELIN-3709] Add releaseResource parameter to Cron ...

2018-10-04 Thread Savalek
Github user Savalek commented on the issue: https://github.com/apache/zeppelin/pull/3137 What about merge? ---