[GitHub] zeppelin issue #1514: ZEPPELIN-1516. NPE LivySparkSQLInterpreter thrown with...

[felixcheung]

Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1514
  
Great, one question LGTM otherwise


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

[anthonycorbacho]

Github user anthonycorbacho commented on the issue:

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


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

[tae-jun]

Github user tae-jun commented on the issue:

https://github.com/apache/zeppelin/pull/1567
  
I guess it's same CI error with #1518.

There is a log which is:
```bash
gzip: stdin: unexpected end of file
tar: Unexpected EOF in archive
tar: Unexpected EOF in archive
tar: Error is not recoverable: exiting now
+echo 'Unable to extract spark-1.3.1-bin-hadoop2.3.tgz'
Unable to extract spark-1.3.1-bin-hadoop2.3.tgz
```

This is because of cache failure of Travis CI, and if it happens, it goes 
forever. Because caching is done only at the first time.

Because of this, if you follow the log more, you can see:
```bash
SPARK HOME detected null
```
Therefore, Zeppelin cannot find Spark, and it goes to failure.

I think it will pass if you run Travis CI on your own repository.


---
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 #1585: ZEPPELIN-1606. Add interpreter option to interpreter-s...

[prabhjyotsingh]

Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/1585
  
LGTM, I like this idea.


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

[minahlee]

Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1567
  
Same Spark 1.3.1 test profile failure exists on master which looks like 
below:
```
Results :

Tests in error: 
  
InterpreterRestApiTest.init:57->AbstractTestRestApi.startUp:233->AbstractTestRestApi.start:201
 » NullPointer
  
CredentialsRestApiTest.init:46->AbstractTestRestApi.startUp:233->AbstractTestRestApi.start:201
 » NullPointer
  
ZeppelinRestApiTest.init:59->AbstractTestRestApi.startUp:233->AbstractTestRestApi.start:201
 » NullPointer
  
NotebookRestApiTest.init:58->AbstractTestRestApi.startUp:233->AbstractTestRestApi.start:201
 » NullPointer
  
SecurityRestApiTest.init:44->AbstractTestRestApi.startUp:233->AbstractTestRestApi.start:201
 » NullPointer
  
NotebookRepoRestApiTest.init:52->AbstractTestRestApi.startUp:233->AbstractTestRestApi.start:201
 » NullPointer
  
ConfigurationsRestApiTest.init:39->AbstractTestRestApi.startUp:233->AbstractTestRestApi.start:201
 » NullPointer
  
ZeppelinSparkClusterTest.init:52->AbstractTestRestApi.startUp:233->AbstractTestRestApi.start:201
 » NullPointer
  
NotebookSecurityRestApiTest.init:49->AbstractTestRestApi.startUpWithAuthenticationEnable:229->AbstractTestRestApi.start:201
 » NullPointer

Tests run: 9, Failures: 0, Errors: 9, Skipped: 0
```
 So I am going to merge this if there is no more discussion.


---
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 #1585: ZEPPELIN-1606. Add interpreter option to interpreter-s...

[jongyoul]

Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1585
  
LGTM. We will simplify registration mechanism by deprecating a old way.


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

[zjffdu]

Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1557
  
Close this one, and open #1585 to allow user to set default mode in 
interpreter-setting.json


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

[zjffdu]

Github user zjffdu closed the pull request at:

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


---
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 #1585: ZEPPELIN-1606. Add interpreter option to interp...

[zjffdu]

GitHub user zjffdu opened a pull request:

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

ZEPPELIN-1606. Add interpreter option to interpreter-setting.json

### What is this PR for?
For livy, the default mode should be scoped per use. This PR is trying to 
add interpreter option to `interpreter-setting.json` to allow user to set the 
default mode. 


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

### Todos
This PR is just a short term solution. There're several followup work needs 
to be done.
* Option should be interpretergroup level setting rather than interpreter 
level setting. 
* Format of `interperter-setting.json` is not consistent with 
`Interpereter.json`, there's some code duplication, like 
`RegisteredInterpereter`,  `InterpreterInfo` and etc. 
* And this PR doesn't address the issue of mode restriction, user can still 
change to an invalid mode manually.

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


### 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-1606

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

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


commit 26e6cc240eb2ba34f6d7680dca4301325dcce80e
Author: Jeff Zhang 
Date:   2016-11-03T04:05:01Z

ZEPPELIN-1606. Add interpreter option to interpreter-setting.json




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

[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. For now, user 
need to implement the following 3 methods to extend ZeppelinContext

* public List getSupportedClasses() 
* public abstract Map getInterpreterClassMap();
* protected abstract String showData(Object obj);


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

[zjffdu]

Github user zjffdu closed the pull request at:

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


---
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 #1535: [Zeppelin-53] Broken newly created Notebook if there i...

[minahlee]

Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1535
  
@kavinkumarks Do you mind to resolve 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 #1584: [gh-pages] Add ignite version info to supported...

[AhyoungRyu]

GitHub user AhyoungRyu opened a pull request:

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

[gh-pages] Add ignite version info to supported interpreters table

### What is this PR for?
As described in [ignite 
docs](https://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/interpreter/ignite.html#installing-and-running-ignite-example),
 the user should use the Ignite version same as Zeppelin provides. It means,

```
Zeppelin-0.5.5: 1.4.0
Zeppelin-0.5.5: 1.4.0
Zeppelin-0.5.5: 1.4.0
```


### What type of PR is it?
Documentation

### What is the Jira issue?
no Jira issue for this

### Screenshots (if appropriate)
![screen shot 2016-11-03 at 11 37 09 
am](https://cloud.githubusercontent.com/assets/10060731/19954332/b8690e68-a1ba-11e6-9649-d7b841e832ea.png)

### 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/AhyoungRyu/zeppelin update/igniteSection

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

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


commit 28cf050b00841dba19e027bfecec674e4e1c9455
Author: AhyoungRyu 
Date:   2016-11-03T02:38:54Z

Add ignite version info in supported interpreters table




---
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 #1580: bug fixed can't load for notebook list when enabled sh...

[cloverhearts]

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/1580
  
@khalidhuseynov 
Sorry, this pr is my mistake.
i will more test and implement test case.


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

[yoonjs2]

Github user yoonjs2 commented on the issue:

https://github.com/apache/zeppelin/pull/1577
  
@AhyoungRyu Fair enough, I'd just removed a todo (L10N support) and It will 
be proposed different 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 #1577: [ZEPPELIN-1554] Update jekyll version to 3.2.1 with it...

[AhyoungRyu]

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1577
  
@yoonjs2 yes i think it would be better to handle " L10N support" in other 
PR since it's a bit different work with this PR. 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 #1583: [MINOR] Turn on authenticaiton in shiro.ini.template b...

[AhyoungRyu]

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1583
  
Yeah it makes sense. 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 #1557: ZEPPELIN-1577. LivyInterpreter should not use FIFOSche...

[Leemoonsoo]

Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1557
  
Exactly. Adding option in interpreter-setting.json and restrict mode usage 
is great idea. Similar approaches has been made in ZEPPELIN-1026 for editor 
mode restriction. 


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

[bikassaha]

Github user bikassaha commented on the issue:

https://github.com/apache/zeppelin/pull/1557
  
@zjffdu Isolated would also be valid but perhaps an overkill. So default of 
scoped makes sense.


---
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 #1514: ZEPPELIN-1516. NPE LivySparkSQLInterpreter thrown with...

[zjffdu]

Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1514
  
I am planing to add such rest api for livy, but for now livy don't have 
such api for checking version. 


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

[zjffdu]

Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1557
  
Yes, that would be better. I think currently zeppelin hard coded the 
default mode (shared mode), there's no way to set the default mode for 
interpreter, we may add one option in interpreter-setting.json.  And also we 
may add one option to restrict the mode usage as for livy only scoped is valid. 


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

[Leemoonsoo]

Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1557
  
@bikassaha Good point. I Agree, that approach seems better.


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

[asfgit]

Github user asfgit closed the pull request at:

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


---
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 #338: ZEPPELIN-335: Apache Pig interpreter

[asfgit]

Github user asfgit closed the pull request at:

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


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

[bikassaha]

Github user bikassaha commented on the issue:

https://github.com/apache/zeppelin/pull/1557
  
Perhaps for this issue and other similar issues related to multiple users, 
the mode for the livy interpreter should be scoped instead of shared. If we run 
1 interpreter per user then that matches the end to end flow wrt sessions. Then 
interpreters will have their own queue and not need this workaround. Also one 
user restarting their interpreter would not affect existing interpreter 
instances for others users. Is that a correct understanding?
Reply


---
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 #1583: [MINOR] Turn on authenticaiton in shiro.ini.template b...

[minahlee]

Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1583
  
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 #1534: ZEPPELIN-1345 - Create a custom matplotlib backend tha...

[felixcheung]

Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1534
  
very cool! could you add some documentation on how html or angular output 
can plug in to this?
would it be possible to add the same tests to Spark pyspark interpreter 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 #1534: ZEPPELIN-1345 - Create a custom matplotlib backend tha...

[agoodm]

Github user agoodm commented on the issue:

https://github.com/apache/zeppelin/pull/1534
  
@Leemoonsoo Try it now, I think it should work. The main thing I changed 
was printing the last image as HTML instead of ANGULAR (for the purpose of 
saving the image in the notebook). I also exported the old note first before 
making these changes, then deleted the old note in place of this new one.


---
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-1605) zeppelin-interpreter: don't publish shaded JAR on maven central

[Korbinian Schmid (JIRA)]

Korbinian Schmid created ZEPPELIN-1605:
--

 Summary: zeppelin-interpreter: don't publish shaded JAR on maven 
central
 Key: ZEPPELIN-1605
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1605
 Project: Zeppelin
  Issue Type: Improvement
  Components: Interpreters
Affects Versions: 0.6.2, 0.6.1
Reporter: Korbinian Schmid


The JAR on Maven central is an uber JAR containing all dependencies inside the 
JAR: 
https://mvnrepository.com/artifact/org.apache.zeppelin/zeppelin-interpreter/0.6.2

This makes dependency conflict resolution difficult for projects consuming that 
artifact. Instead, only publish the JAR without dependencies and declare the 
dependencies in the pom.xml file



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

[GitHub] zeppelin issue #1572: [ZEPPELIN-1593] Enforce sorting on dynamic form elemen...

[Leemoonsoo]

Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1572
  
Thanks @trengrj for the contribution. 
Looks good to me.

CI failure are irrelevant to this change.
Merge to master if there're no more discussions.


---
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 #1534: ZEPPELIN-1345 - Create a custom matplotlib backend tha...

[Leemoonsoo]

Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1534
  
Thanks @agoodm for great contribution.
I have tested this and it works really well on both %pyspark and %python.
Also verified `/lib/python` dir is packaged with -Pbuild-distr flag.

One thing is, with the tutorial note `/notebook/2BQA35CJZ`, Zeppelin fails 
to start with error

```
Caused by: java.lang.NullPointerException
at 
org.apache.zeppelin.notebook.Notebook.loadNoteFromRepo(Notebook.java:447)
at org.apache.zeppelin.notebook.Notebook.loadAllNotes(Notebook.java:469)
at org.apache.zeppelin.notebook.Notebook.(Notebook.java:123)
at 
org.apache.zeppelin.server.ZeppelinServer.(ZeppelinServer.java:107)
... 29 more
```

Without tutorial note `notebook/2BQA35CJZ`, Zeppelin starts without problem.
Do you experience the same problem with tutorial note?


---
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 #1582: [ZEPPELIN-1604] Add Neo4j interpreter and Netwo...

[conker84]

GitHub user conker84 opened a pull request:

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

[ZEPPELIN-1604] Add Neo4j interpreter and Network visualization

### What is this PR for?
This contribution would to introduce Neo4j Cypher intepreter and the new 
network visualization;
at the same time would provide base APIs that allow other graph databases 
(or graph framworks such as GraphX or Giraph).

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

### Todos
* [x] - Create the network visualization (using 
[Sigma.js](http://sigmajs.org/))
* [x] - Provide base APIs to manage graph results (under: 
org.apache.zeppelin.interpreter.graph.*)
* [x] - Create the Neo4j intepreter

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

### How should this be tested?
Donwload and execute [Neo4j](https://neo4j.com/download/?ref=home) v3.x, 
you can also pull a [Docker image](https://neo4j.com/developer/docker/).

In order to execute test cases, if you are running Java 7, you need to also 
provide an environment variable telling the tests where to find Java 8, because 
Neo4j-the-database needs it to run.
```bash
export NEO4J_JAVA=
```

Use this statement to create a dummy dataset
```bash
%neo4j
UNWIND range(1,100) as id
CREATE (p:Person {id:id, name: "Name " + id, age: id % 3}) WITH collect(p) 
as people
UNWIND people as p1
UNWIND range(1,10) as friend
WITH p1, people[(p1.id + friend) % size(people)] as p2
CREATE (p1)-[:KNOWS {years: abs(p2.id - p2.id)}]->(p2)
```

Then you can write some simple queries like:
```bash
%neo4j
%neo4j
MATCH (p:Person)-[r:KNOWS]-(p1:Person)
RETURN p, r, p1
LIMIT 10;
```

```bash
%neo4j
MATCH (p:Person)-[r:KNOWS]-(p1:Person)
RETURN p.id AS ID_A, p.name AS NAME_A, r.years AS YEARS, p1.id AS ID_B, 
p1.name AS NAME_B
LIMIT 20;
```
### Screenshots
![Simple cypher query network 
visualization](https://raw.githubusercontent.com/conker84/zeppelin/master/docs/assets/themes/zeppelin/img/docs-img/neo4j-network-result.png)

![Simple cypher query tabular 
visualization](https://raw.githubusercontent.com/conker84/zeppelin/master/docs/assets/themes/zeppelin/img/docs-img/neo4j-tabular-result.png)

### Video

![localhost_9000___-google-chrome-01_11_2016-15_44_24](https://cloud.githubusercontent.com/assets/185/19947893/6e83ec66-a14a-11e6-978a-c85c6929209a.gif)

### Questions:
* Does the licenses files need update? **Yes**
* Is there breaking changes for older versions? **No**
* Does this needs documentation? **Yes**. I have create the neo4j.md file 
in /docs/intepreter folder

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

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

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

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


commit 4e685ccf2bc23c41f7d48569af584ee535feb474
Author: conker84 
Date:   2016-10-26T20:07:17Z

First commit

commit acc4307dd1b7f90dedf76ecb8a4b27e93658399f
Author: conker84 
Date:   2016-10-26T21:40:47Z

Added the graph footer to provide detailed data information

commit 72f12036047e8fdd518712527380730ac139988d
Author: conker84 
Date:   2016-10-26T22:15:39Z

Code clean

commit bd1b0f23fd1b5e35e5a3c125fff50abccebc4ba3
Author: conker84 
Date:   2016-10-29T13:37:30Z

Changes:

Move the GraphResult to zeppelin-intepreter package in order to use as
base api for other graph databases.
Added neo4.md to the documentation.
Prevented the nodes overlap.

commit 251fba0a16ded784be33995ba83abe0cb23f388a
Author: conker84 
Date:   2016-10-29T13:43:02Z

Added doc images

commit bf68eb6b6ee28de7e9fa8c67c223130d7b802949
Author: conker84 
Date:   2016-10-29T13:45:27Z

doc fix

commit c8249b6c45c1b5957957ea3cac13d8fb4c17ef5d
Author: conker84 
Date:   2016-11-01T13:43:07Z

Removed Jackson Dependencies

commit d5997778c2edf9b75cef66d36ec95bf5a14fc5bb
Author: conker84 
Date:   2016-11-01T14:04:59Z

reactivate "goog-webfont-dl" task

commit 9aefddfc5a95fb469f21cec1a968cc917020dea5
Author: conker84 
Date:   2016-11-02T13:47:48Z

Added tests and changed network icon

commit 793f72922cd9acbe4a5cbf26986d4c4ceedf9df2
Author: conker84 
Date:   2016-11-02T21:07:05Z

Bugfixes




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

[GitHub] zeppelin issue #1485: Prevent from version collision on test task

[doanduyhai]

Github user doanduyhai commented on the issue:

https://github.com/apache/zeppelin/pull/1485
  
I have the same compilation issue using `mvn clean compile`. After 
excluding the same dependencies as this PR from `zeppelin-zengine/pom.xml` my 
`mvn clean compile` is successful


---
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 #1573: [DOC] Add `/ws` for nginx websocket configuration

[Leemoonsoo]

Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1573
  
Thanks @m30m for the contribution.
CI failure is irrelevant to this update.

LGTM and merge into master if there're no more discussions.


---
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 #1579: [MINOR] Remove 0.6.1 specific instruction from 0.6.2 d...

[Leemoonsoo]

Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1579
  
Thanks @AhyoungRyu for the review. I'll merge into gh-pages branch if 
there're no further discussions.


---
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-1604) Add Neo4j interpreter and Network visualization

[Andrea Santurbano (JIRA)]

Andrea Santurbano created ZEPPELIN-1604:
---

 Summary: Add Neo4j interpreter and Network visualization
 Key: ZEPPELIN-1604
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1604
 Project: Zeppelin
  Issue Type: Improvement
  Components: documentation, front-end, Interpreters
Affects Versions: 0.7.0
Reporter: Andrea Santurbano


Add support for querying Neo4j via Cypher, and provide a way to show the result 
as network.



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

[jira] [Created] (ZEPPELIN-1603) Enable users to specify Yarn queues

[Benoy Antony (JIRA)]

Benoy Antony created ZEPPELIN-1603:
--

 Summary: Enable users to specify Yarn queues
 Key: ZEPPELIN-1603
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1603
 Project: Zeppelin
  Issue Type: Improvement
Reporter: Benoy Antony


Livy allows users to specify a queue.
On Zeppelin's Livy Interpreter, one can configure _livy.spark.yarn.queue_.
But this being a global setting , all paragraphs will be run on the the single 
queue. 

In enterprises which uses queues, different users have access to different 
queues. It should be possible for the user to specify a different queue to run 
their paragraphs



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

[jira] [Created] (ZEPPELIN-1602) REST API should return proper error status code & message when user is an anonymous

[Tae Jun Kim (JIRA)]

Tae Jun Kim created ZEPPELIN-1602:
-

 Summary: REST API should return proper error status code & message 
when user is an anonymous
 Key: ZEPPELIN-1602
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1602
 Project: Zeppelin
  Issue Type: Bug
  Components: zeppelin-server
Reporter: Tae Jun Kim


This issue is related to 
[ZEPPELIN-1586|https://github.com/apache/zeppelin/pull/1567]

Currently, when a user does not have permission, REST API returns proper error 
status code *403* with some message:
{code}
{"status":"FORBIDDEN","message":"Insufficient privileges you cannot get this 
note"}
{code}

However, when a user is anonymous, it returns *405* error status code without 
any message. The error status code should be *403*, because it's {{forbidden}}, 
not {{method not allowed}}. And also the server should return some error 
message. If not, users can be confused.



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

[GitHub] zeppelin pull request #1581: Ignite versions fixed

[agura]

GitHub user agura opened a pull request:

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

Ignite versions fixed

### What is this PR for?
This PR fixes Apache Ignite version mentions in interpreter documentation.

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

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

$ git pull https://github.com/agura/incubator-zeppelin ignite-docs

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

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


commit d8710b3c200fb517469313495a11728c12da73c2
Author: agura 
Date:   2016-11-02T17:12:27Z

Ignite versions fixed




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

Re: Update Ignite interpreter related docs on the Zeppelin site

[Andrey Gura]

Thank you, Moon.

I've created PR with update: https://github.com/apache/zeppelin/pull/1581



On Wed, Nov 2, 2016 at 10:30 AM, moon soo Lee  wrote:

> Hi,
>
> All docs and website(gh-pages branch) materials of Zeppelin is in our
> source tree.
> You can make a pullrequest with update
> https://github.com/apache/zeppelin/blob/master/docs/interpreter/ignite.md
>  file.
>
> And looks like ignite interpreter is already in binary package. I think we
> can remove the quote about ignite in a future binary release.
>
> Thanks,
> moon
>
> On Tue, Nov 1, 2016 at 6:23 AM Andrey Gura  wrote:
>
> > 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 #1574: ZEPPELIN-1595. Make ZeppelinContext extensible

[felixcheung]

Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1574
  
To clarify, it doesn't right now, but I recall requests for that capability 
(which would be very useful for some of the more restricting interpreters to 
pass data around)

I will let others to 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.
---

[GitHub] zeppelin issue #1514: ZEPPELIN-1516. NPE LivySparkSQLInterpreter thrown with...

[felixcheung]

Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1514
  
Got it. Is there a way to version check Livy, like how we do Spark? Perhaps 
a PR on Livy to expose that?

Just thinking this could be fragile when moving to SparkSession and so on 
(maybe in the future)



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

[yoonjs2]

Github user yoonjs2 commented on the issue:

https://github.com/apache/zeppelin/pull/1577
  
@AhyoungRyu no, its under progress now. Should I changeunchecked todo 
to other 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 #1538: [ZEPPELIN-1386] Docker images for running Apache Zeppe...

[astroshim]

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1538
  
@mfelgamal Ping.


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

[gss2002]

Github user gss2002 commented on the issue:

https://github.com/apache/zeppelin/pull/1516
  
@zjffdu went and confirmed the previous error from the pull 19 days ago was 
in a set of tests that ran completely fine this time. So I think this patch is 
good to go


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

[gss2002]

Github user gss2002 commented on the issue:

https://github.com/apache/zeppelin/pull/1516
  
@zjffdu I re-run the tests. This error doesn't look to have anything to do 
with the code change here. Let me know your thoughts and how we want to proceed

Tests run: 9, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 196.06 sec 
<<< FAILURE! - in org.apache.zeppelin.integration.ParagraphActionsIT
testEditOnDoubleClick(org.apache.zeppelin.integration.ParagraphActionsIT)  
Time elapsed: 37.614 sec  <<< ERROR!
org.openqa.selenium.ElementNotVisibleException: Element is not currently 
visible and so may not be interacted with
Command duration or timeout: 30.08 seconds
Build info: version: '2.48.2', revision: 
'41bccdd10cf2c0560f637404c2d96164b67d9d67', time: '2015-10-09 13:08:06'
System info: host: 'testing-worker-linux-docker-cccaeb55-3455-linux-3', ip: 
'172.17.1.168', os.name: 'Linux', os.arch: 'amd64', os.version: 
'3.13.0-40-generic', java.version: '1.7.0_76'
Session ID: 461cd88c-98e9-467e-88bd-e44eab394a92
Driver info: org.openqa.selenium.firefox.FirefoxDriver
Capabilities [{platform=LINUX, acceptSslCerts=true, javascriptEnabled=true, 
cssSelectorsEnabled=true, databaseEnabled=true, browserName=firefox, 
handlesAlerts=true, nativeEvents=false, webStorageEnabled=true, 
rotatable=false, locationContextEnabled=true, applicationCacheEnabled=true, 
takesScreenshot=true, version=31.0}]
at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
at 
sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:57)
at 
sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
at java.lang.reflect.Constructor.newInstance(Constructor.java:526)
at 
org.openqa.selenium.remote.ErrorHandler.createThrowable(ErrorHandler.java:206)
at 
org.openqa.selenium.remote.ErrorHandler.throwIfResponseFailed(ErrorHandler.java:158)
at 
org.openqa.selenium.remote.RemoteWebDriver.execute(RemoteWebDriver.java:647)
at 
org.openqa.selenium.remote.RemoteWebElement.execute(RemoteWebElement.java:326)
at 
org.openqa.selenium.remote.RemoteWebElement.sendKeys(RemoteWebElement.java:121)
at 
org.apache.zeppelin.integration.ParagraphActionsIT.testEditOnDoubleClick(ParagraphActionsIT.java:443)
Caused by: org.openqa.selenium.ElementNotVisibleException: Element is not 
currently visible and so may not be interacted with
Build info: version: '2.48.2', revision: 
'41bccdd10cf2c0560f637404c2d96164b67d9d67', time: '2015-10-09 13:08:06'
System info: host: 'testing-worker-linux-docker-cccaeb55-3455-linux-3', ip: 
'172.17.1.168', os.name: 'Linux', os.arch: 'amd64', os.version: 
'3.13.0-40-generic', java.version: '1.7.0_76'


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

[gss2002]

GitHub user gss2002 reopened a pull request:

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

ZEPPELIN-1546 - Zeppelin Livy Interpreter 404 Exception not caught Kerberos 
Enabled

### What is this PR for?

ZEPPELIN-1546: Zeppelin Livy Interpreter 404 Exception not caught with 
Kerberos
Livy Interpreter gets a NestedRuntimeException when running with Kerberized 
components.
### What type of PR is it?

[Bug Fix]
### What is the Jira issue?

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

Enable Kerberos for Zeppelin to talk to Livy
"2BXS1CND2": {
  "id": "2BXS1CND2",
  "name": "livy",
  "group": "livy",
  "properties": {
"livy.spark.driver.cores": "",
"zeppelin.livy.principal": "zeppelin-clustern...@example.com",
"zeppelin.livy.keytab": 
"/etc/security/keytabs/zeppelin.server.kerberos.keytab",
### Screenshots (if appropriate)
### Questions:
- Does the licenses files need update? n
- Is there breaking changes for older versions? n
- Does this needs documentation? n


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

$ git pull https://github.com/gss2002/zeppelin ZEPPELIN-1546

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

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


commit 54d8fe43adff15097aa8075a7976d6b96cd68b3f
Author: gss2002 
Date:   2016-10-14T03:24:35Z

ZEPPELIN-1546 - Zeppelin Livy Interpreter 404 Exception not caught with
KerberosTemplate being enabled




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

[gss2002]

Github user gss2002 closed the pull request at:

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


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

[gss2002]

Github user gss2002 commented on the issue:

https://github.com/apache/zeppelin/pull/1516
  
Reopen to re-check code and 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 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-1601) How to view and capture UserName and other session Details

[Abhishek (JIRA)]

Abhishek created ZEPPELIN-1601:
--

 Summary: How to view and capture UserName and other session Details
 Key: ZEPPELIN-1601
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1601
 Project: Zeppelin
  Issue Type: Improvement
Reporter: Abhishek


I need to capture UserName and may be other session variables, username I may 
then use to provide data level security or logging. Can I do it in 0.6.1 or any 
later version ?



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

[GitHub] zeppelin issue #1578: ZEPPELIN-1599 Remove support on some old versions of S...

[minahlee]

Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1578
  
I think 
https://github.com/apache/zeppelin/blob/master/testing/downloadSpark.sh#L79 
this line should be updated 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 pull request #1567: [ZEPPELIN-1586] Add security check in NotebookR...

[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 #1580: bug fixed can't load for notebook list when ena...

[cloverhearts]

GitHub user cloverhearts opened a pull request:

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

bug fixed can't load for notebook list when enabled shiro

### What is this PR for?
A few sentences describing the overall goals of the pull request's commits.
First time? Check out the contributing guide - 
https://zeppelin.apache.org/contribution/contributions.html


### What type of PR is it?
[Bug Fix | Improvement | Feature | Documentation | Hot Fix | Refactoring]

### Todos
* [ ] - Task

### What is the Jira issue?
* Open an issue on Jira https://issues.apache.org/jira/browse/ZEPPELIN/
* Put link here, and add [ZEPPELIN-*Jira number*] in PR title, eg. 
[ZEPPELIN-533]

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

### Screenshots (if appropriate)

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



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

$ git pull https://github.com/cloverhearts/zeppelin 
selenium-test-notebooklist-bug

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

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


commit e0354e6abda0e18b77af75771a69482f9edffa37
Author: CloverHearts 
Date:   2016-11-02T10:15:40Z

bug fixed can't load for notebook list when enabled shiro




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

[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 4a43b07c05026698fe268b4b45913a3b0f007f74
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 da3415f3b3c12d72e0466a736ccaa0a3231c79ed
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 0e4cc3c974cb52b8e6e039375c4748bc89665201
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 21f9288ef7225f56c6a9d92d26209b7440d64c6a
Author: Anthony Corbacho 
Date:   2016-10-28T09:57:06Z

Replace check of aninonimous by method

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

Add webapp exception handler :)

commit 6030776cd867f7575010ee7fa99e0b0782d308dd
Author: Anthony Corbacho 
Date:   2016-10-28T11:19:54Z

Handle security check

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

Rename permission check note :: be more meaningful

commit c8c42b26a0b7403a02f90cce62336c2eb9498873
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 eacfa8eee2fde6d9fb1f3f8fc52b7b7967b6c665
Author: Anthony Corbacho 
Date:   2016-11-01T08:24:38Z

Fix typo and bad copy paste for isOwner

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

Adress review and fix typos

commit b412266b4dd73fd005a403b77acdec0599e55b3a
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 decd1e9c6c32f761a806e2367eeaa94121931631
Author: Anthony Corbacho 
Date:   2016-11-01T14:49:00Z

Simple implementation of notebook test with shiro (security)

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

Rewording

commit 49d4db9eff1fe908d9256ded67f21d53fa48e4d8
Author: Anthony 

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

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

[AhyoungRyu]

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1577
  
@yoonjs2 Seems "Prepare L10N support on documentation" is unchecked in the 
PR description. Is it done 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.
---

[GitHub] zeppelin issue #1575: ZEPPELIN-1597. Change cdh spark/hadoop dependency to a...

[anthonycorbacho]

Github user anthonycorbacho commented on the issue:

https://github.com/apache/zeppelin/pull/1575
  
@minahlee yes please merge it asap this a total war zone here


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

[minahlee]

Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1575
  
LGTM. I will merge it as hotfix if there is no more 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.
---

[GitHub] zeppelin issue #1578: ZEPPELIN-1599 Remove support on some old versions of S...

[AhyoungRyu]

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1578
  
@jongyoul Yeah thanks for your consideration. Seems there is no conflict 
between this patch and #1339 i think. 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 #1576: ZEPPELIN-415 document for Apache Kylin Interpreter

[AhyoungRyu]

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1576
  
@janzhongi Thanks for your contribution. 
While I'm quickly looking through the docs, it doesn't have `Default Value` 
in the conf table. It would be better to provide it for each property i think. 

```

  
Name
Default
Description
  
  
kylin.api.url 
http://localhost:7070/kylin/api/query
kylin query POST API  The format can be like 
http://host:port/kylin/api/query
  
  
kylin.api.user
ADMIN
kylin user
  
  
kylin.api.password
KYLIN
kylin password
  
  
kylin.query.project
default
String, Project to perform query.
  
  
kylin.query.ispartial
true
true|false  (@Deprecated since Apache Kylin V1.5)  
Whether accept a partial result or not, default be “false”. Set to 
“false” for production use.
  
  
kylin.query.limit
5000
int, Query limit  If limit is set in sql, perPage will be 
ignored.
  
  
kylin.query.offset
0
int, Query offset  If offset is set in sql, curIndex will be 
ignored.
  

```

You can just copy & paste the above table if you want. I added some 
markdown escaping for `<` and `>`.


---
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 #1579: [MINOR] Remove 0.6.1 specific instruction from 0.6.2 d...

[AhyoungRyu]

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1579
  
@Leemoonsoo 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 pull request #1579: [MINOR] Remove 0.6.1 specific instruction from ...

[Leemoonsoo]

GitHub user Leemoonsoo opened a pull request:

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

[MINOR] Remove 0.6.1 specific instruction from 0.6.2 download section to 
avoid confusion.

### What is this PR for?
There're 0.6.1 specific instruction in the 0.6.2 download section. which is 
bit confusing.


### What type of PR is it?
Improvement

### Todos
* [x] - Move confusing instruction to 0.6.1 download section

### What is the Jira issue?

### How should this be tested?

### Screenshots (if appropriate)

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

…d confusion

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

$ git pull https://github.com/Leemoonsoo/zeppelin 
remove_061_specific_description_from_062_download_section

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

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


commit f3d59282377ebe5d942984d5316842f0ed8efcbe
Author: Lee moon soo 
Date:   2016-11-02T05:59:35Z

Remove 0.6.1 specific instruction from 0.6.2 download section to avoid 
confusion




---
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-1600) Integration with Siteminder with Single Sign On

[Abhishek (JIRA)]

Abhishek created ZEPPELIN-1600:
--

 Summary: Integration with Siteminder with Single Sign On
 Key: ZEPPELIN-1600
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1600
 Project: Zeppelin
  Issue Type: Test
Reporter: Abhishek


In my enterprise we use Siteminder Single Sign On for user authentication 
across all applications. We are looking to use Zeppelin for specific set of 
users for their analytical needs in Production, can you suggest how Zeppelin be 
integrated with Siteminder Single Sign On ? 



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

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

[zjffdu]

Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1574
  
Do you mean ZeppelinContext would support communication between interpreter 
groups across JVM ? Currently ZeppelinContext only support spark interpreter, I 
don't see any case that it support communication between interpreter group. 
Maybe I miss something.


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