[GitHub] zeppelin issue #1041: Get note revision of note - git repo

2016-06-17 Thread onkarshedge
Github user onkarshedge commented on the issue:

https://github.com/apache/zeppelin/pull/1041
  
@khalidhuseynov  will this be merged soon ? Currently I have extended the 
`Rev` class for my Ipfs Revision. If this is merged  I won't have to extend.

```java
private class IpfsRevision extends NotebookRepoVersioned.Rev {
private String commitMessage;

IpfsRevision(String name, int time, String message) {
  super(name, time);
  commitMessage = message;
}
```


---
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 #1042: [WIP] bin/install-interpreter.sh for netinst pa...

2016-06-17 Thread Leemoonsoo
GitHub user Leemoonsoo opened a pull request:

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

[WIP] bin/install-interpreter.sh for netinst package

### What is this PR for?
This is proof of the concept implementation of bin/install-interpreter.sh 
for netinst package which suggested in the 
[discussion](http://apache-zeppelin-users-incubating-mailing-list.75479.x6.nabble.com/Ask-opinion-regarding-0-6-0-release-package-tp3298p3314.html).

Some usages will be

```
# download all interpreters provided by Apache Zeppelin project
bin/install-interpreter.sh --all

# download an interpreter with name (for example markdown interpreter)
bin/install-interpreter.sh --name md

# download an (3rd party) interpreter with specific maven artifact name
bin/install-interpreter.sh --name md -t 
org.apache.zeppelin:zeppelin-markdown:0.6.0-SNAPSHOT
```

If it looks fine, i'll continue the work (refactor code, and add test)

### What type of PR is it?
Feature

### Todos
* [x] - working implementation
* [ ] - refactor
* [ ] - add test

### 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/Leemoonsoo/zeppelin netinst

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

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


commit efdef3d293472930776405dbeeb8e092f26981ae
Author: Lee moon soo 
Date:   2016-06-18T03:32:59Z

Initial implementation of install-interpreter.sh




---
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 #458: [WIP] Fix notebook clone bug

2016-06-17 Thread minahlee
Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/458
  
Duplicated with #933


---
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 #458: [WIP] Fix notebook clone bug

2016-06-17 Thread minahlee
Github user minahlee closed the pull request at:

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


---
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 #966: [ZEPPELIN-958] Support syntax highlight for python and ...

2016-06-17 Thread felixcheung
Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/966
  
That was only to map the list of interpreter to the language though.

To figure out which interpreter is active in the interpreter right now we 
have regex to match "%something" at the beginning which won't work for the 
default interpreter at all.



---
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 #1041: Get note revision of note - git repo

2016-06-17 Thread khalidhuseynov
GitHub user khalidhuseynov opened a pull request:

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

Get note revision of note - git repo

### What is this PR for?
This PR implements the backend of git storage layer for `get(noteId, 
revision)`, so that you can get the note of certain revision. It doesn't 
provide frontend api yet, which is future work.

### What type of PR is it?
Improvement

### Todos
* [ ] - implement `get` of GitNotebookRepo
* [ ] - add tests


### What is the Jira issue?
N/A

### How should this be tested?
two tests for `get` function should pass

### 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/khalidhuseynov/incubator-zeppelin 
feat/git-get-note-revision-of-note

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

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


commit c91dff08c255efd69b9fde5068c36b7a3ad74dd2
Author: Khalid Huseynov 
Date:   2016-06-16T23:53:58Z

change field name: revId -> id

commit 01d16463048fb65551e716af2da540d109347f1f
Author: Khalid Huseynov 
Date:   2016-06-16T23:55:07Z

basic impl of get specific version of note

commit d71b4036069422a70bdd2242b7bf3755182d7bd6
Author: Khalid Huseynov 
Date:   2016-06-17T00:23:32Z

add edge case fail 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 #747: [ZEPPELIN-605][WIP] Add support for Scala 2.11

2016-06-17 Thread lresende
Github user lresende commented on the issue:

https://github.com/apache/zeppelin/pull/747
  
Ok, looks like the hanging I was seeing was mostly a local issue in my 
Spark conf and is now fixed. 

I have Zeppelin and Spark built with Scala 2.11, and the tutorial notebook 
works well, I can also update the interpreter properties and that works well as 
well. Export is working, but Importing as mentioned by @chbrown just don't do 
anything. I will try with master and see if this is an issue specific to the 
scala 2.11 branch or not.

Btw,  I am not sure if the CI build these days is helping or not, as it's 
very hard to trust that failures are related to changes.


---
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 #1040: [ZEPPELIN-1031] Update Elasticsearch to release...

2016-06-17 Thread lresende
GitHub user lresende opened a pull request:

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

[ZEPPELIN-1031] Update Elasticsearch to release 2.3.3

### What is this PR for?
Update ElasticSearch interpreter to use ElasticSearch 2.3.3

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

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

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

$ git pull https://github.com/lresende/incubator-zeppelin elasticsearch

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

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


commit 67cc8539235488e02f05ddc0b782fa0f4addc0a9
Author: Luciano Resende 
Date:   2016-06-17T17:58:09Z

[ZEPPELIN-1031] Update Elasticsearch to release 2.3.3




---
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 #1039: [ZEPPELIN-1032] Update Apache Flink to 1.0.3 re...

2016-06-17 Thread lresende
GitHub user lresende opened a pull request:

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

[ZEPPELIN-1032] Update Apache Flink to 1.0.3 release

### What is this PR for?
Update Flink interpreter to use most recent Apache Flink 1.0.3 release


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

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

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

$ git pull https://github.com/lresende/incubator-zeppelin flink

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

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


commit 0d55fd774dc29636752bbd5e7b1812e991651a71
Author: Luciano Resende 
Date:   2016-06-17T17:59:38Z

[ZEPPELIN-1032] Update Apache Flink to 1.0.3 release




---
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 #928: [ZEPPELIN-116] Add Mahout Support for Spark Interpreter

2016-06-17 Thread rawkintrevo
Github user rawkintrevo commented on the issue:

https://github.com/apache/zeppelin/pull/928
  
Lot's of connection test failures when building Zeppelin-Server?


---
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 #1028: [Zeppelin-1022] Apply new mechanism to LivyInte...

2016-06-17 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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-1032) Update Apache Flink to 1.0.3 release

2016-06-17 Thread Luciano Resende (JIRA)
Luciano Resende created ZEPPELIN-1032:
-

 Summary: Update Apache Flink to 1.0.3 release
 Key: ZEPPELIN-1032
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1032
 Project: Zeppelin
  Issue Type: Bug
Reporter: Luciano Resende


Update to latest Flink release



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


[GitHub] zeppelin issue #928: [ZEPPELIN-116][WIP] Add Mahout Support for Spark Interp...

2016-06-17 Thread rawkintrevo
Github user rawkintrevo commented on the issue:

https://github.com/apache/zeppelin/pull/928
  
@bzz and @Leemoonsoo 

A big part of the refactor was introducing no new dependencies- instead 
loading from maven or MAHOUT_HOME at interpretter start up via dependency 
resolver.

This was done to get around version mismatches.

Here is output from `mvn dependency:tree`  I'm not super familiar, so I'm 
not 100% sure there are nothing new, but I am like 97% sure. 
```
[INFO] --- maven-dependency-plugin:2.8:tree (default-cli) @ zeppelin-mahout 
---
[INFO] org.apache.zeppelin:zeppelin-mahout:jar:0.6.0-SNAPSHOT
[INFO] +- org.slf4j:slf4j-api:jar:1.7.10:compile
[INFO] +- org.slf4j:slf4j-log4j12:jar:1.7.10:compile
[INFO] |  \- log4j:log4j:jar:1.2.17:compile
[INFO] +- 
org.apache.zeppelin:zeppelin-interpreter:jar:0.6.0-SNAPSHOT:provided
[INFO] +- org.apache.zeppelin:zeppelin-spark:jar:0.6.0-SNAPSHOT:compile
[INFO] +- 
org.apache.zeppelin:zeppelin-spark-dependencies:jar:0.6.0-SNAPSHOT:compile
[INFO] +- com.google.code.gson:gson:jar:2.2:compile
[INFO] +- org.datanucleus:datanucleus-core:jar:3.2.10:compile
[INFO] +- org.datanucleus:datanucleus-api-jdo:jar:3.2.6:compile
[INFO] +- org.datanucleus:datanucleus-rdbms:jar:3.2.9:compile
[INFO] +- org.scala-lang:scala-library:jar:2.10.4:compile
[INFO] +- org.scala-lang:scala-compiler:jar:2.10.4:compile
[INFO] +- org.scala-lang:scala-reflect:jar:2.10.4:compile
[INFO] \- junit:junit:jar:4.11:test
[INFO]\- org.hamcrest:hamcrest-core:jar:1.3: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 pull request #1037: fix typo the description of interpreter menu.

2016-06-17 Thread astroshim
GitHub user astroshim opened a pull request:

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

fix typo the description of interpreter menu.

### What is this PR for?
This PR is for fixing typo of the interpreter menu description.


### What type of PR is it?
Bug Fix


### Screenshots (if appropriate)
  - before
 
![image](https://cloud.githubusercontent.com/assets/3348133/16156875/1dba142e-34f1-11e6-937b-c3d2c75a6c42.png)


  - after

![image](https://cloud.githubusercontent.com/assets/3348133/16156901/365979a2-34f1-11e6-91a7-4c99b78a1d86.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/astroshim/zeppelin feat/typoInterpreters

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

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


commit 64f1f608a38f42eadbee202f037a4d3c5ac6189d
Author: astroshim 
Date:   2016-06-17T16:04:24Z

fix typo description of interpreter menu.




---
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 #1025: ZEPPELIN-964 ] Jobmanagement - (2) basic front-end

2016-06-17 Thread astroshim
Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1025
  
@cloverhearts Thanks for the quick response. I understand. :+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 #1025: ZEPPELIN-964 ] Jobmanagement - (2) basic front-end

2016-06-17 Thread cloverhearts
Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/1025
  
@astroshim 
check to sub-task section.
https://issues.apache.org/jira/browse/ZEPPELIN-531
Thank you.


---
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 #1034: [ZEPPELIN-1009] [HOTFIX] Fix Selenium test error

2016-06-17 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1034
  
I'm getting different error ...

```
Results :

Tests in error: 
  
ZeppelinIT.testAngularDisplay:151->AbstractZeppelinIT.waitForParagraph:84->AbstractZeppelinIT.pollingWait:110
 » Timeout
```



---
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-1030) Add example settings for Hive, Phoenix, Tajo

2016-06-17 Thread Jongyoul Lee (JIRA)
Jongyoul Lee created ZEPPELIN-1030:
--

 Summary: Add example settings for Hive, Phoenix, Tajo
 Key: ZEPPELIN-1030
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1030
 Project: Zeppelin
  Issue Type: Sub-task
  Components: zeppelin-interpreter
Reporter: Jongyoul Lee






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


[GitHub] zeppelin issue #1028: [Zeppelin-1022] Apply new mechanism to LivyInterpreter

2016-06-17 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/1028
  
Will merge this if 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 #1027: [ZEPPELIN-1020] Remove suggestion list on click

2016-06-17 Thread ravicodder
Github user ravicodder commented on the issue:

https://github.com/apache/zeppelin/pull/1027
  
@suvam97 Thanks for the contribution!! 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 #1005: ZEPPELIN-995 Change scheduler for JDBC interpreter to ...

2016-06-17 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1005
  
Merging it into master and branch-0.6


---
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 #806: [ZEPPELIN-707]Automatically adds %.* of previous paragr...

2016-06-17 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/806
  
@mwkang I have one more question. See below.
https://cloud.githubusercontent.com/assets/3612566/16153092/d3b8e618-34df-11e6-9649-3540c11f86cd.png;>

Can you fix the case of wrong interpreter name? Or do you have any idea to 
deal with?



---
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 #1036: [MINOR] Add .Rhistory in .gitignore

2016-06-17 Thread astroshim
Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1036
  
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 #1032: ZEPPELIN-1027: Python - add basic matplotlib example n...

2016-06-17 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/1032
  
Good one, we should have it. 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 #1036: [MINOR] Add .Rhistory in .gitignore

2016-06-17 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue:

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


[jira] [Created] (ZEPPELIN-1029) Configuration REST API

2016-06-17 Thread JIRA
Kévin Giroux created ZEPPELIN-1029:
--

 Summary: Configuration REST API
 Key: ZEPPELIN-1029
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1029
 Project: Zeppelin
  Issue Type: Improvement
  Components: zeppelin-server
Affects Versions: 0.5.6
 Environment: Zeppelin Server
Reporter: Kévin Giroux
 Fix For: 0.6.1


The Zeppelin Server provide a lot of REST API for creating, cloning, editing, 
getting and deleting notebook and paragraph, 

However, it is still impossible to change the configuration, as for example, 
the type of a graphic, or data that need to be displayed. 

It would be very usefull for user that doesn't have access to websocket or 
can't manage modification easily. 





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


[GitHub] zeppelin issue #1027: [ZEPPELIN-1020] Remove suggestion list on click

2016-06-17 Thread suvam97
Github user suvam97 commented on the issue:

https://github.com/apache/zeppelin/pull/1027
  
@prabhjyotsingh @r-kamath Thanks for the review!
@prabhjyotsingh done required changes.


---
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 #1027: [ZEPPELIN-1020] Remove suggestion list on click

2016-06-17 Thread r-kamath
Github user r-kamath commented on the issue:

https://github.com/apache/zeppelin/pull/1027
  
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 #1027: [ZEPPELIN-1020] Remove suggestion list on click

2016-06-17 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/1027
  
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 #1034: [ZEPPELIN-1009] [HOTFIX] Fix Selenium test error

2016-06-17 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1034
  
Looks great to me


---
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 #1028: [Zeppelin-1022] Apply new mechanism to LivyInterpreter

2016-06-17 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/1028
  
@bzz I was referring to https://github.com/apache/zeppelin/pull/1034, 
ZEPPELIN-1009. 

```
Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 104.524 sec 
- in org.apache.zeppelin.integration.ZeppelinIT

Results :

Tests in error: 
  ParagraphActionsIT.testRemoveButton:153 » NoSuchElement Unable to 
locate eleme...
  ParagraphActionsIT.testCreateNewButton:86 » NoSuchElement Unable to 
locate ele...
```


@jongyoul  Thank you for testing 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.
---


[GitHub] zeppelin issue #806: [ZEPPELIN-707]Automatically adds %.* of previous paragr...

2016-06-17 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/806
  
@mwkang Those tests are enough. I've tested it and LGTM. Thanks for 
contribution.


---
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 #986: [Zeppelin 946] Permissions not honoring group

2016-06-17 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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 #1035: [MINOR] Changed guava version to 19.0 for selen...

2016-06-17 Thread jongyoul
GitHub user jongyoul opened a pull request:

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

[MINOR] Changed guava version to 19.0 for selenium test

### What is this PR for?
Enable selenium test with SafariDriver.

### What type of PR is it?
[Bug Fix]

### Todos
* [x] - Upgrade guava version

### What is the Jira issue?
N/A

### How should this be tested?
1. Install selenium driver for Safari
1. Make sure that you don't have firefox installed
1. Start Zeppelin without this patch
1. run a selenium test. `TEST_SELENIUM="true" mvn package 
-DfailIfNoTests=false -pl 
'zeppelin-interpreter,zeppelin-zengine,zeppelin-server' 
-Dtest=ParagraphActionsIT`
1. Got error
1. Repeat after patch
1. Success

### Screenshots (if appropriate)
Got error like:
```
java.lang.NoSuchMethodError: 
com.google.common.util.concurrent.MoreExecutors.directExecutor()Ljava/util/concurrent/Executor;
at 
org.openqa.selenium.safari.WebSocketConnection.send(WebSocketConnection.java:139)
at 
org.openqa.selenium.safari.SafariDriverCommandExecutor.execute(SafariDriverCommandExecutor.java:188)
at 
org.openqa.selenium.remote.RemoteWebDriver.execute(RemoteWebDriver.java:620)
at 
org.openqa.selenium.remote.RemoteWebDriver.startSession(RemoteWebDriver.java:247)
at 
org.openqa.selenium.remote.RemoteWebDriver.(RemoteWebDriver.java:129)
at 
org.openqa.selenium.remote.RemoteWebDriver.(RemoteWebDriver.java:142)
at org.openqa.selenium.safari.SafariDriver.(SafariDriver.java:62)
at org.openqa.selenium.safari.SafariDriver.(SafariDriver.java:42)
at 
org.apache.zeppelin.WebDriverManager.getWebDriver(WebDriverManager.java:103)
at 
org.apache.zeppelin.integration.ParagraphActionsIT.startUp(ParagraphActionsIT.java:51)
```

### 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/jongyoul/zeppelin 
minor-upgrade-guava-for-selenium-with-safaridriver

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

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


commit 6f932789d84fb313d2865067d946dc51e9fd3d1d
Author: Jongyoul Lee 
Date:   2016-06-17T08:33:49Z

Changed guava version to 19.0




---
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 #746: [ZEPPELIN-699] Add new synchronous paragraph run...

2016-06-17 Thread doanduyhai
GitHub user doanduyhai reopened a pull request:

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

[ZEPPELIN-699] Add new synchronous paragraph run REST API

### What is this PR for?
Right now, when calling the REST API 
`http://:/api/notebook/job//` Zeppelin always 
returns **OK** as shown by this source code: 
https://github.com/apache/incubator-zeppelin/blob/master/zeppelin-server/src/main/java/org/apache/zeppelin/rest/NotebookRestApi.java#L477

This ticket will update the behavior so that Zeppelin also return the 
result of the paragraph execution

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

### Todos
* [ ] - Code Review
* [ ] - Simple Test

### Is there a relevant Jira issue?
**[ZEPPELIN-699]**

### How should this be tested?
* `git fetch origin pull/746/head:ParagraphExecutionRESTAPI`
* `git checkout ParagraphExecutionRESTAPI`
* `mvn clean package -DskipTests`
* `bin/zeppelin-daemon.sh restart`
* Create a new note
* In the first paragraph, put the following code

```scala
%sh

echo "Current time = "`date +"%T"
```

* Retrieve the current note id in the URL
* Retrieve the current paragraph id
* Use a REST Client like **[POSTman]** to create a HTTP POST query 
`http://:/api/notebook/run//` 
* You should receive something similar as follow for answer

```
{
"status": "OK",
"body": {
"code": "SUCCESS",
"type": "TEXT",
"msg": "Current time = 16:14:18\n"
}
}
```

### Screenshots (if appropriate)

![zeppelin_synchronous_rest_api](https://cloud.githubusercontent.com/assets/1532977/15748069/b4a26a46-28dd-11e6-8f51-aa13ddba3f1c.gif)

API Documentation update

**Existing asynchronous API**

![image](https://cloud.githubusercontent.com/assets/1532977/15773274/5b508cae-2976-11e6-9e52-14d8b7e7828e.png)

**New synchronous API**

![image](https://cloud.githubusercontent.com/assets/1532977/15773309/84965a94-2976-11e6-9719-81d8b555c3c4.png)


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

[ZEPPELIN-699]: https://issues.apache.org/jira/browse/ZEPPELIN-699
[POSTman]: https://www.getpostman.com/

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

$ git pull https://github.com/doanduyhai/incubator-zeppelin ZEPPELIN-699

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

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


commit 090d9de360b2130d2fee135949ef31a35642934f
Author: DuyHai DOAN 
Date:   2016-06-02T14:15:46Z

[ZEPPELIN-699] Add new synchronous paragraph run REST API

commit faf9542feee910cb4d584b4b49927136a3f839f7
Author: DuyHai DOAN 
Date:   2016-06-02T14:24:38Z

[ZEPPELIN-699] Update Notebook REST API documentation

commit 0106cf5735912467ee36aaf37edefd617548863e
Author: DuyHai DOAN 
Date:   2016-06-03T08:20:17Z

[ZEPPELIN-699] Fix doc after code 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 #746: [ZEPPELIN-699] Add new synchronous paragraph run...

2016-06-17 Thread doanduyhai
Github user doanduyhai closed the pull request at:

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


---
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 #998: Authenticated user aware notebook storage layer

2016-06-17 Thread vgmartinez
Github user vgmartinez commented on the issue:

https://github.com/apache/zeppelin/pull/998
  
LGTM...I have tested and works well for me.


---
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 #1024: Add force layout visualization in paragraphs for SQL q...

2016-06-17 Thread davidtzoor
Github user davidtzoor commented on the issue:

https://github.com/apache/zeppelin/pull/1024
  
Sorry for the mess. I will create a new branch with the commits and reopen 
a new pull request. Thank you for your 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 issue #1019: [ZEPPELIN-1015] Cron job fails to run a paragraph when...

2016-06-17 Thread jmrr
Github user jmrr commented on the issue:

https://github.com/apache/zeppelin/pull/1019
  
LGTM too, thanks @Leemoonsoo I've also tried with same interpreter and also 
works


---
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 #1013: [ZEPPELIN-1002] Move configuration menus under dropdow...

2016-06-17 Thread minahlee
Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1013
  
I also don't feel the necessity of showing full user name since logged user 
would know his own username. Let me remove username under dropdown 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 #1013: [ZEPPELIN-1002] Move configuration menus under dropdow...

2016-06-17 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/1013
  
@minahlee Actually, the long name would normally be truncated by CSS 
instead of javascript, so it wouldn't be good. (Since it was done elsewhere, no 
need to fix it here)

Is it necessary to show long name though? Maybe this can be handled in a 
different PR, so that we can think of other way of doing it, and not block this 
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.
---


[GitHub] zeppelin issue #998: Authenticated user aware notebook storage layer

2016-06-17 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue:

https://github.com/apache/zeppelin/pull/998
  
rebased from master


---
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 #1028: [Zeppelin-1022] Apply new mechanism to LivyInterpreter

2016-06-17 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1028
  
I've tested it. 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 #1034: [ZEPPELIN-1009] [HOTFIX] Fix Selenium test error

2016-06-17 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/1034
  
@Leemoonsoo Previous errors are gone, nice find


---
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 #1033: Python: fix for 'run all' paragraphs

2016-06-17 Thread felixcheung
Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1033
  
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 #1034: [ZEPPELIN-1009] [HOTFIX] Fix Selenium test erro...

2016-06-17 Thread Leemoonsoo
GitHub user Leemoonsoo reopened a pull request:

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

[ZEPPELIN-1009] [HOTFIX] Fix Selenium test error

### What is this PR for?
Fix selenium test error described in 
[ZEPPELIN-1009](https://issues.apache.org/jira/browse/ZEPPELIN-1009)

### What type of PR is it?
Hot Fix

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

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

### 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/Leemoonsoo/zeppelin ZEPPELIN-1009

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

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


commit 13dee87619c11ab8770ad81ea2693a41840d4192
Author: Lee moon soo 
Date:   2016-06-17T06:06:57Z

fix




---
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 #1034: [ZEPPELIN-1009] [HOTFIX] Fix Selenium test erro...

2016-06-17 Thread Leemoonsoo
Github user Leemoonsoo closed the pull request at:

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


---
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 #1032: ZEPPELIN-1027: Python - add basic matplotlib example n...

2016-06-17 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1032
  
Yeah  👍  for @minahlee. And it looks good! :)


---
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 #1032: ZEPPELIN-1027: Python - add basic matplotlib example n...

2016-06-17 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1032
  
Makes perfect sense to me


---
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 #1033: Python: fix for 'run all' paragraphs

2016-06-17 Thread bzz
GitHub user bzz opened a pull request:

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

Python: fix for 'run all' paragraphs

### What is this PR for?
Switch to FIFO scheduler as in current implementation `.interpret()` is not 
thread-safe and so in parallel one 'Run All' fails some paragraphs with NPE in 
logs

### What type of PR is it?
Bug Fix | Improvement

### How should this be tested?
'Run All' passes without NPE in logs i.e on this [Zeppelin notebook for 
python](https://www.zeppelinhub.com/viewer/notebooks/aHR0cHM6Ly9yYXcuZ2l0aHVidXNlcmNvbnRlbnQuY29tL2J6ei9pbmN1YmF0b3ItemVwcGVsaW4vMTkyZjU3YjZjMGZkMjc4NzgwZDI3NDAzMGY1YmJlOTZlZThkNzdiYi9ub3RlYm9vay8yQlFBMzVDSlovbm90ZS5qc29u)
### 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/bzz/incubator-zeppelin fix/python-run-all

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

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


commit 72e9d62406b8525ac81deef7a04165be03441e02
Author: Alexander Bezzubov 
Date:   2016-06-17T04:59:50Z

Python: switch to FIFO scheduler

In current implementation `.interpret()` is not thread-safe and so in
parallel one (esp on empty or graph paragraphs) 'Run All' fails with NPE in 
logs




---
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 #1032: ZEPPELIN-1027: Python - add basic matplotlib example n...

2016-06-17 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1032
  
I'm planning to add at least one more tutorial notebook of using python, 
something similar to what we have for Apache Spark but with pandas and 
matplotlib etc.

Do you guys think advanced one should be incorporated in here - :confused:, 
or shall we have a separate one 😄  , say title "Zeppelin Tutorial: Python - 
pandas, matplotlib".



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