[GitHub] zeppelin issue #2687: [ZEPPELIN-3077] Cron scheduler is easy to get stuck wh...

2017-12-07 Thread kjmrknsn
Github user kjmrknsn commented on the issue: https://github.com/apache/zeppelin/pull/2687 @felixcheung Thanks for your reviewing. > to be explicitly, let's add manually -> manually by an user opening this notebook I fixed it. (I used a correct phrase of `a user`.)

[GitHub] zeppelin issue #2699: [MINOR] Fix notebook title bar margin

2017-12-07 Thread tinkoff-dwh
Github user tinkoff-dwh commented on the issue: https://github.com/apache/zeppelin/pull/2699 LGTM ---

[GitHub] zeppelin issue #2695: [ZEPPELIN-3089] Create user folders under Trash folder

2017-12-07 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/2695 this https://github.com/apache/zeppelin/pull/2695#issuecomment-349882841 is a good point... ---

[GitHub] zeppelin issue #2676: [ZEPPELIN-3048] Add the option which prevents paragrap...

2017-12-07 Thread kjmrknsn
Github user kjmrknsn commented on the issue: https://github.com/apache/zeppelin/pull/2676 @tinkoff-dwh The folder level permission looks nice. However, in my company's use case, we'd like to make a notebook readable for many users and restrict the user who can edit it only to the cron

[GitHub] zeppelin issue #2687: [ZEPPELIN-3077] Cron scheduler is easy to get stuck wh...

2017-12-07 Thread kjmrknsn
Github user kjmrknsn commented on the issue: https://github.com/apache/zeppelin/pull/2687 I'd be glad if any committers would review this PR. ---

[GitHub] zeppelin pull request #2699: [MINOR] Fix notebook title bar margin

2017-12-07 Thread Leemoonsoo
GitHub user Leemoonsoo opened a pull request: https://github.com/apache/zeppelin/pull/2699 [MINOR] Fix notebook title bar margin ### What is this PR for? Notebook title bar location is little bit shifted to left. This minor fix notebook title bar margin. See screenshots below

[GitHub] zeppelin pull request #2685: [ZEPPELIN-3075]Fix unqiue algo for the web side...

2017-12-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2685 ---

[GitHub] zeppelin issue #2696: [ZEPPELIN-3091] Correct aggregation functionality in c...

2017-12-07 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2696 Thanks @namanmishra91 for important fix. Do you think you can add test case like https://github.com/apache/zeppelin/pull/2685, to ensure no future change break this aggregation behavior? Let me

[jira] [Created] (ZEPPELIN-3096) a non owner of a note can change notes permissions

2017-12-07 Thread gilbert marx (JIRA)
gilbert marx created ZEPPELIN-3096: -- Summary: a non owner of a note can change notes permissions Key: ZEPPELIN-3096 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3096 Project: Zeppelin

[GitHub] zeppelin issue #2696: [ZEPPELIN-3091] Correct aggregation functionality in c...

2017-12-07 Thread namanmishra91
Github user namanmishra91 commented on the issue: https://github.com/apache/zeppelin/pull/2696 Ping ---

[GitHub] zeppelin issue #2676: [ZEPPELIN-3048] Add the option which prevents paragrap...

2017-12-07 Thread tinkoff-dwh
Github user tinkoff-dwh commented on the issue: https://github.com/apache/zeppelin/pull/2676 The security is also important for our team. I agree with @Leemoonsoo and @felixcheung about the sufficiency of existing roles. We have PR about permissions for folders https://gi

[jira] [Created] (ZEPPELIN-3095) Fix UI when all paragraphs executing sequentially

2017-12-07 Thread Tinkoff DWH (JIRA)
Tinkoff DWH created ZEPPELIN-3095: - Summary: Fix UI when all paragraphs executing sequentially Key: ZEPPELIN-3095 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3095 Project: Zeppelin Is