[GitHub] zeppelin issue #1574: ZEPPELIN-1595. Make ZeppelinContext extensible

2016-11-01 Thread felixcheung
Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1574
  
In my mind the longer term goal of Zeppelin Context is to support 
communication between interpreter groups. Would we be going the opposite 
direction somewhat by having interpreter/interpreter group specific Zeppelin 
Context, and so they are not longer shared?


---
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 #1557: ZEPPELIN-1577. LivyInterpreter should not use FIFOSche...

2016-11-01 Thread felixcheung
Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1557
  
Probably something @Leemoonsoo should review


---
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 #1576: ZEPPELIN-415 document for Apache Kylin Interpreter

2016-11-01 Thread minahlee
Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1576
  
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 #1567: [ZEPPELIN-1586] Add security check in NotebookRestApi

2016-11-01 Thread tae-jun
Github user tae-jun commented on the issue:

https://github.com/apache/zeppelin/pull/1567
  
@anthonycorbacho Thanks :-) Don't worry! I do this because I want to do 
😄 


---
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 #1578: ZEPPELIN-1599 Remove support on some old versions of S...

2016-11-01 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1578
  
@Leemoonsoo We'd better leave some notes in upgrade.md in docs, too.


---
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 #1578: ZEPPELIN-1599 Remove support on some old versions of S...

2016-11-01 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1578
  
I was trying to say mark 'deprecated' in the doc. but it's also good idea 
to mention it in mailinglist :-)


---
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 #1559: [ZEPPELIN-1461] Doesn't display "description" v...

2016-11-01 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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 #1559: [ZEPPELIN-1461] Doesn't display "description" value in...

2016-11-01 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1559
  
I'll merge 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 issue #1578: ZEPPELIN-1599 Remove support on some old versions of S...

2016-11-01 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1578
  
@Leemoonsoo I'll remove those support from docs and send an email for this 
PR.


---
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 #1561: [Zeppelin 914]- Apply new mechanism to IgniteInterpret...

2016-11-01 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1561
  
@meenakshisekar The failures of this PR is related to actual behavior, not 
flaky. Can you take a look into the result of the test?


---
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 #1578: ZEPPELIN-1599 Remove support on some old versions of S...

2016-11-01 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1578
  
@jongyoul Then, I think we can update related doc and say spark 1.1~1.3 
support is deprecated.


---
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 #1578: ZEPPELIN-1599 Remove support on some old versions of S...

2016-11-01 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1578
  
I just want to remove official Spark support by Zeppelin, but if user still 
want to use old version, we don't have to block their usages.


---
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 #1578: ZEPPELIN-1599 Remove support on some old versions of S...

2016-11-01 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1578
  
@zjffdu Good catch up, but I think if users use Spark 1.3 with SPARK_HOME, 
we'd better support it. If we change the minimum version of Spark to 1.4 in 
`SparkVersion`, user cannot use spark 1.3 any more even if user set SPARK_HOME 
as kind of 1.3. What do you think of 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 issue #1578: ZEPPELIN-1599 Remove support on some old versions of S...

2016-11-01 Thread zjffdu
Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1578
  
`SparkVersion.java`  `SparkInterpreter.java` also needs to be updated. 


---
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 #1578: ZEPPELIN-1599 Remove support on some old versions of S...

2016-11-01 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1578
  
@AhyoungRyu Can you take a look into it briefly? I didn't change your part 
to avoid conflicts.


---
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 #1578: ZEPPELIN-1599 Remove support on some old versio...

2016-11-01 Thread jongyoul
GitHub user jongyoul opened a pull request:

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

ZEPPELIN-1599 Remove support on some old versions of Spark.

### What is this PR for?
removing support on old versions of Spark including testing and building 
them.


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

### Todos
* [x] - Remove .travis.yml
* [x] - Remove pom.xml
* [x] - Remove some docs

### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-1599

### How should this be tested?
No test. Check travis simplified

### Screenshots (if appropriate)

### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? You cannot use spark from 
1.1 to 1.3 any longer
* Does this needs documentation? Yes, should remove some docs

Removed some profiles concerning old versions of Spark

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

$ git pull https://github.com/jongyoul/zeppelin ZEPPELIN-1599

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

https://github.com/apache/zeppelin/pull/1578.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 #1578


commit 207502d22bed0415f67a67dab711a0dc3139d467
Author: Jongyoul Lee 
Date:   2016-11-02T04:08:46Z

Removed some tests for old versions of Spark
Removed some profiles concerning old versions of Spark




---
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-1599) Remove support on some old versions of Spark.

2016-11-01 Thread Jongyoul Lee (JIRA)
Jongyoul Lee created ZEPPELIN-1599:
--

 Summary: Remove support on some old versions of Spark.
 Key: ZEPPELIN-1599
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1599
 Project: Zeppelin
  Issue Type: New Feature
Reporter: Jongyoul Lee
Assignee: Jongyoul Lee


Spark deprecated some old version. I think we don't need to support it by 
Zeppelin itself. I'll remove the versions from 1.1.x to 1.3.x including test 
and pom



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[GitHub] zeppelin issue #1564: [ZEPPELIN-1585] Testcase for PySparkInterpreter.

2016-11-01 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1564
  
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 #1567: [ZEPPELIN-1586] Add security check in NotebookRestApi

2016-11-01 Thread anthonycorbacho
Github user anthonycorbacho commented on the issue:

https://github.com/apache/zeppelin/pull/1567
  
@tae-jun creating an issue doenst mean that you have to handle it, of 
course if you want you are welcome to do so, but remember we are a community so 
that mean we are here to help each others so if you create the PR i can spend 
some time and work with you on 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 #1569: pom.xml change

2016-11-01 Thread hunylee
Github user hunylee closed the pull request at:

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


---
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 #1567: [ZEPPELIN-1586] Add security check in NotebookR...

2016-11-01 Thread anthonycorbacho
GitHub user anthonycorbacho reopened a pull request:

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

[ZEPPELIN-1586] Add security check in NotebookRestApi

### What is this PR for?

Bring some security check in `NotebookRestApi`.
### What type of PR is it?

[Bug Fix | Improvement | Refactoring]
### Todos
- [x] - Create a proper way to throw webapp error
- [x] - Add in `NotebookAuthorization` some method to check if user is 
owner, reader or writer
- [x] - Add Authorization check in `NotebookRestapi` 
- [x] - Add New test for security in notebook rest api

### What is the Jira issue?
- [ZEPPELIN-1586](https://issues.apache.org/jira/browse/ZEPPELIN-1586)
### How should this be tested?

First, force Zeppelin to use auth.
- In `conf/zeppelin-site.xml` change `zeppelin.anonymous.allowed` to 
**false**
  
  ```
  
  zeppelin.anonymous.allowed
  false
  Anonymous user allowed by default
  
  ```
- In `conf/shiro.ini` set Shiro to use `Auth` at the end of the file
  
  ```
  #/** = anon   

  /** = authc
  ```
- Start Zeppelin, login and set some permission to a note
- try to get a note from Zeppelin Rest Api 
`http://localhost:8080/api/notebook/{noteId}` (you can use your browser or curl 
(if you use curl please add shiro token to curl cookie))
### Screenshots (if appropriate)


![note_permission_rest_api](https://cloud.githubusercontent.com/assets/3139557/19827600/ffd68a06-9dea-11e6-8dd5-43f3bd401011.gif)
### Questions:
- Does the licenses files need update? No
- Is there breaking changes for older versions? No
- Does this needs documentation? Maybe


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

$ git pull https://github.com/anthonycorbacho/incubator-zeppelin 
fix/ZEPPELIN-1586

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

https://github.com/apache/zeppelin/pull/1567.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 #1567


commit a69a4e10c46452dcaaf8174c37e51705d6bc3214
Author: Anthony Corbacho 
Date:   2016-10-28T05:49:10Z

Add new method on ZeppelinConfiguration to get is zeppelin is running on 
anonimous mode or not

commit 859fba52799bc1690b0726dc6dc672f9c4b016a7
Author: Anthony Corbacho 
Date:   2016-10-28T07:22:13Z

Add new method to help to determinate if user is part of writer and/or 
owner for the given note

commit 67c63148233c0bf31715c9d97b37f4702fb3d2e6
Author: Anthony Corbacho 
Date:   2016-10-28T09:56:04Z

Add new method to check if user and roles are member of the note (at least 
owner, reader, writer)

commit 43ec38932300db156a602507ae82a34a0b416257
Author: Anthony Corbacho 
Date:   2016-10-28T09:57:06Z

Replace check of aninonimous by method

commit 860f4b4ff789ac7087932920f71a73b30cbc9ba8
Author: Anthony Corbacho 
Date:   2016-10-28T11:19:32Z

Add webapp exception handler :)

commit 01bec7ae0ece70fe5eed6f068bc40b4cc61ffd8d
Author: Anthony Corbacho 
Date:   2016-10-28T11:19:54Z

Handle security check

commit c0b98ffe2f1a3c4ee005ffe8b376bdc507eab2f0
Author: Anthony Corbacho 
Date:   2016-10-29T05:58:04Z

Rename permission check note :: be more meaningful

commit a1cc1b81bd8f69f068803e6e87d110805905a039
Author: Anthony Corbacho 
Date:   2016-11-01T03:33:20Z

Change cxf version from 2.7.7 to 2.7.8 to avoid method not found where 
throw WebAppException

commit 0cf19d2c41b288dc3a02cefc64de07c30b91e9ec
Author: Anthony Corbacho 
Date:   2016-11-01T08:24:38Z

Fix typo and bad copy paste for isOwner

commit 2fd359cda8fe8aab72177c9c8c59b5bde3eb1c6c
Author: Anthony Corbacho 
Date:   2016-11-01T08:30:17Z

Adress review and fix typos

commit 4768e3a73f3cec87c86ab8cabb05a7fe483aa922
Author: Anthony Corbacho 
Date:   2016-11-01T14:48:14Z

Refactored Abstract rest api test to also handle the case of tests with 
shiro (security), I also added some utility http method to do action with 
authenticated user

commit 1055a837cb138d86618c895b71346d95f743496e
Author: Anthony Corbacho 
Date:   2016-11-01T14:49:00Z

Simple implementation of notebook test with shiro (security)

commit 15c776d5889bd9c9d06d6246c3101aa1a2f4cb6d
Author: Anthony Corbacho 
Date:   2016-11-01T14:49:26Z

Rewording

commit 216b5cdd32183d9907ce33f8a91713909e790404
Author: Anthony 

[GitHub] zeppelin issue #1569: pom.xml change

2016-11-01 Thread hunylee
Github user hunylee commented on the issue:

https://github.com/apache/zeppelin/pull/1569
  
@AhyoungRyu Closed. 


---
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 #1567: [ZEPPELIN-1586] Add security check in NotebookRestApi

2016-11-01 Thread tae-jun
Github user tae-jun commented on the issue:

https://github.com/apache/zeppelin/pull/1567
  
@anthonycorbacho Nice! I agree with you :)

I will open the issue on JIRA. But since I don't know much about the code 
structure, I may need some help 😃


---
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 #1571: [DOC] Update shiro after merging #1568

2016-11-01 Thread anthonycorbacho
Github user anthonycorbacho closed the pull request at:

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


---
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 #1567: [ZEPPELIN-1586] Add security check in NotebookRestApi

2016-11-01 Thread anthonycorbacho
Github user anthonycorbacho commented on the issue:

https://github.com/apache/zeppelin/pull/1567
  
@tae-jun After looking at the code base, I think this case should be handle 
in another PR, this is kinda out of the scope of this PR and its already 
becoming super big.

But your made a very valid point here and I guess it deserve a Jira ticket. 
i will try to find some time to handle this special case.

What do you think?


---
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 #1575: ZEPPELIN-1597. Change cdh spark/hadoop dependency to a...

2016-11-01 Thread zjffdu
Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1575
  
@prabhjyotsingh @felixcheung Please help review.


---
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 #1577: [ZEPPELIN-1554] Update jekyll version to 3.2.1 with it...

2016-11-01 Thread yoonjs2
Github user yoonjs2 commented on the issue:

https://github.com/apache/zeppelin/pull/1577
  
@AhyoungRyu @minahlee Thanks! Let me know if patch needs any change :)


---
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 #1516: ZEPPELIN-1546 - Zeppelin Livy Interpreter 404 Exceptio...

2016-11-01 Thread zjffdu
Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1516
  
@gss2002 CI is failed, could you check whether it is related ?


---
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 #1577: [ZEPPELIN-1554] Update jekyll version to 3.2.1 with it...

2016-11-01 Thread minahlee
Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1577
  
@yoonjs2 just added your name to contributor role


---
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 #1569: pom.xml change

2016-11-01 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1569
  
@hunylee Yeah I think the version is not the problem now. 
Did you reopen the PR on purpose? If not, could you close this then? 


---
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 #1577: [ZEPPELIN-1554] Update jekyll version to 3.2.1 with it...

2016-11-01 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1577
  
@yoonjs2 Sure. And let me test this patch :)
@minahlee Could you help me add @yoonjs2 (Jira username:Jesang Yoon) to the 
contributor list in Jira so that he can assign himself to assignee? 


---
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 #1575: ZEPPELIN-1597. Change cdh spark/hadoop dependen...

2016-11-01 Thread zjffdu
GitHub user zjffdu reopened a pull request:

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

ZEPPELIN-1597. Change cdh spark/hadoop dependency to apache spark/hadoop

### What is this PR for?
I see several times of travis failure due to downloading cdh spark/hadoop 
for livy interpreter. This may be due to cdh repository issue, so in this PR I 
change it to apache spark/hadoop.


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

### Todos
* [ ] - Task

### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-1597

### How should this be tested?
No test added

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



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

$ git pull https://github.com/zjffdu/zeppelin ZEPPELIN-1597

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

https://github.com/apache/zeppelin/pull/1575.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 #1575


commit 5077f2041f016bcfb2e295fc0d8399be4088156b
Author: Jeff Zhang 
Date:   2016-11-01T09:41:49Z

ZEPPELIN-1597. Change cdh spark/hadoop dependency to apache spark/hadoop




---
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 #1575: ZEPPELIN-1597. Change cdh spark/hadoop dependen...

2016-11-01 Thread zjffdu
Github user zjffdu closed the pull request at:

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


---
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 #1561: [Zeppelin 914]- Apply new mechanism to IgniteIn...

2016-11-01 Thread meenakshisekar
Github user meenakshisekar closed the pull request at:

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


---
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 #1561: [Zeppelin 914]- Apply new mechanism to IgniteIn...

2016-11-01 Thread meenakshisekar
GitHub user meenakshisekar reopened a pull request:

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

[Zeppelin 914]- Apply new mechanism to IgniteInterpreter

What is this PR for?

This handles replacing the registration of interpreter with static block by 
the interpreter-setting.json file

What type of PR is it?

[| Improvement |

Todos

Sub-Task

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-914

How should this be tested?

There shouldn't be any warning like below on starting the server
INFO 2016-09-29 00:25:46,247 - Bootstrapping Ignite Interpreter WARN 
2016-09-29 00:25:46,250 - Static initialization is deprecated for interpreter 
Ignite, You should change it to use interpreter-setting.json in your jar or 
interpreter/{interpreter}/interpreter-setting.json INFO 2016-09-29 00:25:46,250 
- Inclass=org.apache.zeppelin.ignite

And ensure that the Ignite related paragraphs run without any error

Screenshots (if appropriate)

Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No


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

$ git pull https://github.com/meenakshisekar/zeppelin ZEPPELIN-914

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

https://github.com/apache/zeppelin/pull/1561.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 #1561


commit 57241368855c62a2975bd44852f77618ddffdbb1
Author: meenakshisekar 
Date:   2016-10-24T12:40:12Z

Added interpreter setting and removed registration from interperter.

commit 640ad3020a53ce4fbd7965402e877c514232e0c2
Author: meenakshisekar 
Date:   2016-10-24T12:44:43Z

Added interpreter setting file.

commit d7d846b1bc4f42b427cd69aa96e4d6ad195ee3a3
Author: meenakshisekar 
Date:   2016-10-24T12:44:43Z

Added interpreter setting file.

commit 18ee9694fb295790171ccd2917ec32f93ef0d883
Author: meenakshisekar 
Date:   2016-10-26T10:14:38Z

Corrected the type O

commit 7c328a17c6f043ba4fb3ca358390614cbb66750f
Author: meenakshisekar 
Date:   2016-10-26T10:27:25Z

Added indentation

commit 44bc6d1d93893b5c2ef717281c14b4bf620c084b
Author: meenakshisekar 
Date:   2016-10-27T11:54:51Z

Added igniteSQl interpreter to the group.

commit 0ec5da8d017507fe2a2bbbd0948edecc70766104
Author: meenakshisekar 
Date:   2016-10-31T07:36:35Z

Removed properties in interpreter file and added new line in json file.

commit a8599a49f789c2fdaf2bde424ba47ea964a7ba68
Author: meenakshisekar 
Date:   2016-10-31T07:36:35Z

Removed properties in interpreter file and added new line in json file.




---
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 #1548: [ZEPPELIN-917] Apply new mechanism to LensInter...

2016-11-01 Thread meenakshisekar
GitHub user meenakshisekar reopened a pull request:

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

[ZEPPELIN-917] Apply new mechanism to LensInterpreter

### What is this PR for?

This handles replacing the registration of interpreter with static block by 
the interpreter-setting.json file
### What type of PR is it?

[| Improvement | 
### Todos

Sub-Task
### What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-917
### How should this be tested?

here shouldn't be any warning like below on starting the server
INFO [2016-09-29 00:25:46,247]({main} LensInterpreter.java[]:155) - 
Bootstrapping Cassandra Interpreter WARN [2016-09-29 00:25:46,250]({main} 
Interpreter.java[register]:347) - Static initialization is deprecated for 
interpreter lens, You should change it to use interpreter-setting.json in your 
jar or interpreter/{interpreter}/interpreter-setting.json INFO [2016-09-29 
00:25:46,250]({main} InterpreterFactory.java[init]:204) - 
Inclass=org.apache.zeppelin.lens.LensInterpreter

And ensure that the lens related paragraphs run without any error
### Screenshots (if appropriate)
### Questions:
- Does the licenses files need update? No
- Is there breaking changes for older versions? No
- Does this needs documentation? No


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

$ git pull https://github.com/meenakshisekar/zeppelin ZEPPELIN-917

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

https://github.com/apache/zeppelin/pull/1548.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 #1548


commit db2827f867199cdd5e3fe2bc05f614dd88e013f5
Author: meenakshisekar 
Date:   2016-10-21T09:59:38Z

Commented the properties in interpreter file and added setting file.

commit f94c8a3d44d441174fff2e89708e9ec62c1f5d22
Author: meenakshisekar 
Date:   2016-10-21T11:41:58Z

Commented the properties in interpreter file and added setting file.

commit ad3b83ca856c54727f1ffef324f187f69a276c83
Author: meenakshisekar 
Date:   2016-10-24T11:57:46Z

Added indentation

commit cc8bdc274f581e7725307e2a4706b4d8ce4e243c
Author: meenakshisekar 
Date:   2016-10-27T07:12:54Z

Added the indentation similar to other interpreter setting file.

commit 2cba9e74df90073bd03c2867a70c345e5346dc7e
Author: meenakshisekar 
Date:   2016-10-27T07:12:54Z

Added the indentation similar to other interpreter setting file.

commit 269e8cb3c547d5f1ea932c571206c13931a462ca
Author: meenakshisekar 
Date:   2016-10-27T08:02:10Z

Removed the tabs and added space.




---
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 #1577: [ZEPPELIN-1554] Update jekyll version to 3.2.1 with it...

2016-11-01 Thread yoonjs2
Github user yoonjs2 commented on the issue:

https://github.com/apache/zeppelin/pull/1577
  
@AhyoungRyu Can I ask you to allow me to change JIRA ZEPPELIN-1554 assignee 
and progress? In case of self proposal and PR like this, It would be great to 
use JIRA as contributor access :)


---
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 #1577: [ZEPPELIN-1554] Update jekyll version to 3.2.1 ...

2016-11-01 Thread yoonjs2
GitHub user yoonjs2 opened a pull request:

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

[ZEPPELIN-1554] Update jekyll version to 3.2.1 with its dependencies and 
fix document…

### What is this PR for?
- Update outdated jekyll version of zeppelin documentation from 1.x to 3.x
- Eliminate kramdown only warnings when serving documentation as github 
pages.
- Prepare base environment for L10N related plugins for jekyll.

### What type of PR is it?
Improvement | Documentation

### Todos
* [*] - Update Jekyll version to 3.x
* [*] - Update documentation style as kramdown
* [  ] - Prepare L10N support on documentation

### What is the Jira issue?
ZEPPELIN-1554

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

… style as kramdown.

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

$ git pull https://github.com/yoonjs2/zeppelin ZEPPELIN-1554

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

https://github.com/apache/zeppelin/pull/1577.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 #1577


commit 81bc244a25f274a1ba3910ae0e36ef6370db125a
Author: Jesang Yoon 
Date:   2016-10-09T15:30:15Z

Update jekyll version to 3.2.1 with its dependencies and fix document style 
as kramdown.




---
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 #1559: [ZEPPELIN-1461] Doesn't display "description" value in...

2016-11-01 Thread cloverhearts
Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/1559
  
Tested.
Looks good to me!
Very good and clear code!
+1



---
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 #1565: [ZEPPELIN-1564] Enable note deletion and paragraph out...

2016-11-01 Thread tae-jun
Github user tae-jun commented on the issue:

https://github.com/apache/zeppelin/pull/1565
  
@minahlee Awesome! I just changed status to `IN PROGRESS` now. Thanks 👍


---
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 #1565: [ZEPPELIN-1564] Enable note deletion and paragraph out...

2016-11-01 Thread minahlee
Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1565
  
@tae-jun I just added your name to contributor list so you will be able to 
set yourself as assignee from now.


---
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.
---


Update Ignite interpreter related docs on the Zeppelin site

2016-11-01 Thread Andrey Gura
Hi all.

Guys, could anybody point me how to edit documentation on Zeppelin site?

In particular I want to fix Ignite and Zeppelin version mentions and may be
other things here
https://zeppelin.apache.org/docs/0.6.2/interpreter/ignite.html.

Also I have question about this quote: "We are planning to include ignite
in a future binary release." What does it mean from Ignie community
perspective (should we do smth) and from Zeppelin community perspective
(what are you planning do exactly).

Thanks!


[GitHub] zeppelin issue #1565: [ZEPPELIN-1564] Enable note deletion and paragraph out...

2016-11-01 Thread tae-jun
Github user tae-jun commented on the issue:

https://github.com/apache/zeppelin/pull/1565
  
Thanks @minahlee! I appreciate your kind comment 😄

I created a ticket on JIRA: 
[ZEPPELIN-1598](https://issues.apache.org/jira/browse/ZEPPELIN-1598)

But, I don't know how to set myself to assignee, and change status to 
`PROGRESS`. I guess I don't have a permission to do that 😢

I will check #1567. Thanks 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.
---


[jira] [Created] (ZEPPELIN-1598) Enable renaming note & folder from the main page & REST API

2016-11-01 Thread Tae Jun Kim (JIRA)
Tae Jun Kim created ZEPPELIN-1598:
-

 Summary: Enable renaming note & folder from the main page & REST 
API
 Key: ZEPPELIN-1598
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1598
 Project: Zeppelin
  Issue Type: Improvement
  Components: front-end, zeppelin-server
Affects Versions: 0.6.0
Reporter: Tae Jun Kim


A new feature to rename note & folder from the main page and REST API.

I will open a PR after merge these two PRs: 
[ZEPPELIN-1564|https://github.com/apache/zeppelin/pull/1565], 
[ZEPPELIN-1586|https://github.com/apache/zeppelin/pull/1567]

Required steps:
* Enable note renaming from the main page
* Enable folder renaming from the main page
* Add REST API for renaming note & folder



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[GitHub] zeppelin pull request #1575: ZEPPELIN-1597. Change cdh spark/hadoop dependen...

2016-11-01 Thread zjffdu
GitHub user zjffdu opened a pull request:

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

ZEPPELIN-1597. Change cdh spark/hadoop dependency to apache spark/hadoop

### What is this PR for?
I see several times of travis failure due to downloading cdh spark/hadoop 
for livy interpreter. This may be due to cdh repository issue, so in this PR I 
change it to apache spark/hadoop.


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

### Todos
* [ ] - Task

### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-1597

### How should this be tested?
No test added

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



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

$ git pull https://github.com/zjffdu/zeppelin ZEPPELIN-1597

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

https://github.com/apache/zeppelin/pull/1575.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 #1575


commit 5077f2041f016bcfb2e295fc0d8399be4088156b
Author: Jeff Zhang 
Date:   2016-11-01T09:41:49Z

ZEPPELIN-1597. Change cdh spark/hadoop dependency to apache spark/hadoop




---
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 #1574: ZEPPELIN-1595. Make ZeppelinContext extensible

2016-11-01 Thread zjffdu
Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1574
  
@Leemoonsoo @jongyoul @felixcheung  Please help review. 


---
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 #1569: pom.xml change

2016-11-01 Thread hunylee
GitHub user hunylee reopened a pull request:

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

pom.xml change

### What is this PR for?
## pom.xml is plugin version change
- pom.xml
- beam/pom.xml
- cassandra/pom.xml
- flink/pom.xml
- ignite/pom.xml
- r/pom.xml
- scalding/pom.xml
- spark/pom.xml
- spark-dependencies/pom.xml
- zeppelin-display/pom.xml

### What type of PR is it?

[Improvement | Feature | Documentation]
### Todos
- [ ] zeppelin is pom.xml in the plugin version change [ZEPPELIN-1569]
### How should this be tested?

- Is there breaking changes for older versions?



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

$ git pull https://github.com/hunylee/zeppelin master

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

https://github.com/apache/zeppelin/pull/1569.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 #1569


commit 869ee334fe36ab275633e688b569c2910efbf669
Author: hunylee 
Date:   2016-10-30T06:09:18Z

pom.xml change

commit 54343dae5c89d1ca4d8e6d902bef137b58ba5adc
Author: hunylee 
Date:   2016-10-31T07:21:48Z

Version change

commit 8ff2389b743205cb44167b10cd8276f1549d3917
Author: hunylee 
Date:   2016-10-31T08:24:25Z

change




---
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 #1569: pom.xml change

2016-11-01 Thread hunylee
Github user hunylee closed the pull request at:

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


---
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 #1574: ZEPPELIN-1595. Make ZeppelinContext extensible

2016-11-01 Thread zjffdu
GitHub user zjffdu reopened a pull request:

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

ZEPPELIN-1595. Make ZeppelinContext extensible

### What is this PR for?
For now, ZeppelinContext only support Spark Interpreter. I'd like to make 
it extensible, so that it can support other interpreters as well.


### What type of PR is it?
[Feature | Refactoring]

### Todos
* [ ] - Task

### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-1595

### How should this be tested?
Outline the steps to test the PR here.

### Screenshots (if appropriate)

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



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

$ git pull https://github.com/zjffdu/zeppelin ZEPPELIN-1595

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

https://github.com/apache/zeppelin/pull/1574.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 #1574


commit 95ca5e51a458cbb930a794e108eac85c73525d76
Author: Jeff Zhang 
Date:   2016-10-31T07:33:51Z

ZEPPELIN-1595. Make ZeppelinContext extensible




---
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 #1567: [ZEPPELIN-1586] Add security check in NotebookRestApi

2016-11-01 Thread anthonycorbacho
Github user anthonycorbacho commented on the issue:

https://github.com/apache/zeppelin/pull/1567
  
@minahlee you are right, I guess the next step will be to abstract this 
logic from rest api and apply to both rest and websocket.



---
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 #1567: [ZEPPELIN-1586] Add security check in NotebookRestApi

2016-11-01 Thread minahlee
Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1567
  
I tested some of rest apis and it works well. Next step would be applying 
same policies to websocket. For example `reader` cannot change bound 
interpreter to note via rest api after this PR, but it is possible to do it via 
websocket(or GUI).


---
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 #1567: [ZEPPELIN-1586] Add security check in NotebookRestApi

2016-11-01 Thread anthonycorbacho
Github user anthonycorbacho commented on the issue:

https://github.com/apache/zeppelin/pull/1567
  
@minahlee yeah, actually i am doing this right now, I also updated to todo 
tasks thanks for your review!


---
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 #1567: [ZEPPELIN-1586] Add security check in NotebookRestApi

2016-11-01 Thread minahlee
Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1567
  
Thank you for quick response, I only went through the code and it looks 
good to me. It would be nice if you can add some tests. Meanwhile let me build 
this branch and test it out. 


---
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-1595) Make ZeppelinContext extensible

2016-11-01 Thread Jeff Zhang (JIRA)
Jeff Zhang created ZEPPELIN-1595:


 Summary: Make ZeppelinContext extensible
 Key: ZEPPELIN-1595
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1595
 Project: Zeppelin
  Issue Type: New Feature
Affects Versions: 0.6.2
Reporter: Jeff Zhang
Assignee: Jeff Zhang
 Fix For: 0.7.0


For now, ZeppelinContext only support Spark Interpreter. I'd like to make it 
extensible, so that it can support other interpreters as well. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[GitHub] zeppelin pull request #1567: [ZEPPELIN-1586] Add security check in NotebookR...

2016-11-01 Thread anthonycorbacho
Github user anthonycorbacho closed the pull request at:

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


---
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 #1567: [ZEPPELIN-1586] Add security check in NotebookR...

2016-11-01 Thread anthonycorbacho
GitHub user anthonycorbacho reopened a pull request:

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

[ZEPPELIN-1586] Add security check in NotebookRestApi

### What is this PR for?

Bring some security check in `NotebookRestApi`.
### What type of PR is it?

[Bug Fix | Improvement | Refactoring]
### Todos
- [x] - Create a proper way to throw webapp error
- [x] - Add in `NotebookAuthorization` some method to check if user is 
owner, reader or writer
- [x] - Add Authorization check in `NotebookRestapi` 
### What is the Jira issue?
- [ZEPPELIN-1586](https://issues.apache.org/jira/browse/ZEPPELIN-1586)
### How should this be tested?

First, force Zeppelin to use auth.
- In `conf/zeppelin-site.xml` change `zeppelin.anonymous.allowed` to 
**false**
  
  ```
  
  zeppelin.anonymous.allowed
  false
  Anonymous user allowed by default
  
  ```
- In `conf/shiro.ini` set Shiro to use `Auth` at the end of the file
  
  ```
  #/** = anon   

  /** = authc
  ```
- Start Zeppelin, login and set some permission to a note
- try to get a note from Zeppelin Rest Api 
`http://localhost:8080/api/notebook/{noteId}` (you can use your browser or curl 
(if you use curl please add shiro token to curl cookie))
### Screenshots (if appropriate)


![note_permission_rest_api](https://cloud.githubusercontent.com/assets/3139557/19827600/ffd68a06-9dea-11e6-8dd5-43f3bd401011.gif)
### Questions:
- Does the licenses files need update? No
- Is there breaking changes for older versions? No
- Does this needs documentation? Maybe


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

$ git pull https://github.com/anthonycorbacho/incubator-zeppelin 
fix/ZEPPELIN-1586

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

https://github.com/apache/zeppelin/pull/1567.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 #1567


commit a69a4e10c46452dcaaf8174c37e51705d6bc3214
Author: Anthony Corbacho 
Date:   2016-10-28T05:49:10Z

Add new method on ZeppelinConfiguration to get is zeppelin is running on 
anonimous mode or not

commit 859fba52799bc1690b0726dc6dc672f9c4b016a7
Author: Anthony Corbacho 
Date:   2016-10-28T07:22:13Z

Add new method to help to determinate if user is part of writer and/or 
owner for the given note

commit 67c63148233c0bf31715c9d97b37f4702fb3d2e6
Author: Anthony Corbacho 
Date:   2016-10-28T09:56:04Z

Add new method to check if user and roles are member of the note (at least 
owner, reader, writer)

commit 43ec38932300db156a602507ae82a34a0b416257
Author: Anthony Corbacho 
Date:   2016-10-28T09:57:06Z

Replace check of aninonimous by method

commit 860f4b4ff789ac7087932920f71a73b30cbc9ba8
Author: Anthony Corbacho 
Date:   2016-10-28T11:19:32Z

Add webapp exception handler :)

commit 01bec7ae0ece70fe5eed6f068bc40b4cc61ffd8d
Author: Anthony Corbacho 
Date:   2016-10-28T11:19:54Z

Handle security check

commit c0b98ffe2f1a3c4ee005ffe8b376bdc507eab2f0
Author: Anthony Corbacho 
Date:   2016-10-29T05:58:04Z

Rename permission check note :: be more meaningful

commit a1cc1b81bd8f69f068803e6e87d110805905a039
Author: Anthony Corbacho 
Date:   2016-11-01T03:33:20Z

Change cxf version from 2.7.7 to 2.7.8 to avoid method not found where 
throw WebAppException




---
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 #1567: [ZEPPELIN-1586] Add security check in NotebookR...

2016-11-01 Thread anthonycorbacho
Github user anthonycorbacho closed the pull request at:

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


---
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 #1561: [Zeppelin 914]- Apply new mechanism to IgniteInterpret...

2016-11-01 Thread meenakshisekar
Github user meenakshisekar commented on the issue:

https://github.com/apache/zeppelin/pull/1561
  
Reopening for TRavis


---
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 #1561: [Zeppelin 914]- Apply new mechanism to IgniteIn...

2016-11-01 Thread meenakshisekar
GitHub user meenakshisekar reopened a pull request:

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

[Zeppelin 914]- Apply new mechanism to IgniteInterpreter

What is this PR for?

This handles replacing the registration of interpreter with static block by 
the interpreter-setting.json file

What type of PR is it?

[| Improvement |

Todos

Sub-Task

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-914

How should this be tested?

There shouldn't be any warning like below on starting the server
INFO 2016-09-29 00:25:46,247 - Bootstrapping Ignite Interpreter WARN 
2016-09-29 00:25:46,250 - Static initialization is deprecated for interpreter 
Ignite, You should change it to use interpreter-setting.json in your jar or 
interpreter/{interpreter}/interpreter-setting.json INFO 2016-09-29 00:25:46,250 
- Inclass=org.apache.zeppelin.ignite

And ensure that the Ignite related paragraphs run without any error

Screenshots (if appropriate)

Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No


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

$ git pull https://github.com/meenakshisekar/zeppelin ZEPPELIN-914

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

https://github.com/apache/zeppelin/pull/1561.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 #1561


commit 57241368855c62a2975bd44852f77618ddffdbb1
Author: meenakshisekar 
Date:   2016-10-24T12:40:12Z

Added interpreter setting and removed registration from interperter.

commit 640ad3020a53ce4fbd7965402e877c514232e0c2
Author: meenakshisekar 
Date:   2016-10-24T12:44:43Z

Added interpreter setting file.

commit d7d846b1bc4f42b427cd69aa96e4d6ad195ee3a3
Author: meenakshisekar 
Date:   2016-10-24T12:44:43Z

Added interpreter setting file.

commit 18ee9694fb295790171ccd2917ec32f93ef0d883
Author: meenakshisekar 
Date:   2016-10-26T10:14:38Z

Corrected the type O

commit 7c328a17c6f043ba4fb3ca358390614cbb66750f
Author: meenakshisekar 
Date:   2016-10-26T10:27:25Z

Added indentation

commit 44bc6d1d93893b5c2ef717281c14b4bf620c084b
Author: meenakshisekar 
Date:   2016-10-27T11:54:51Z

Added igniteSQl interpreter to the group.

commit 0ec5da8d017507fe2a2bbbd0948edecc70766104
Author: meenakshisekar 
Date:   2016-10-31T07:36:35Z

Removed properties in interpreter file and added new line in json file.

commit a8599a49f789c2fdaf2bde424ba47ea964a7ba68
Author: meenakshisekar 
Date:   2016-10-31T07:36:35Z

Removed properties in interpreter file and added new line in json file.




---
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 #1561: [Zeppelin 914]- Apply new mechanism to IgniteIn...

2016-11-01 Thread meenakshisekar
Github user meenakshisekar closed the pull request at:

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


---
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 #1414: filter applied to graph

2016-11-01 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1414
  
@lobamba Great feature indeed!
As @Leemoonsoo said, I also would suggest to discuss about this newly 
proposed feature (maybe in `dev@zeppelin.apache.org`) first and get other ppl's 
opinion. It would be more helpful to proceed your work i think.  


---
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 #1564: [ZEPPELIN-1585] Testcase for PySparkInterpreter.

2016-11-01 Thread astroshim
Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1564
  
@felixcheung I fixed it and CI has passed. Thank you very much!


---
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 #1472: Add message of localhost:port on script

2016-11-01 Thread djKooks
Github user djKooks commented on the issue:

https://github.com/apache/zeppelin/pull/1472
  
@AhyoungRyu Okay. Thanks~


---
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-1594) Personalized mode

2016-11-01 Thread Jongyoul Lee (JIRA)
Jongyoul Lee created ZEPPELIN-1594:
--

 Summary: Personalized mode
 Key: ZEPPELIN-1594
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1594
 Project: Zeppelin
  Issue Type: New Feature
Reporter: Jongyoul Lee
Assignee: Jongyoul Lee


This enables two different users to change the view of the same paragraph's 
result with different view even owner change paragraph and run it again.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)