[GitHub] zeppelin pull request #1441: [ ZEPPELIN-967 ] Implement search bar in job ma...

2016-09-27 Thread cloverhearts
GitHub user cloverhearts reopened a pull request:

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

[ ZEPPELIN-967 ] Implement search bar in job manager

### 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?
Feature

### Todos
- [x] add serach bar ui
- [x] implement filter
- [x] show filter loading status.

### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-967
### How should this be tested?
1. click to jobmanager menu
2. try to use filter by jobmanager filter bar on top.

Enter, Enter, it has been filtered properly?
After inputting, wait about 2 seconds, it has been filtering?

### Screenshots (if appropriate)
https://cloud.githubusercontent.com/assets/10525473/18658644/d6359a62-7f3f-11e6-8c03-030fc107ec2d.gif;>Screenshots
### 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/cloverhearts/zeppelin 
ZEPPELIN-FILTERBAR-JOBMANAGER

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

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


commit 01b9828e369bcdb4ca55b30f63269b2a76a8e2cc
Author: CloverHearts 
Date:   2016-09-20T05:27:08Z

Implement search bar in job manager

commit 8ddffd02e88b7f9f1874947f1b462afee848ab5b
Author: CloverHearts 
Date:   2016-09-22T05:47:36Z

Merge branch 'master' into ZEPPELIN-FILTERBAR-JOBMANAGER

commit 76e0fb0111c35580f28c4ff22dfe2954df134a67
Author: CloverHearts 
Date:   2016-09-28T04:52:12Z

Merge branch 'master' into ZEPPELIN-FILTERBAR-JOBMANAGER




---
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 #1441: [ ZEPPELIN-967 ] Implement search bar in job ma...

2016-09-27 Thread cloverhearts
Github user cloverhearts closed the pull request at:

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


---
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 #1441: [ ZEPPELIN-967 ] Implement search bar in job manager

2016-09-27 Thread cloverhearts
Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/1441
  
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 #1430: [ZEPPELIN-868] Notebook import fails when notebook is ...

2016-09-27 Thread vensant
Github user vensant commented on the issue:

https://github.com/apache/zeppelin/pull/1430
  
Reopening the pull request as the build failed


---
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 #1430: [ZEPPELIN-868] Notebook import fails when noteb...

2016-09-27 Thread vensant
Github user vensant closed the pull request at:

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


---
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 #1430: [ZEPPELIN-868] Notebook import fails when noteb...

2016-09-27 Thread vensant
GitHub user vensant reopened a pull request:

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

[ZEPPELIN-868] Notebook import fails when notebook is large

### What is this PR for?
A bug fix: Added validation in the note import dialog box to check for the 
uploaded json file size and throw error report if the file size exceeds 1MB, as 
the websocket frame is not able to send json file of size over 1MB.

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

### Todos
NA

### What is the Jira issue?

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

### How should this be tested?
1. Deploy Zeppelin and click on 'Import Note' in the Welcome to Zeppelin 
page.
2. Click 'Choose a JSON here' and upload a json file whose file size is 
over 1MB

### Screenshots (if appropriate)

![1](https://cloud.githubusercontent.com/assets/12062069/18556899/7add7a12-7b8a-11e6-85e6-f8b4fcef2195.png)

![2](https://cloud.githubusercontent.com/assets/12062069/18556966/b10bcb0c-7b8a-11e6-9448-1381d8e05d8f.png)

![3](https://cloud.githubusercontent.com/assets/12062069/18556999/d166b8a8-7b8a-11e6-927b-caa3a56618d1.png)

![4](https://cloud.githubusercontent.com/assets/12062069/18557002/d58f8d74-7b8a-11e6-8955-710eb093a795.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/vensant/zeppelin ZEPPELIN-868

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

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


commit 5a1f6ae88349644be0d2e2b6b1ad63607fe984ed
Author: vensant 
Date:   2016-09-15T14:38:47Z

Added validation in the note import dialog box to check for the uploaded 
json file size as the websocket frame is not able to send file of size over 1MB

commit 2c5ba4a35ee91bd41b28c00dafc675cde3a5940c
Author: vensant 
Date:   2016-09-16T06:06:30Z

Rectified the errors due to grunt build no color failure

commit 55c934973120701e27160eb0bb1812496ed9b82c
Author: vensant 
Date:   2016-09-22T06:29:58Z

made changes for reading the max limit from the configuration list and 
dynamically showing it in UI

commit 23c05ed911da25f612cd9d01c45960171bd85f07
Author: vensant 
Date:   2016-09-27T06:37:12Z

client level validation added for import file size check




---
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 #1445: [Zeppelin-1001] Take care of comma/tab escape in csv/t...

2016-09-27 Thread meenakshisekar
Github user meenakshisekar commented on the issue:

https://github.com/apache/zeppelin/pull/1445
  
Re opening the pull request to run CI build.


---
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 #1445: [Zeppelin-1001] Take care of comma/tab escape i...

2016-09-27 Thread meenakshisekar
GitHub user meenakshisekar reopened a pull request:

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

[Zeppelin-1001]  Take care of comma/tab escape in csv/tsv download

### What is this PR for?
When the data is downloaded as CSV/TSV, the comma/tab in the actual data 
has to be handled so that they come exactly as the same data when downloaded.

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

### Todos
* [ ] - Task

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

### How should this be tested?
Modify the data to be loaded to have a comma.
Create a paragraph to pull up that data and display.
Now click on the download as CSV/TSV button in the tool bar.
Once the data is downloaded verify whether the original data is unaltered 
and the comma is escaped.

### Screenshots (if appropriate)

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



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

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

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

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


commit 73b1868dbf60d2ee1fb90bd0f168a64858d3593b
Author: meenakshisekar 
Date:   2016-09-19T11:16:12Z

Zeppelin-1001 Modified the data with comma/tab to be surronded by double 
quotes so that they are escaped.




---
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 #1445: [Zeppelin-1001] Take care of comma/tab escape i...

2016-09-27 Thread meenakshisekar
Github user meenakshisekar closed the pull request at:

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


---
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 #1445: [Zeppelin-1001] Take care of comma/tab escape in csv/t...

2016-09-27 Thread meenakshisekar
Github user meenakshisekar commented on the issue:

https://github.com/apache/zeppelin/pull/1445
  
@corneadoug 

Based on your comments i shall proceed further.

Thanks,
Meenakshi


---
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 #1465: [ZEPPELIN-1500] pyspark completion doesn't work.

2016-09-27 Thread felixcheung
Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1465
  
do we have tests for completion?


---
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 #1461: [MINOR] Rearrange the contents in "About Zeppelin"

2016-09-27 Thread felixcheung
Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1461
  
Hmmm I don't think anyone has reported this being an issue so I think we 
are good.
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 #1449: [MINOR] Change Markdown class name from Markdow...

2016-09-27 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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 #1436: ZEPPELIN-1452: Include Phoenix 'thin client' in...

2016-09-27 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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 #1462: ZEPPELIN-1477. Add Integration Test for LivyInterprete...

2016-09-27 Thread zjffdu
Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1462
  
I am afraid this is the only workaround for now. Because these jars are not 
available in maven 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 pull request #1466: Add SSL Cassandra support

2016-09-27 Thread robertdkmarshall
GitHub user robertdkmarshall opened a pull request:

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

Add SSL Cassandra support

### What is this PR for?
The Cassandra Interpreter does not support talking to clusters that use 
SSL/client to node encryption. It does not have the properties needed to 
configure the SSL Context. This PR adds the properties to the driver config and 
sets up the ssl options when they are requested.


### What type of PR is it?
Improvement

### Todos
* [ ] - Task

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

### How should this be tested?
Using CQL interpreter connect to a Cassandra cluster that uses client to 
node encryption. 

### Screenshots (if appropriate)
N/A

### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this need documentation?  Yes. Should add the new properties to the 
list (cassandra.ssl.enabled, cassandra.ssl.truststore.path and 
cassandra.ssl.truststore.password).


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

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

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

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


commit df82ecc8a2d181d9f92dca81357b4400a8c7ba58
Author: Robert Marshall 
Date:   2016-09-18T23:41:23Z

Add SSL Cassandra support




---
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 #1463: [ZEPPELIN-1495] Add Good Practice #4

2016-09-27 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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 #1408: [ZEPPELIN-808] Changed Permissions on Notebooks...

2016-09-27 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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 #1461: [MINOR] Rearrange the contents in "About Zeppelin"

2016-09-27 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1461
  
@felixcheung AFAIK since this `Patua One` is from 
[here](https://github.com/apache/zeppelin/blob/848dbd0302a4e6d4917a70c1cbc2220a7a814488/zeppelin-web/Gruntfile.js#L82),
 if some browser/platform can't recognize this font, it means that 
browser/platform users can't see any "Apache Zeppelin" logo written with `Patua 
One`. 
Then they can't see this one as well. Do we need to cover this case?
https://cloud.githubusercontent.com/assets/10060731/18896823/d63be1a8-855f-11e6-8c32-8ccaf9ad2f52.png;>




---
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 #1465: [ZEPPELIN-1500] pyspark completion doesn't work.

2016-09-27 Thread astroshim
Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1465
  
@felixcheung Thank you for quick review. 
It's because we should get result message even if `pythonScriptInitialized` 
is `true` and I thought `pythonScriptInitialized` on the `completion()` 
increases the complexity of code.



---
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 #133: [ZEPPELIN-127] Fix issue raised by ZEPPELIN-127 related...

2016-09-27 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/133
  
@doanduyhai Then can you create a Jira issue for it and maybe update the PR 
description?



---
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-1501) Cassandra Interpreter SSL support

2016-09-27 Thread Robert Marshall (JIRA)
Robert Marshall created ZEPPELIN-1501:
-

 Summary: Cassandra Interpreter SSL support
 Key: ZEPPELIN-1501
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1501
 Project: Zeppelin
  Issue Type: Bug
  Components: Interpreters
Affects Versions: 0.6.1, 0.6.0, 0.5.6
Reporter: Robert Marshall
Priority: Minor


Cassandra Interpreter is missing configuration properties and implementation 
for connection to cluster using SSL.



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


[GitHub] zeppelin issue #1465: [ZEPPELIN-1500] pyspark completion doesn't work.

2016-09-27 Thread felixcheung
Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1465
  
why is `pythonScriptInitialized` check removed?


---
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 #1461: [MINOR] Rearrange the contents in "About Zeppelin"

2016-09-27 Thread felixcheung
Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1461
  
would `Patua One` font be not available on some platforms?


---
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 #264: ZEPPELIN-270 added env varibles for pyspark's correct f...

2016-09-27 Thread felixcheung
Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/264
  
I'm not sure if we do but the py4j version changes for the last few Spark 
releases, so if we do, we would need a way to set the right version as per 
spark 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 #86: Initial support for CassandraSQLContext for %sql queries

2016-09-27 Thread benbromhead
Github user benbromhead commented on the issue:

https://github.com/apache/zeppelin/pull/86
  
I'll look to address the remarks. I've also noticed the project now has a 
formal tracker via Jira. I'll also look to complete the requirements listed in 
https://zeppelin.apache.org/contribution/contributions.html 


---
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 #86: Initial support for CassandraSQLContext for %sql queries

2016-09-27 Thread doanduyhai
Github user doanduyhai commented on the issue:

https://github.com/apache/zeppelin/pull/86
  
@benbromhead some remarks before rebasing

1) maybe upgrade C* version and Spark/Cassandra connector version
2) maybe use Achilles-embedded (look Cassandra interpreter unit tests) 
instead of cassandra-unit. The project is no longer actively maintained 
3) the SparkInterpreter class has changed a lot, probably a good idea to 
cherry-pick your changes and apply them instead of going through the complete 
rebase
4) you've hard-coded the Spark/Cassandra connector dependency directly in 
`spark/pom.xml`. I'm not sure it's the cleanest way because people who don't 
use Cassandra still need to pull this dependency. Maybe it's better to create a 
separate Maven build profile for this. Until now it is done in 
`spark-dependencies/pom.xml` but since it will be removed soon by @AhyoungRyu, 
see https://github.com/apache/zeppelin/pull/1339



---
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 #409: Allow the spark app name to be configurable

2016-09-27 Thread craigching
Github user craigching commented on the issue:

https://github.com/apache/zeppelin/pull/409
  
It's not.  Someone else added this exact feature are part of another PR, so 
this has been implemented.


---
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 #530: ZEPPELIN-497 ] pyspark completion

2016-09-27 Thread astroshim
Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/530
  
@fanlu As @felixcheung mentioned, you can report issue to 
dev@zeppelin.apache.org. 
and Could you test https://github.com/apache/zeppelin/pull/1465 for solving 
your problem?


---
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 #1465: [ZEPPELIN-1500] pyspark completion doesn't work...

2016-09-27 Thread astroshim
GitHub user astroshim opened a pull request:

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

[ZEPPELIN-1500] pyspark completion doesn't work.

### What is this PR for?
@fanlu reported bug on https://github.com/apache/zeppelin/pull/530.
This PR fixes bug of checking condition in PySparkInterpreter.

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

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


### 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/astroshim/zeppelin ZEPPELIN-1500

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

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


commit ee2a29b95b0c1e5a3968fe0f722b57084d33910a
Author: astroshim 
Date:   2016-09-27T16:05:04Z

fix bug of condition check.




---
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-1500) pyspark completion doesn't work.

2016-09-27 Thread Shim HyungSung (JIRA)
Shim HyungSung created ZEPPELIN-1500:


 Summary: pyspark completion doesn't work. 
 Key: ZEPPELIN-1500
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1500
 Project: Zeppelin
  Issue Type: Bug
Reporter: Shim HyungSung
 Fix For: 0.7.0


@fanlu commented pyspark completion doesn't work.
https://github.com/apache/zeppelin/pull/530
https://issues.apache.org/jira/browse/ZEPPELIN-497



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


[GitHub] zeppelin issue #1334: [ZEPPELIN-682] New interpreter for Apache Beam (incuba...

2016-09-27 Thread kennknowles
Github user kennknowles commented on the issue:

https://github.com/apache/zeppelin/pull/1334
  
:+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 pull request #1464: [Zeppelin 796] [WIP] Duplicated notebook names ...

2016-09-27 Thread rajarajan-g
GitHub user rajarajan-g opened a pull request:

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

[Zeppelin 796] [WIP] Duplicated notebook names should not be allowed

### What is this PR for?
When a notebook is created/cloned/imported the title/name of the new 
notebook should not be duplicate of the existing notebook's title/name

### What type of PR is it?
Improvement

### Todos
* [ ] - Task

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

### How should this be tested?
1. Create a notebook with name 'Note', Notebook should allow user to create 
a note
2. When a user try to create/clone/import a notebook with name 'Note', 
Notebook should not 
allow user to create note.

### Screenshots (if appropriate)
Screenshot of error message displayed during create/clone notebook

![create](https://cloud.githubusercontent.com/assets/11922701/18878102/05b401ea-84ed-11e6-92c4-539cae37c4f9.png)

Screenshot of error message displayed during import notebook

![import](https://cloud.githubusercontent.com/assets/11922701/18878072/f69716a2-84ec-11e6-965a-3fbb29dd63d7.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/rajarajan-g/zeppelin ZEPPELIN-796

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

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


commit 681c600cd96dbab8ffe9fb44e93524bfbaacf109
Author: rajarajan-g 
Date:   2016-09-27T05:43:03Z

Code changes for ZEPPELIN-796

commit bfe7091902bb3e3c16fcf4b50322b7b2b6f9a2e0
Author: rajarajan-g 
Date:   2016-09-27T05:52:12Z

Removed unnecessary statements

commit 9071d2ae093f8fdfc1803e9a86f2234ee9eb78bf
Author: rajarajan-g 
Date:   2016-09-27T06:15:38Z

code formatted

commit 055637f56deede4a4fa19c5bf31ff13880c827fe
Author: rajarajan-g 
Date:   2016-09-27T06:53:56Z

removed unnecessary statement

commit a4dc418f6a89eaeca53775a7bf86a0868e440fb2
Author: rajarajan-g 
Date:   2016-09-27T07:26:54Z

fix for changes

commit 3991eceb93192536855d4550241951e23d5b57c3
Author: rajarajan-g 
Date:   2016-09-27T14:02:41Z

code fix for 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 #132: ZEPPELIN-142 add SparkSqlCompleter

2016-09-27 Thread nevillelyh
Github user nevillelyh commented on the issue:

https://github.com/apache/zeppelin/pull/132
  
Sorry haven't worked on this in a while and probably won't have time to. 
Closing it for 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 #133: [ZEPPELIN-127] Fix issue raised by ZEPPELIN-127 related...

2016-09-27 Thread doanduyhai
Github user doanduyhai commented on the issue:

https://github.com/apache/zeppelin/pull/133
  
@corneadoug Yes we need it, it's a very convenient helper class to display 
html/img ...


---
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 #1462: ZEPPELIN-1477. Add Integration Test for LivyInterprete...

2016-09-27 Thread zjffdu
Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1462
  
@Leemoonsoo @prabhjyotsingh @felixcheung Please help review, the failed 
test is irrelevant, I have created 
[ZEPPELIN-1498](https://issues.apache.org/jira/browse/ZEPPELIN-1498) for it.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[jira] [Created] (ZEPPELIN-1499) Create checkbox to choose how to show empty points in line charts

2016-09-27 Thread Nerea (JIRA)
Nerea created ZEPPELIN-1499:
---

 Summary: Create checkbox to choose how to show empty points in 
line charts
 Key: ZEPPELIN-1499
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1499
 Project: Zeppelin
  Issue Type: Improvement
Reporter: Nerea


If two lines want to be plotted, but they have different values in the X axis, 
the  empty points are taken as 0s. This makes it impossible to plot timeseries 
data with different timestamps. 

It would be necessary to have a checkbox where you can choose to plot empty 
points as 0s, as the average of the two nearest points, or just draw a line 
between them. 



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


[jira] [Created] (ZEPPELIN-1498) Flaky test: http://stackoverflow.com/questions/6121403/how-do-i-get-the-scala-version-from-within-scala-itself

2016-09-27 Thread Jeff Zhang (JIRA)
Jeff Zhang created ZEPPELIN-1498:


 Summary: Flaky test: 
http://stackoverflow.com/questions/6121403/how-do-i-get-the-scala-version-from-within-scala-itself
 Key: ZEPPELIN-1498
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1498
 Project: Zeppelin
  Issue Type: Bug
Affects Versions: 0.6.1
Reporter: Jeff Zhang


https://travis-ci.org/apache/zeppelin/builds/163016648
https://s3.amazonaws.com/archive.travis-ci.org/jobs/163016652/log.txt
{code}
---
Running org.apache.zeppelin.jdbc.JDBCInterpreterTest
Tests run: 8, Failures: 0, Errors: 8, Skipped: 0, Time elapsed: 0.199 sec <<< 
FAILURE! - in org.apache.zeppelin.jdbc.JDBCInterpreterTest
testAutoCompletion(org.apache.zeppelin.jdbc.JDBCInterpreterTest)  Time elapsed: 
0.074 sec  <<< ERROR!
java.nio.file.NoSuchFileException: 
/home/travis/build/apache/zeppelin/jdbc/target/tmp/h2-test-1299779248643275151
at sun.nio.fs.UnixException.translateToIOException(UnixException.java:86)
at sun.nio.fs.UnixException.rethrowAsIOException(UnixException.java:102)
at sun.nio.fs.UnixException.rethrowAsIOException(UnixException.java:107)
at 
sun.nio.fs.UnixFileSystemProvider.createDirectory(UnixFileSystemProvider.java:383)
at java.nio.file.Files.createDirectory(Files.java:630)
at java.nio.file.TempFileHelper.create(TempFileHelper.java:136)
at java.nio.file.TempFileHelper.createTempDirectory(TempFileHelper.java:173)
at java.nio.file.Files.createTempDirectory(Files.java:944)
at 
org.apache.zeppelin.jdbc.JDBCInterpreterTest.getJdbcConnection(JDBCInterpreterTest.java:58)
at 
org.apache.zeppelin.jdbc.JDBCInterpreterTest.setUp(JDBCInterpreterTest.java:80)
 
{code}



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


[jira] [Created] (ZEPPELIN-1497) %spark and %python interpreters are able to read credentials.json file

2016-09-27 Thread Dmitry Zhukov (JIRA)
Dmitry Zhukov created ZEPPELIN-1497:
---

 Summary: %spark and %python interpreters are able to read 
credentials.json file
 Key: ZEPPELIN-1497
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1497
 Project: Zeppelin
  Issue Type: Bug
Affects Versions: 0.6.1
Reporter: Dmitry Zhukov


Currently DB credentials for all users are stored in a plain text file on disk. 
This file is readable by any interpreter capable of running arbitrary code with 
Zeppelin user privileges, e.g. %python or %spark.

{code}
%spark
scala.io.Source.fromFile("/home/ubuntu/zeppelin-0.6.1-bin-netinst/conf/credentials.json").mkString
{code}
{noformat}
res6: String =
{
  "credentialsMap": {
"anonymous": {
  "userCredentials": {
"testdb": {
  "username": "user",
  "password": "pass"
}
  }
}
  }
}
{noformat}

{code}
%python
open("/home/ubuntu/zeppelin-0.6.1-bin-netinst/conf/credentials.json", 
'r').read()
{code}
{noformat}
'{\n  "credentialsMap": {\n"anonymous": {\n  "userCredentials": {\n 
   "testdb": {\n  "username": "user",\n  "password": "pass"\n   
 }\n  }\n}\n  }\n}'
{noformat}

Basically if one has %python or %spark interpreters enabled it makes the whole 
credentials store useless and even dangerous.



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


[GitHub] zeppelin issue #318: Save all repos, and not just the first two

2016-09-27 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue:

https://github.com/apache/zeppelin/pull/318
  
@corneadoug it can still be rebased, however more than two repos 
concurrently still not supported, so basically it wouldn't change anything. so 
it can be closed safely as well in case no activity 


---
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 #1458: [ZEPPELIN-1486] Allow configuring whether shown values...

2016-09-27 Thread spektom
Github user spektom commented on the issue:

https://github.com/apache/zeppelin/pull/1458
  
@AhyoungRyu Added, thanks for pointing it out 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.
---


[jira] [Created] (ZEPPELIN-1496) Apply Zeppelin-Web Good Practice guide #1 to the code

2016-09-27 Thread CORNEAU Damien (JIRA)
CORNEAU Damien created ZEPPELIN-1496:


 Summary: Apply Zeppelin-Web Good Practice guide #1 to the code
 Key: ZEPPELIN-1496
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1496
 Project: Zeppelin
  Issue Type: Improvement
Reporter: CORNEAU Damien
Assignee: CORNEAU Damien


This is some refactoring of zeppelin-web angular modules, to apply the IIFE as 
stated in the Good Practice Guide #1 
(https://zeppelin.apache.org/contribution/zeppelinweb/goodPracticeGuide01.html)



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


[GitHub] zeppelin issue #884: Note Permission: If userName does not exist in the shir...

2016-09-27 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/884
  
@wind0727 Any plan on taking care of this PR?
I think it would need a new one, based on a special branch instead of your 
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 #560: Add finish alert with blinking tab title

2016-09-27 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/560
  
@Stefan9149 Any plan on trying to finish this PR? Or should we just close 
it?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #418: [DOCS] Zeppelin Flink Spark tutorial

2016-09-27 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/418
  
@rawkintrevo Lets remove the `incubator` from URLs, rebase, then merge it


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #409: Allow the spark app name to be configurable

2016-09-27 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/409
  
@craigching Is this PR still needed?


---
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 #389: ZEPPELIN-385 Read-only mode for zeppelin notebooks, for...

2016-09-27 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/389
  
@babokim 
Going around old PRs,
Is it still needed considering the efforts for user authentication, and 
notebook acl restrictions?


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

2016-09-27 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/338
  
@abajwa-hw 
I'm going around old PRs, any plan on finishing 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 #336: Add bullet graph support

2016-09-27 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/336
  
Going around old PRs.
As stated before, we had issues with version 1.8.1, and have no plan on 
upgrading.
In case newer versions don't have issues we can create a Jira issue to 
upgrade.
I will be closing this PR if there is no other 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 #320: ZEPPELIN-289: User can now enter custom expressions in ...

2016-09-27 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/320
  
@rolmovel If this PRs is still needed, can we try to rebase it?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #318: Save all repos, and not just the first two

2016-09-27 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/318
  
@khalidhuseynov Is this still an issue with our current implementation?


---
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 #234: ZEPPELIN-234

2016-09-27 Thread guptarajat
Github user guptarajat commented on the issue:

https://github.com/apache/zeppelin/pull/234
  
Noyou can close this PR. As I remember many of the issues I tried to 
address in this PR were fixed later on by other guys. Thanks for pinging me on 
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 pull request #234: ZEPPELIN-234

2016-09-27 Thread guptarajat
Github user guptarajat closed the pull request at:

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


---
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 #299: a Js Intepreter, based on Nashorn

2016-09-27 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/299
  
@leonardofoderaro 
I'm going around old PRs,
Any plan on finishing 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 #273: ZEPPELIN-277: Set the (TAB) key as an auto-completion s...

2016-09-27 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/273
  
@tzolov Should we close 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 #291: Zeppelin-275: Add SpringXD Stream and Job Interpreters

2016-09-27 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/291
  
@tzolov @bzz 
Do we want to rebase and try to merge this PR?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #264: ZEPPELIN-270 added env varibles for pyspark's correct f...

2016-09-27 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/264
  
@riverajo @felixcheung @Leemoonsoo 
Any changes to be done here?
Is this documentation change still needed?


---
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 #252: Add ability to truncate displayed results while exporti...

2016-09-27 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/252
  
@azeloo Any plan on finishing this PR?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #246: ZEPPELIN-256 Self diagnosis spark configuration

2016-09-27 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/246
  
@Leemoonsoo Any plan on finishing 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 #234: ZEPPELIN-234

2016-09-27 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/234
  
@guptarajat Hi, any plan on finishing this PR?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #228: Restart spark interpreter when the spark context has sh...

2016-09-27 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/228
  
@akhiluanandh 
I'm going around old PRs.
Any plan on continuing that 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 #86: Initial support for CassandraSQLContext for %sql queries

2016-09-27 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/86
  
@bbromhead @jongyoul @Leemoonsoo any more feedback on changes needed on 
this PR?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1334: [ZEPPELIN-682] New interpreter for Apache Beam (incuba...

2016-09-27 Thread iemejia
Github user iemejia commented on the issue:

https://github.com/apache/zeppelin/pull/1334
  
@mfelgamal or @bzz you must announce this milestone in the beam mailing 
list 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 #197: Update SparkInterpreter.java

2016-09-27 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/197
  
We don't usually add to old branch if there is no more releases.
It seems the PR branch doesn't exist anymore.
Closing this PR is there is 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 #193: ZEPPELIN-211: Add Geode Interpreter remote completer

2016-09-27 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/193
  
@tzolov 
I'm going around old PRs,
What is the status of 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 #139: Move ZeppelinContext code to Scala

2016-09-27 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/139
  
@doanduyhai 
I'm going around old PRs,
What should we do about that 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 pull request #1462: ZEPPELIN-1477. Add Integration Test for LivyInt...

2016-09-27 Thread zjffdu
GitHub user zjffdu reopened a pull request:

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

ZEPPELIN-1477. Add Integration Test for LivyInterpreter

### What is this PR for?
This PR fix several issues of LivyInterpeter test. 
* Livy interpreter's test code is not in the right place, so it never runs.
* LivyHelperTest would fail.
* No Integration test for LivyInterpreter so any following change is not 
easy to be tested.

This PR would fix the above issues. Regarding the integration test, some of 
livy's artifact is not available in repository, so I have to copy them to 
livy/local-maven-repo as local repository.  And LivyInterperter's integration 
test require spark and livy to be installed. For now you have to download spark 
and livy manually. Please use spark 1.5.x and livy 0.2 which is currently 
supported. Download livy 0.2.0 from here 
[https://github.com/cloudera/livy/releases](https://github.com/cloudera/livy/releases)

And use the following command to execute the LivyIntegrationTest.
```
export SPARK_HOME=
export LIVY_HOME=
mvn clean package -pl 'livy' -Dtest=LivyIntegrationTest
```
If you hit any issues, you can check logs under `livy/target/tmp`.

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

### Todos
* [ ] - Task

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

### Screenshots (if appropriate)

![image](https://cloud.githubusercontent.com/assets/164491/18861677/b1389622-84b9-11e6-8b0a-424457ded975.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/zjffdu/zeppelin ZEPPELIN-1477

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

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


commit 152d346bb89f9b2de1c0f844ca53c93d8e9301ee
Author: Jeff Zhang 
Date:   2016-09-22T08:42:14Z

ZEPPELIN-1477. Add Integration Test for LivyInterpreter

commit 3d02606642c2892ffc5a910a718145c6fb629ed7
Author: Jeff Zhang 
Date:   2016-09-27T06:35:15Z

fix rat check

commit 80657633ae18c50b556100da4360d81c57cfbc1a
Author: Jeff Zhang 
Date:   2016-09-27T06:50:45Z

add missing livy jars




---
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 #133: [ZEPPELIN-127] Fix issue raised by ZEPPELIN-127 related...

2016-09-27 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/133
  
@doanduyhai Do we have the still the need of that 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 pull request #1462: ZEPPELIN-1477. Add Integration Test for LivyInt...

2016-09-27 Thread zjffdu
Github user zjffdu closed the pull request at:

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


---
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 #152: ZEPPELIN-157: Adding Map Visualization for Zeppelin

2016-09-27 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/152
  
@Madhuka Let's close this one since you opened another 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 #1436: ZEPPELIN-1452: Include Phoenix 'thin client' instructi...

2016-09-27 Thread minahlee
Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1436
  
Merge 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 pull request #61: Fixed pyspark indentation issue: ZEPPELIN-34

2016-09-27 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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 #1430: [ZEPPELIN-868] Notebook import fails when notebook is ...

2016-09-27 Thread vensant
Github user vensant commented on the issue:

https://github.com/apache/zeppelin/pull/1430
  
@corneadoug instead of validating the error in server side, I have just 
changed the code in the client side, that compares the input file size with the 
maximum limit of file size that is already set in the server side as a 
property. 
It is simpler than the one that catches the exception that is thrown in the 
server side. Please check with the code change and let me know.

Thanks,


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #132: ZEPPELIN-142 add SparkSqlCompleter

2016-09-27 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/132
  
@nevillelyh 
I'm going around old PRs, is there any news on this issue?
Or do you want to close it?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #125: [WIP] Automated test with Yarn cluster for Spark integr...

2016-09-27 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/125
  
@Leemoonsoo Do we want to keep this PR, or should we close it?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #72: [ZEPPELIN-9]: Provide Spark profiles for older 1.2.x and...

2016-09-27 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/72
  
If there is no more discussions, I will close this PR


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1463: [ZEPPELIN-1495] Add Good Practice #4

2016-09-27 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1463
  
@corneadoug Yeah tested again and now the code blocks are shown well. 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 #1463: [ZEPPELIN-1495] Add Good Practice #4

2016-09-27 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/1463
  
@AhyoungRyu Nice catch, I also escaped those in Good Practice #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 issue #1463: [ZEPPELIN-1495] Add Good Practice #4

2016-09-27 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1463
  
I can't see the code block! 
![screen shot 2016-09-27 at 3 30 52 
pm](https://cloud.githubusercontent.com/assets/10060731/18862486/71d7b1ee-84c7-11e6-82fe-5405547220bd.png)



---
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 #1463: [ZEPPELIN-1495] Add Good Practice #4

2016-09-27 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/1463
  
:) Merging it 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 #1463: [ZEPPELIN-1495] Add Good Practice #4

2016-09-27 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1463
  
@corneadoug 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 #1449: [MINOR] Change Markdown class name from MarkdownInterp...

2016-09-27 Thread minahlee
Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1449
  
Merging 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 #1458: [ZEPPELIN-1486] Allow configuring whether shown values...

2016-09-27 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1458
  
Hi @spektom. Thanks for your contribution :)

As you said 
>Yes. Documenting zeppelin.livy.spark.sql.truncateResult option.

But it seems there is no docs file change. The below code needs to be added 
[here](https://github.com/apache/zeppelin/blob/master/docs/interpreter/livy.md#configuration)
 as well. 

```

  zeppelin.livy.spark.sql.truncateResult
  false
  Truncate long strings in SQL display

```


---
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 #1463: [ZEPPELIN-1495] Add Good Practice #4

2016-09-27 Thread corneadoug
GitHub user corneadoug opened a pull request:

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

[ZEPPELIN-1495] Add Good Practice #4

### What is this PR for?
This Good Practice Guide promotes the usage of `ng-bind` instead of the 
classic `{{}}` syntax, to have a performance boost.


### What type of PR is it?
Documentation

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

### 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/corneadoug/incubator-zeppelin ZEPPELIN-1495

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

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


commit c420d8ad34ee670ce215105c5516ac9d3ad0748c
Author: Damien CORNEAU 
Date:   2016-09-27T06:04:46Z

Add Good Practice #4




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