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

2018-07-04 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2742 I'll close this PR because original repo was deleted ---

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

2018-04-26 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2742 I agree. It should do ---

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

2018-04-26 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2742 This PR seems didn't touch ConfInterpreter, but it should do I believe. ---

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

2018-04-26 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2742 I have a very basic question. Does this feature prevent from changing configurations by `ConfInterpreter` dynamically? ---

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

2018-04-26 Thread Tagar
Github user Tagar commented on the issue: https://github.com/apache/zeppelin/pull/2742 @necosta thanks for this great feature. It makes a lot of sense for Zeppelin instance shared with different set of users. Would it be possible to do this interpreter settings authorization for LDAP

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

2018-04-06 Thread necosta
Github user necosta commented on the issue: https://github.com/apache/zeppelin/pull/2742 FYI, got this one rebased on top of latest master. Happy to wait for 0.8.0 release before this MR gets reviewed ---

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

2018-02-14 Thread necosta
Github user necosta commented on the issue: https://github.com/apache/zeppelin/pull/2742 No problem @zjffdu ! Didn't know we were releasing 0.8.0. Happy we hear that :) ---

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

2018-02-14 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2742 sorry @necosta I will review it after 0.8.0 release ---

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

2018-02-14 Thread necosta
Github user necosta commented on the issue: https://github.com/apache/zeppelin/pull/2742 ping @zjffdu , @Leemoonsoo , @mebelousov , @felixcheung Many thanks ---

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

2018-02-07 Thread necosta
Github user necosta commented on the issue: https://github.com/apache/zeppelin/pull/2742 ping @zjffdu , @Leemoonsoo , @mebelousov , Feedback appreciated :) Thanks! ---

[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 us

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

2018-01-23 Thread necosta
Github user necosta commented on the issue: https://github.com/apache/zeppelin/pull/2742 Hi @zjffdu , I didn't create a design doc but I hope the interpreter/overview.md file helps. We can always improve the .md file of course. In a nutshell, I tried to use as much functi