[GitHub] zeppelin issue #2512: [ZEPPELIN-2823] Notebook saved status is wrong if ther...

2017-08-10 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2512
  
LGTM!


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2462: ZEPPELIN-2527 Changed editor cursor to thin

2017-08-10 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2462
  
@malayhm Merged to master. Thanks for the contribution again!


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2524: [ZEPPELIN-2846] Add selenium test case for AnyOfRolesA...

2017-08-10 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/2524
  
Ready for review, 
https://travis-ci.org/prabhjyotsingh/zeppelin/builds/263300475. 439.2 Failed 
which is unrelated (have restarted it).


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #2524: [ZEPPELIN-2846] Add selenium test case for AnyO...

2017-08-10 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request:

https://github.com/apache/zeppelin/pull/2524

[ZEPPELIN-2846] Add selenium test case for AnyOfRolesAuthorizationFilter

### What is this PR for?
This is to test the new feature that was brought in with ZEPPELIN-2825 
(org.apache.zeppelin.utils.AnyOfRolesAuthorizationFilter)


### What type of PR is it?
[Improvement]

### What is the Jira issue?
* [ZEPPELIN-2846](https://issues.apache.org/jira/browse/ZEPPELIN-2846)

### How should this be tested?
CI should be green

### Questions:
* Does the licenses files need update? N/A
* Is there breaking changes for older versions? N/A
* Does this needs documentation? N/A


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/prabhjyotsingh/zeppelin ZEPPELIN-2846

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2524.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2524


commit e2a7ad548516a91c774afeb80d905859c47f8983
Author: Prabhjyot Singh 
Date:   2017-08-10T23:27:28Z

add selenium test case for AnyOfRolesAuthorizationFilter




---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[jira] [Created] (ZEPPELIN-2846) Add selenium test case for AnyOfRolesAuthorizationFilter

2017-08-10 Thread Prabhjyot Singh (JIRA)
Prabhjyot Singh created ZEPPELIN-2846:
-

 Summary: Add selenium test case for AnyOfRolesAuthorizationFilter
 Key: ZEPPELIN-2846
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2846
 Project: Zeppelin
  Issue Type: Bug
  Components: front-end
Affects Versions: 0.8.0
Reporter: Prabhjyot Singh
Assignee: Prabhjyot Singh


This is to test the new feature that was brought in with ZEPPELIN-2825 
(org.apache.zeppelin.utils.AnyOfRolesAuthorizationFilter)



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[GitHub] zeppelin issue #2512: [ZEPPELIN-2823] Notebook saved status is wrong if ther...

2017-08-10 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/2512
  
@1ambda  thanks for the review, have handled your review comments.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #2514: replace pegdown to markdown zeppelin interprete...

2017-08-10 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/zeppelin/pull/2514


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2521: [ZEPPELIN-2841] fix a problem in shell interpreter . W...

2017-08-10 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2521
  
LGTM and merge to master and branch-0.7 if no further comment.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[jira] [Created] (ZEPPELIN-2845) WebSocke will sent "COMMIT_PARAGRAPH" when load nodebook

2017-08-10 Thread zhouqi (JIRA)
zhouqi created ZEPPELIN-2845:


 Summary: WebSocke will sent "COMMIT_PARAGRAPH" when load nodebook  
 Key: ZEPPELIN-2845
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2845
 Project: Zeppelin
  Issue Type: Bug
  Components: front-end
Affects Versions: 0.8.0
 Environment: chrome browser
Reporter: zhouqi
Priority: Critical


when chat types are area chat、bar chat and line chat webSocke will sent 
"COMMIT_PARAGRAPH" when load nodebook,this will result in failure to access 
users without write privileges,please help me.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[GitHub] zeppelin issue #2508: [ZEPPELIN-2817] Support default interpreter setting in...

2017-08-10 Thread reminia
Github user reminia commented on the issue:

https://github.com/apache/zeppelin/pull/2508
  
I decide to leave the api no change at now.  Maybe another jira should be 
created to address this. I've found that several create operation api doesnt 
return 201 but 200, such as repo add, paragraph add. @felixcheung @Leemoonsoo 


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---