[GitHub] zeppelin issue #3238: ZEPPELIN-3870. Configuration in spark-defaults.conf do...

2018-12-04 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/3238 @zjffdu i tried to build the latest branch-0.8 where this PR has been merged recently. build fails with `RemoteInterpreterServer.java:[22,32] package org.apache.commons.lang3

[GitHub] zeppelin pull request #3202: [ZEPPELIN-3528] [branch-0.8] Ordering by interp...

2018-10-07 Thread weand
Github user weand closed the pull request at: https://github.com/apache/zeppelin/pull/3202 ---

[GitHub] zeppelin pull request #3202: [ZEPPELIN-3528] [branch-0.8] Ordering by interp...

2018-10-05 Thread weand
GitHub user weand opened a pull request: https://github.com/apache/zeppelin/pull/3202 [ZEPPELIN-3528] [branch-0.8] Ordering by interpreter name in same interpreter group ### What is this PR for? Merge for branch-0.8. In 'Create new note' diagram, 'Default Interpreter

[GitHub] zeppelin issue #2951: ZEPPELIN-3435. Interpreter timeout lifecycle leads to ...

2018-04-30 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2951 :+1: no more orphans were spotted since applying changes of this PR. ---

[GitHub] zeppelin issue #2951: ZEPPELIN-3435. Interpreter timeout lifecycle leads to ...

2018-04-28 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2951 ... but we are now trying this in production. ---

[GitHub] zeppelin issue #2951: ZEPPELIN-3435. Interpreter timeout lifecycle leads to ...

2018-04-28 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2951 I can't confirm that. LifecycleManager tried to close the interpreter group at 13:06 (see log pasted in the Jira) - 60 minutes after the last job was finished. The same interpreter group

[GitHub] zeppelin issue #2925: ZEPPELIN-3404. Fail to run cronjob when user doesn't r...

2018-04-26 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2925 great. LGTM :+1: please merge to master and branch-0.8 ---

[GitHub] zeppelin issue #2942: [HOTFIX] ZEPPELIN-3405 fix permission checks with role...

2018-04-25 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2942 works ---

[GitHub] zeppelin issue #2937: ZEPPELIN-3401. Deadlock while restarting interpreter

2018-04-25 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2937 so far no more issues. LGTM :+1: ---

[GitHub] zeppelin issue #2935: ZEPPELIN-3411 Long running logic inside synchronized b...

2018-04-23 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2935 are you going to merge that into branch-0.8 as well? ---

[GitHub] zeppelin issue #2937: ZEPPELIN-3401. Deadlock while restarting interpreter

2018-04-23 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2937 Today we had that bug again (seems to occur daily) without the patch added to PROD. Now just added the patch to PROD. Will wait another 48h from now before I tell you that it's working. ---

[GitHub] zeppelin issue #2925: ZEPPELIN-3404. Fail to run cronjob when user doesn't r...

2018-04-23 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2925 @zjffdu uhm, I can't confirm it. Today after checking out this PR from scratch my first point now works correctly. sorry for the spam. created issue [ZEPPELIN-3427](https

[GitHub] zeppelin issue #2925: ZEPPELIN-3404. Fail to run cronjob when user doesn't r...

2018-04-21 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2925 The notebook now runs successfully after restart with the user who enabled the scheduler. 2 remaining issues: 1) user1 schedules the cron. then the note is executed via cron

[GitHub] zeppelin issue #2925: ZEPPELIN-3404. Fail to run cronjob when user doesn't r...

2018-04-21 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2925 The notebook now runs successfully after restart with the user who enabled the scheduler. There is one more valid use case left in my mind (dealing with permissions): - interpreter

[GitHub] zeppelin issue #2925: ZEPPELIN-3404. Fail to run cronjob when user doesn't r...

2018-04-19 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2925 @mebelousov I fully agree that running document as the group is not what a user would expect. That's why I'm proposing another approach again: 1) prior to #2914 the cronExecutionUser

[GitHub] zeppelin issue #2925: ZEPPELIN-3404. Fail to run cronjob when user doesn't r...

2018-04-17 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2925 and optionally (or alternatively) introduce a config parameter for a static cronExecutionUser (kind of service user) to be defined by ops team. when this config parameter is set, cron schedules

[GitHub] zeppelin issue #2925: ZEPPELIN-3404. Fail to run cronjob when user doesn't r...

2018-04-17 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2925 @zjffdu is there a chance to set the cronExecutionUser implictly to the user who activates the cron schedule. and in addtion only allow owners (which can be defined by username or groupnames

[GitHub] zeppelin issue #2925: ZEPPELIN-3404. Fail to run cronjob when user doesn't r...

2018-04-16 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2925 so running via cron will use the last user that executed a paragraph. that does not fit exactly to the docu change from original PR: > Cron executing user (It is removed from 0.8 wh

[GitHub] zeppelin issue #2914: [ZEPPELIN-3350] Don't allow set cronExecutionUser

2018-04-12 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2914 not directly when logged out. instead the NPE happens when Zeppelin gets restartet and next cron runs before the note was run manually, which is a frequent use case IMHO. > ERROR [2018-04-12

[GitHub] zeppelin issue #2914: [ZEPPELIN-3350] Don't allow set cronExecutionUser

2018-04-11 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2914 @zjffdu are you sure this works when logged out? i followed the call stack and I'm wondering if there would happen NPE if no AuthenticationInfo is present

[GitHub] zeppelin issue #2749: [ZEPPELIN-3180] BUGFIX - save operation on interpreter...

2018-03-08 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2749 *no more comment*, we really would like to see this feature in master and branch-0.8 branches ---

[GitHub] zeppelin pull request #2842: [ZEPPELIN-3300] branch-0.8 broken due to 0.9.0 ...

2018-03-06 Thread weand
Github user weand closed the pull request at: https://github.com/apache/zeppelin/pull/2842 ---

[GitHub] zeppelin issue #2835: [ZEPPELIN-3289] Table not using full height after para...

2018-03-06 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2835 Opened another table issue with table option 'useFilter' - [ZEPPELIN-3301](https://issues.apache.org/jira/browse/ZEPPELIN-3301). Probably related to this issue here. ---

[GitHub] zeppelin pull request #2842: [ZEPPELIN-3300] branch-0.8 broken due to 0.9.0 ...

2018-03-06 Thread weand
GitHub user weand opened a pull request: https://github.com/apache/zeppelin/pull/2842 [ZEPPELIN-3300] branch-0.8 broken due to 0.9.0 parent dependency in sap interpreter ### What is this PR for? Hotfix PR for branch-0.8 build ### What type of PR is it? [Hot

[GitHub] zeppelin issue #2835: [ZEPPELIN-3289] Table not using full height after para...

2018-03-06 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2835 LGTM. no errors in console and ZEPPELIN-3292 data duplication fixed as well. ---

[GitHub] zeppelin issue #2808: [ZEPPELIN-3243] fix IndexOutOfBoundsException when 'No...

2018-03-05 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2808 We did not face any other IndexOutOfBoundException so far. So I would propose to merge that first. ---

[GitHub] zeppelin issue #2835: [ZEPPELIN-3289] Table not using full height after para...

2018-03-05 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2835 height issue is solved. thanks. but now got the bahvior, that results are duplicated: [https://ibin.co/3trkDDqHas5m.gif](https://ibin.co/3trkDDqHas5m.gif) and the following

[GitHub] zeppelin issue #2809: [ZEPPELIN-3249] Add support for streaming table

2018-03-04 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2809 seems like it has changed table behavior when height of paragraph has been changed. https://issues.apache.org/jira/browse/ZEPPELIN-3289 @prabhjyotsingh can you fix that bug? ---

[GitHub] zeppelin issue #2824: ZEPPELIN-3278. Avoid duplicated interpreter setting

2018-03-03 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2824 I can confirm that it works - tested on branch-0.8 ---

[GitHub] zeppelin issue #2819: ZEPPELIN-3265. DevInterpreter doesn't work

2018-03-01 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2819 @zjffdu this PR creates duplicate interpreters in interpreter.json. after upgrading to this commit, all installed interpreters are duplicated, e.g. %angular, %md, %spark, %python

[GitHub] zeppelin pull request #2808: [ZEPPELIN-3243] fix IndexOutOfBoundsException w...

2018-02-22 Thread weand
Github user weand closed the pull request at: https://github.com/apache/zeppelin/pull/2808 ---

[GitHub] zeppelin pull request #2808: [ZEPPELIN-3243] fix IndexOutOfBoundsException w...

2018-02-22 Thread weand
GitHub user weand reopened a pull request: https://github.com/apache/zeppelin/pull/2808 [ZEPPELIN-3243] fix IndexOutOfBoundsException when 'No interpreter is binded to this note' ### What is this PR for? I observed IndexOutOfBoundsException when using zeppelin locally

[GitHub] zeppelin issue #2439: [ZEPPELIN-2680] allow opening notebook as a reader

2018-02-22 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2439 @herval: I merged latest master: [https://github.com/weand/zeppelin/tree/ZEPPELIN-2680](https://github.com/weand/zeppelin/tree/ZEPPELIN-2680) But I also face continuous requests against

[GitHub] zeppelin issue #2749: [ZEPPELIN-3180] BUGFIX - save operation on interpreter...

2018-02-22 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2749 LGTM @felixcheung what do you think? ---

[GitHub] zeppelin issue #2439: [ZEPPELIN-2680] allow opening notebook as a reader

2018-02-19 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2439 @herval would you mind merging again on latest master? @soralee @zjffdu as we face the same issue, and we are using helium, I will also test it. ---

[GitHub] zeppelin pull request #2808: [ZEPPELIN-3243] fix IndexOutOfBoundsException w...

2018-02-18 Thread weand
GitHub user weand opened a pull request: https://github.com/apache/zeppelin/pull/2808 [ZEPPELIN-3243] fix IndexOutOfBoundsException when 'No interpreter is binded to this note' ### What is this PR for? I observed IndexOutOfBoundsException when using zeppelin locally

[GitHub] zeppelin issue #2749: [ZEPPELIN-3180] BUGFIX - save operation on interpreter...

2018-02-16 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2749 any progress here? ---

[GitHub] zeppelin pull request #2806: [ZEPPELIN-3241] fix NPE when restarting Interpr...

2018-02-16 Thread weand
GitHub user weand opened a pull request: https://github.com/apache/zeppelin/pull/2806 [ZEPPELIN-3241] fix NPE when restarting Interpreter ### What is this PR for? We often (but not always) face an NPE when restarting Spark interpreter. Our Spark interpreter run in scoped mode

[GitHub] zeppelin pull request #2706: ZEPPELIN-3105 Notebook not running via REST API...

2018-01-11 Thread weand
Github user weand closed the pull request at: https://github.com/apache/zeppelin/pull/2706 ---

[GitHub] zeppelin pull request #2706: ZEPPELIN-3105 Notebook not running via REST API...

2018-01-11 Thread weand
GitHub user weand reopened a pull request: https://github.com/apache/zeppelin/pull/2706 ZEPPELIN-3105 Notebook not running via REST API after Zeppelin Restart ### What is this PR for? Fixes regression introduced with [ZEPPELIN-3013](https://issues.apache.org/jira/browse

[GitHub] zeppelin issue #2706: ZEPPELIN-3105 Notebook not running via REST API after ...

2018-01-04 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2706 @zjffdu As this fixes a regression from [ZEPPELIN-3013](https://issues.apache.org/jira/browse/ZEPPELIN-3013) would you mind helping here out? ---

[GitHub] zeppelin issue #2706: ZEPPELIN-3105 Notebook not running via REST API after ...

2017-12-22 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2706 Don't get the Selenium tests running properly. Instead of the error from Travis CI, I face other errors, that number of python processes is wrong. ``` 21:05:52,088 INFO

[GitHub] zeppelin pull request #2706: ZEPPELIN-3105 Notebook not running via REST API...

2017-12-22 Thread weand
GitHub user weand reopened a pull request: https://github.com/apache/zeppelin/pull/2706 ZEPPELIN-3105 Notebook not running via REST API after Zeppelin Restart ### What is this PR for? Fixes regression introduced with [ZEPPELIN-3013](https://issues.apache.org/jira/browse

[GitHub] zeppelin pull request #2706: ZEPPELIN-3105 Notebook not running via REST API...

2017-12-22 Thread weand
Github user weand closed the pull request at: https://github.com/apache/zeppelin/pull/2706 ---

[GitHub] zeppelin issue #2706: ZEPPELIN-3105 Notebook not running via REST API after ...

2017-12-21 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2706 This time it failed here: org.apache.zeppelin.integration.InterpreterModeActionsIT.testPerUserIsolatedAction(InterpreterModeActionsIT.java:729) the last time it failed at another line

[GitHub] zeppelin issue #2706: ZEPPELIN-3105 Notebook not running via REST API after ...

2017-12-21 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2706 close and reopen PR does NOT trigger the CI. now merged latest master again, and awaiting test results. ---

[GitHub] zeppelin pull request #2706: ZEPPELIN-3105 Notebook not running via REST API...

2017-12-21 Thread weand
Github user weand closed the pull request at: https://github.com/apache/zeppelin/pull/2706 ---

[GitHub] zeppelin pull request #2706: ZEPPELIN-3105 Notebook not running via REST API...

2017-12-21 Thread weand
GitHub user weand reopened a pull request: https://github.com/apache/zeppelin/pull/2706 ZEPPELIN-3105 Notebook not running via REST API after Zeppelin Restart ### What is this PR for? Fixes regression introduced with [ZEPPELIN-3013](https://issues.apache.org/jira/browse

[GitHub] zeppelin issue #2706: ZEPPELIN-3105 Notebook not running via REST API after ...

2017-12-20 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2706 @zjffdu can u confirm it's a flaky test? ---

[GitHub] zeppelin pull request #2706: ZEPPELIN-3105 Notebook not running via REST API...

2017-12-20 Thread weand
Github user weand closed the pull request at: https://github.com/apache/zeppelin/pull/2706 ---

[GitHub] zeppelin pull request #2706: ZEPPELIN-3105 Notebook not running via REST API...

2017-12-20 Thread weand
GitHub user weand reopened a pull request: https://github.com/apache/zeppelin/pull/2706 ZEPPELIN-3105 Notebook not running via REST API after Zeppelin Restart ### What is this PR for? Fixes regression introduced with [ZEPPELIN-3013](https://issues.apache.org/jira/browse

[GitHub] zeppelin pull request #2706: ZEPPELIN-3105 Notebook not running via REST API...

2017-12-20 Thread weand
GitHub user weand reopened a pull request: https://github.com/apache/zeppelin/pull/2706 ZEPPELIN-3105 Notebook not running via REST API after Zeppelin Restart ### What is this PR for? Fixes regression introduced with [ZEPPELIN-3013](https://issues.apache.org/jira/browse

[GitHub] zeppelin pull request #2706: ZEPPELIN-3105 Notebook not running via REST API...

2017-12-20 Thread weand
Github user weand closed the pull request at: https://github.com/apache/zeppelin/pull/2706 ---

[GitHub] zeppelin issue #2706: ZEPPELIN-3105 Notebook not running via REST API after ...

2017-12-19 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2706 Tests run: 3, Failures: 0, Errors: 3, Skipped: 0, Time elapsed: 164.418 sec <<< FAILURE! - in org.apache.zeppelin.integration.InterpreterModeActionsIT seems to be a flaky test

[GitHub] zeppelin issue #2706: ZEPPELIN-3105 Notebook not running via REST API after ...

2017-12-19 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2706 how can I retrigger CI? ---

[GitHub] zeppelin pull request #2706: ZEPPELIN-3105 Notebook not running via REST API...

2017-12-17 Thread weand
GitHub user weand opened a pull request: https://github.com/apache/zeppelin/pull/2706 ZEPPELIN-3105 Notebook not running via REST API after Zeppelin Restart ### What is this PR for? Fixes regression introduced with [ZEPPELIN-3013](https://issues.apache.org/jira/browse/ZEPPELIN

[GitHub] zeppelin pull request #2675: [ZEPPELIN-3060] Unable to use interpreter names...

2017-11-28 Thread weand
Github user weand closed the pull request at: https://github.com/apache/zeppelin/pull/2675 ---

[GitHub] zeppelin issue #2675: [ZEPPELIN-3060] Unable to use interpreter names with s...

2017-11-28 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2675 Closed and created https://issues.apache.org/jira/browse/ZEPPELIN-3083 instead. ---

[GitHub] zeppelin issue #2675: [ZEPPELIN-3060] Unable to use interpreter names with s...

2017-11-21 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2675 Well, the current implementation does not enforce any naming pattern when creating a new interpreter. Before ZEPPELIN-3013 interpreters with any characters e.g. - could be created

[GitHub] zeppelin pull request #2675: [ZEPPELIN-3060] Unable to use interpreter names...

2017-11-16 Thread weand
GitHub user weand opened a pull request: https://github.com/apache/zeppelin/pull/2675 [ZEPPELIN-3060] Unable to use interpreter names with special characters (e.g. dash or underscore) [ZEPPELIN-3060] Unable to use interpreter names with special characters (e.g. dash

[GitHub] zeppelin issue #2647: [ZEPPELIN-3012] Interpreter Permissions not working pr...

2017-11-03 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2647 Tests added added. Can you merge that into master? ---

[GitHub] zeppelin pull request #2647: [ZEPPELIN-3012] Interpreter Permissions not wor...

2017-11-02 Thread weand
GitHub user weand opened a pull request: https://github.com/apache/zeppelin/pull/2647 [ZEPPELIN-3012] Interpreter Permissions not working properly for groups ### What is this PR for? This fixes issues when using Zeppelin with permissions for users belonging to more than one

[GitHub] zeppelin issue #2062: ZEPPELIN-2161 Nested Group Support in LdapRealm for AD

2017-03-02 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2062 Rebased and green now. Thanks for your help. Would you mind merging that into branch-0.7 as well? Regarding docu: The LDAP Realm section in the docu [shiroauthentication.md](https

[GitHub] zeppelin issue #2062: ZEPPELIN-2161 Nested Group Support in LdapRealm for AD

2017-02-27 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2062 @prabhjyotsingh There isn't any documentation for org.apache.zeppelin.realm.LdapRealm yet. @Leemoonsoo Any special reason why LdapRealm wasn't documented in 0.7.0 yet? Can anyone

[GitHub] zeppelin issue #2062: ZEPPELIN-2161 Nested Group Support in LdapRealm for AD

2017-02-23 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2062 any idea why the build has failed? I don't get the error. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does

[GitHub] zeppelin pull request #2062: ZEPPELIN-2161 Nested Group Support in LdapRealm...

2017-02-23 Thread weand
GitHub user weand opened a pull request: https://github.com/apache/zeppelin/pull/2062 ZEPPELIN-2161 Nested Group Support in LdapRealm for AD ### What is this PR for? A common use case in LDAP/AD setup is the hierarchical structuring of groups - a.k.a. adding groups to other

[GitHub] zeppelin pull request #2061: ZEPPELIN-2161 Nested Group Support in LdapRealm...

2017-02-23 Thread weand
Github user weand closed the pull request at: https://github.com/apache/zeppelin/pull/2061 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] zeppelin issue #2061: ZEPPELIN-2161 Nested Group Support in LdapRealm for AD

2017-02-23 Thread weand
Github user weand commented on the issue: https://github.com/apache/zeppelin/pull/2061 any idea why the build has failed? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled

[GitHub] zeppelin pull request #2061: ZEPPELIN-2161 Nested Group Support in LdapRealm...

2017-02-23 Thread weand
GitHub user weand opened a pull request: https://github.com/apache/zeppelin/pull/2061 ZEPPELIN-2161 Nested Group Support in LdapRealm for AD ### What is this PR for? A common use case in LDAP/AD setup is the hierarchical structuring of groups - a.k.a. adding groups to other