date:20160922

[jira] [Created] (ZEPPELIN-1478) Emails are not being delivered to the corporate email

2016-09-22 Thread Kavin Kumar (JIRA)

Kavin Kumar created ZEPPELIN-1478:
-

 Summary: Emails are not being delivered to the corporate email
 Key: ZEPPELIN-1478
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1478
 Project: Zeppelin
  Issue Type: Bug
Reporter: Kavin Kumar
Assignee: Lee moon soo


Ticket to track the below issue since I couldn't find any other queue to report 
this issue.Could you please do the needful?

===

On Mon, Sep 12, 2016 at 12:59 PM, Kavin Kumar K S  
wrote:

HI Moon,

This is w.r.t the issue of not getting emails when we send to 
dev@zeppelin.apache.org When we checked on the google side, they asked to check 
with apache side and whitelist our domain (imaginea.com) or do something else 
related to it.Please let me know whom should I contact for this.

Thanks,
Kavin

On Fri, Sep 9, 2016 at 4:17 PM, Ravi Gokara  wrote:

--fwd---
-- Forwarded message --
From: Google for Work Support 
Date: Fri, Sep 9, 2016 at 4:14 PM
Subject: Message sent from Imaginea to external mail group not received 
by recipient party [ ref:_00D00VNwG._50060119mxE:ref ]
To: "rav...@pramati.com" 


Hello Ravi,

Thanks again for calling in sir.

As discussed the when I tracked the following message ID you provided

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

2016-09-22 Thread bzz

Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1334
  
Great, thank you @mfelgamal for great job laying foundation for Apache Beam 
integration! I hate to ask but, could you please rebase it on latest master one 
more time? Looks like it have diverged a bit..

I will be happy to merge it to master ASAP right after that.


---
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 #1452: ZEPPELIN-1442. UDF can not be found due to 2 in...

2016-09-22 Thread zjffdu

GitHub user zjffdu opened a pull request:

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

ZEPPELIN-1442. UDF can not be found due to 2 instances of SparkSession is 
created

### What is this PR for?
The issue is that we create 2 SparkSession in zeppelin_pyspark.py (Because 
we create SQLContext first which will create SparkSession underlying). This 
cause 2 instances of SparkSession in JVM side and this means we have 2 
instances of Catalog as well. So udf registered in SQLContext can be used in 
SparkSession. This PR will create SparkSession first and then assign its 
internal SQLContext to sqlContext in pyspark. 


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

### Todos
* [ ] - Task

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

### How should this be tested?
Integration test is added. 

### Screenshots (if appropriate)

![image](https://cloud.githubusercontent.com/assets/164491/18774832/7f270de4-818f-11e6-9e4f-c4def4353e5c.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-1442

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

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


commit 948e8657634686f16a405b3938884a5fe48dfc1c
Author: Jeff Zhang 
Date:   2016-09-23T05:08:49Z

ZEPPELIN-1442. UDF can not be found due to 2 instances of SparkSession is 
created




---
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 #1447: [ZEPPELIN-1293] Re-create Livy session if it's lost

2016-09-22 Thread spektom

Github user spektom commented on the issue:

https://github.com/apache/zeppelin/pull/1447
  
Of course, it would be great.
On Fri, Sep 23, 2016 at 06:28 Jeff Zhang  wrote:

> @spektom  Thanks for the contribution, I am
> working on a PR about integration test of livy interpreter, could you mind
> to wait for that PR and then add integration test to that ?
>
> â
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> , or 
mute
> the thread
> 

> .
>



---
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 #1447: [ZEPPELIN-1293] Re-create Livy session if it's lost

2016-09-22 Thread zjffdu

Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1447
  
@spektom Thanks for the contribution, I am working on a PR about 
integration test of livy interpreter, could you mind to wait for that PR and 
then add integration test to that ?


---
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 #1450: ZEPPELIN-1473. It is not necessary to create SQLContex...

2016-09-22 Thread prabhjyotsingh

Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/1450
  
Merging 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 #1429: ZEPPELIN-1440 Notebook clone: prefix name with "Copy o...

2016-09-22 Thread felixcheung

Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1429
  
I think it might be easier if we don't prefix "Copy of"? We might need to 
localize or make this localizable at some point


---
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 #1427: [ZEPPELIN-1435] Move zeppelin-server-*.jar under lib i...

2016-09-22 Thread felixcheung

Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1427
  
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 #1450: ZEPPELIN-1473. It is not necessary to create SQLContex...

2016-09-22 Thread felixcheung

Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1450
  
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-1477) Add Integration Test for LivyInterpreter

2016-09-22 Thread Jeff Zhang (JIRA)

Jeff Zhang created ZEPPELIN-1477:


 Summary: Add Integration Test for LivyInterpreter
 Key: ZEPPELIN-1477
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1477
 Project: Zeppelin
  Issue Type: Sub-task
  Components: Interpreters
Affects Versions: 0.6.1
Reporter: Jeff Zhang
Assignee: Jeff Zhang
 Fix For: 0.7.0


Currently the unit test for LivyInterpreter is not sufficient, it can not cover 
many of things. I create this ticket to add Integration test for 
LivyInterpreter, so that we can don't need to manually verify LivyInterpreter 
for any following change. 



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

[GitHub] zeppelin issue #1431: [ZEPPELIN-1446] Fix broken layout of `Create new inter...

2016-09-22 Thread corneadoug

Github user corneadoug commented on the issue:

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

[jira] [Created] (ZEPPELIN-1476) 파라그래프에 특정 차트 On/Off 기능이 있으면 좋겠습니다.

2016-09-22 Thread SanAh Kang (JIRA)

SanAh Kang created ZEPPELIN-1476:


 Summary: 파라그래프에 특정 차트 On/Off 기능이 있으면 좋겠습니다.
 Key: ZEPPELIN-1476
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1476
 Project: Zeppelin
  Issue Type: New Feature
Affects Versions: 0.6.1
Reporter: SanAh Kang
Priority: Critical


제플린 파라그래프에서 막대, 선형 그래프 등은 큰 문제가 없으나, 분산형 차트를 누를 경우 멈춰버리는 문제가 있습니다.

파라그래프 옵션에서 사용할 차트 종류를 필터할 수 있는 수단이 있으면 좋을 것 같아요.
*사용자 교육으로 해결할 수 있지만 인간이 하는 일은 언제나 실수가 따르니까...

[~Ahyoung] [~moon]



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

[GitHub] zeppelin pull request #1396: [ZEPPELIN-1405] ConnectionPool for JDBCInterpre...

2016-09-22 Thread astroshim

Github user astroshim closed the pull request at:

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


---
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 #1396: [ZEPPELIN-1405] ConnectionPool for JDBCInterpre...

2016-09-22 Thread astroshim

GitHub user astroshim reopened a pull request:

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

[ZEPPELIN-1405] ConnectionPool for JDBCInterpreter.

### What is this PR for?
This PR is for refactoring code for JDBCInterpreter.
There is no putting 'Connection' to 'propertyKeyUnusedConnectionListMap' 
anywhere in the original code.


### What type of PR is it?
Improvement 


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


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

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

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


commit 12dd7cb239cd22cb85d55221a18bbf157665a45a
Author: astroshim 
Date:   2016-09-02T09:05:50Z

remove propertyKeyUnusedConnectionListMap map

commit 4dabbcc98d5f8e8013502ba786ec7f60f115efff
Author: astroshim 
Date:   2016-09-04T18:08:36Z

wip) changing to use dbcp

commit 9f1e368fe16ef889715b8a58e9592cb0393ae464
Author: astroshim 
Date:   2016-09-05T15:58:25Z

replace ConnectionPool

commit 4d4f85cf1e59aaecbabe025b6c261349c94abe4a
Author: astroshim 
Date:   2016-09-05T16:10:41Z

refactoring code of close()

commit 21217a7340438bbd39b75a1dd3756885c4d2d911
Author: astroshim 
Date:   2016-09-07T02:06:50Z

fix indentation.

commit efc2bfc2c590ea6ccc3ca99490e326f25a3aa3c4
Author: astroshim 
Date:   2016-09-12T14:53:09Z

rebase

commit 1862ae6c694681861a6e51f26960b61478c6df4d
Author: astroshim 
Date:   2016-09-21T12:58:38Z

Merge branch 'master' into ZEPPELIN-1405

commit f6998c27ddad2bd93d48a3acbdfca24c698095d6
Author: astroshim 
Date:   2016-09-22T12:52:35Z

Merge branch 'master' into ZEPPELIN-1405




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

2016-09-22 Thread kavinkumarks

Github user kavinkumarks commented on the issue:

https://github.com/apache/zeppelin/pull/1408
  
The CI build is green.


---
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 #1431: [ZEPPELIN-1446] Fix broken layout of `Create new inter...

2016-09-22 Thread AhyoungRyu

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1431
  
CI is green now! 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 #1449: [MINOR][DOC] Change Markdown class name from Markdown ...

2016-09-22 Thread Leemoonsoo

Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1449
  
In case users who already have conf/interpreter.json upgrades Zeppelin, the 
file includes old classname. for example

```
  "interpreterSettings": {
"2BUVF72FJ": {
  "id": "2BUVF72FJ",
  "name": "md",
  "group": "md",
  "interpreterGroup": [
{
  "name": "md",
  "class": "org.apache.zeppelin.markdown.Markdown",
  "defaultInterpreter": false
}
  ],
```

Will there be any problem because of classname change in this case?
Should we change `MarkdownInterpreter` to `Markdown`? or `Markdown` to 
`MarkdownInterpreter`?

What do you think?


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

[GitHub] zeppelin issue #1448: [MINOR] Remove spaces between intepreter group and nam...

2016-09-22 Thread AhyoungRyu

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1448
  
@minahlee Nice catch! 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 #1449: [MINOR][DOC] Change Markdown class name from Markdown ...

2016-09-22 Thread AhyoungRyu

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1449
  
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 #1339: [ZEPPELIN-1332] Remove spark-dependencies & suggest ne...

2016-09-22 Thread AhyoungRyu

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1339
  
I think this PR is working well as expected(at least to me haha). So ready 
for review again.
@moon If you possible, could you please check this one again? :)


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

[GitHub] zeppelin issue #1363: [ZEPPELIN-1371]add text/numeric conversion support to ...

2016-09-22 Thread Peilin-Yang

Github user Peilin-Yang commented on the issue:

https://github.com/apache/zeppelin/pull/1363
  
@corneadoug I have fixed the bug you mentioned. Testing was done and it 
seems ok.


---
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-22 Thread kavinkumarks

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

2016-09-22 Thread kavinkumarks

GitHub user kavinkumarks reopened a pull request:

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

[ZEPPELIN-808] Changed Permissions on Notebooks do not provide helpful 
error to user

### What is this PR for?
This is about showing information to the user when there are errors on 
running paragraphs eg. there could be permission related issue with notebook.

### What type of PR is it?
Improvement

### Todos
NA

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

### How should this be tested?
* Create a notebook and change the file system permission for the notebook 
folder to have system level write permission to a different user.For eg. if you 
are running your local zeppelin server with id [USERNAME], then change the file 
system permission for one of your notebooks created with the former username to 
different one eg. ROOT user who will only have the write permission
* Try to run all the paragraphs or any individual paragraph for the notebook
* The information as shown in the screenshot should be displayed and the 
dialog could be closed by the 'Close' button.

### Screenshots (if appropriate)

![erroraboutrunningparagraph-2](https://cloud.githubusercontent.com/assets/20789766/18507272/4cdffe08-7a8d-11e6-8ec7-c712d28cd155.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/kavinkumarks/zeppelin 
zeppelin-808-handle-notebook-permission-error

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

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


commit 57907afd33d5cb554419e6ab8fd08e0a672fa0a0
Author: Kavin 
Date:   2016-09-06T12:21:26Z

Show information to the user when there are errors with related to
permission on running paragraphs.

commit b797a3ce01bcdc105735cdd1a0a8f64102879a7b
Author: Kavin 
Date:   2016-09-12T14:55:49Z

Handle FileSystemException on running paragraph and show the relevant
error message to the user.

commit c3885a1abd7c452d970bba72d8b8e6d7a742e4ac
Author: Kavin 
Date:   2016-09-22T12:49:32Z

Exit the run paragraph execution when there are errors with the notebook
file system.




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

2016-09-22 Thread kavinkumarks

Github user kavinkumarks commented on the issue:

https://github.com/apache/zeppelin/pull/1408
  
Reopening to trigger the 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 #1451: [ZEPPELIN-1279] Zeppelin with CDH5.x docker doc...

2016-09-22 Thread astroshim

GitHub user astroshim opened a pull request:

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

[ZEPPELIN-1279] Zeppelin with CDH5.x docker document.

### What is this PR for?
This PR is for the documentation of running zeppelin with CDH docker 
environment.
and This PR is the part of 
https://issues.apache.org/jira/browse/ZEPPELIN-1198.

Tested CDH5.7 on ubuntu.


### What type of PR is it?
Documentation


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


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

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

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


commit a7b5b2d9c8885c2bce77600790c039074ec2f86b
Author: astroshim 
Date:   2016-09-22T07:55:23Z

cdh docker environment

commit e66993fdae06de5c2f2c4bbfc9764084af4c49c1
Author: astroshim 
Date:   2016-09-22T13:15:08Z

fix doc




---
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-1475) Maven build fails on zeppelin-server

2016-09-22 Thread Danilo Ascione (JIRA)

Danilo Ascione created ZEPPELIN-1475:


 Summary: Maven build fails on zeppelin-server
 Key: ZEPPELIN-1475
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1475
 Project: Zeppelin
  Issue Type: Bug
Affects Versions: 0.6.1
Reporter: Danilo Ascione
Priority: Blocker


[ERROR] Failed to execute goal on project zeppelin-server: Could not resolve 
dependencies for project 
org.apache.zeppelin:zeppelin-server:jar:0.7.0-SNAPSHOT: Could not find artifact 
org.apache.zeppelin:zeppelin-zengine:jar:0.7.0-SNAPSHOT in apache.snapshots 
(http://repository.apache.org/snapshots)

Package zeppelin-zengine:jar:0.7.0 is not in repository.apache.org



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

[GitHub] zeppelin issue #1408: [ZEPPELIN-808] Changed Permissions on Notebooks do not...

2016-09-22 Thread kavinkumarks

Github user kavinkumarks commented on the issue:

https://github.com/apache/zeppelin/pull/1408
  
@corneadoug Made changes to exit the paragraph execution when there are 
errors with the notebook file system.

Thanks,
Kavin
MailTo: kavin.ku...@imaginea.com


---
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-1474) Maven build fails on zeppelin-web

2016-09-22 Thread Danilo Ascione (JIRA)

Danilo Ascione created ZEPPELIN-1474:


 Summary: Maven build fails on zeppelin-web
 Key: ZEPPELIN-1474
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1474
 Project: Zeppelin
  Issue Type: Bug
Affects Versions: 0.6.1
 Environment: Mac OS X 10.11.6, Maven 3.3.9, Java version: 1.8.0_92, 
vendor: Oracle Corporation
Reporter: Danilo Ascione
Priority: Blocker


[ERROR] Failed to execute goal 
com.github.eirslett:frontend-maven-plugin:0.0.25:npm (npm install) on project 
zeppelin-web: Failed to run task: 'npm install --color=false' failed. (error 
code 1) -> [Help 1]

I was able to build adding this line to .bowerrc file:

"registry": "https://bower.herokuapp.com;



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

[GitHub] zeppelin pull request #1431: [ZEPPELIN-1446] Fix broken layout of `Create ne...

2016-09-22 Thread astroshim

Github user astroshim closed the pull request at:

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


---
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 #1431: [ZEPPELIN-1446] Fix broken layout of `Create ne...

2016-09-22 Thread astroshim

GitHub user astroshim reopened a pull request:

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

[ZEPPELIN-1446] Fix broken layout of `Create new interpreter` UI.

### What is this PR for?
This PR fixes broken layout of `Create new interpreter` UI.


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


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


### How should this be tested?
- Try to create new interpreter on the Interpreters menu.


### Screenshots (if appropriate)
- before

![image](https://cloud.githubusercontent.com/assets/3348133/18592384/b3bfe120-7c71-11e6-80c2-31d0b4363009.png)


- after

![image](https://cloud.githubusercontent.com/assets/3348133/18592410/c777aef0-7c71-11e6-9379-9b424823fbed.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 ZEPPELIN-1446

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

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


commit 5f0a461a3ebf81a36cb126cf216f3ed9a8231f9f
Author: astroshim 
Date:   2016-09-16T15:53:21Z

fix align checkboxs.

commit 2d3ec76702c1127072f92489f5be333f7d85679f
Author: astroshim 
Date:   2016-09-18T13:58:21Z

fix space

commit fada36bc7076c040bbc903fbb1708d5b448204d2
Author: astroshim 
Date:   2016-09-19T07:42:47Z

fix spaces

commit 69cc1a076a725948a3c99e405a006cce78e81974
Author: astroshim 
Date:   2016-09-21T12:56:10Z

Merge branch 'master' into ZEPPELIN-1446

commit 729215b67757de84d2f10b7c65e971de777a4f87
Author: astroshim 
Date:   2016-09-22T05:41:03Z

Merge branch 'master' into ZEPPELIN-1446




---
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 #1440: ZEPPELIN-1427. Scala z.show() doesn't work on v.0.6.1

2016-09-22 Thread zjffdu

Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1440
  
@Leemoonsoo  I added some unit test, and also fix another bug. The bug is 
that exception will be throw when display `Dataset`, because we assume the type 
of each Record is Row while it is not true for `Dataset`, so I convert 
`Dataset` to `DataFrame` first when displaying Dataset. 

The failed test is irrelevant. 


---
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 #1265: [ZEPPELIN-1210] Run interpreter per user

2016-09-22 Thread jongyoul

Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1265
  
@corneadoug Done. :-)


---
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 #1290: [ZEPPELIN-708]shift positions automatically to ...

2016-09-22 Thread mwkang

Github user mwkang commented on the issue:

https://github.com/apache/zeppelin/pull/1290
  
@corneadoug Thanks for your comment. I rebased my branch.


---
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][DOC] Change Markdown class name from Markdown ...

2016-09-22 Thread prabhjyotsingh

Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/1449
  
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 #1450: ZEPPELIN-1473. It is not necessary to create SQLContex...

2016-09-22 Thread prabhjyotsingh

Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/1450
  
Tested with latest version of LIVY, works as expected.
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 #1442: ZEPPELIN-1459: Zeppelin JDBC URL properties man...

2016-09-22 Thread asfgit

Github user asfgit closed the pull request at:

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


---
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 #1450: ZEPPELIN-1473. It is not necessary to create SQ...

2016-09-22 Thread zjffdu

GitHub user zjffdu opened a pull request:

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

ZEPPELIN-1473. It is not necessary to create SQLContext in LivyInterpreter

### What is this PR for?
Livy will create SQLContext/HiveContext internally, (LIVY-94), so it is not 
necessary to create that in LivyInterpreter. Otherwise sqlContext in zeppelin 
will override that in livy. 



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

### Todos
* [ ] - Task

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

### How should this be tested?
Tested manually.  HiveContext is created properly in livy, and can access 
hive data. 

### Screenshots (if appropriate)

![image](https://cloud.githubusercontent.com/assets/164491/18743886/bff7ae8e-80ed-11e6-83e6-0769c30e4094.png)

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

â¦reter

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

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

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

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


commit bae46a1f025855939e559bef8196157eb8317b25
Author: Jeff Zhang 
Date:   2016-09-22T09:53:51Z

ZEPPELIN-1473. It is not necessary to create SQLContext in LivyInterpreter




---
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 #1450: ZEPPELIN-1473. It is not necessary to create SQLContex...

2016-09-22 Thread zjffdu

Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1450
  
@prabhjyotsingh Mind to help review it ? 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 pull request #1437: ZEPPELIN-1454: Wrong property value on interpre...

2016-09-22 Thread asfgit

Github user asfgit closed the pull request at:

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


---
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][DOC] Change Markdown class name from Ma...

2016-09-22 Thread minahlee

GitHub user minahlee opened a pull request:

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

[MINOR][DOC] Change Markdown class name from Markdown to MarkdownInterpreter

### What is this PR for?
Markdown interpreter's class name have changed from `Markdown` to 
`MarkdownInterpreter` in #1384 so this PR change markdown class name 
accordingly.

### What type of PR is it?
Documentation

### 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/minahlee/zeppelin update/markdownClassName

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

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


commit d0917308246a94a621694dc43ebbc6f1360ca37d
Author: Mina Lee 
Date:   2016-09-22T08:31:24Z

Change Markdown class name from Markdown -> MarkdownInterpreter




---
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 #1448: [MINOR] Remove spaces between intepreter group ...

2016-09-22 Thread minahlee

GitHub user minahlee opened a pull request:

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

[MINOR] Remove spaces between intepreter group and name

### What is this PR for?
#1395 changed alignment of html tag and it caused to make space between 
interpreter group and name.

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

### Screenshots (if appropriate)
**Before**
https://cloud.githubusercontent.com/assets/8503346/18743269/7fa34b16-80b8-11e6-979a-194e7f6bef8f.png;>

**After**
https://cloud.githubusercontent.com/assets/8503346/18743271/83807038-80b8-11e6-80b1-775c3369940e.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/minahlee/zeppelin showInterpreterList

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

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


commit 5e2315a262010b5e853b7beb207434c56ab80203
Author: Mina Lee 
Date:   2016-09-22T09:31:14Z

Remove space in available interpreters list




---
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 #1443: [ZEPPELIN-1462] Zeppelin-Web Good Practices #3

2016-09-22 Thread asfgit

Github user asfgit closed the pull request at:

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


---
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-22 Thread vensant

Github user vensant commented on the issue:

https://github.com/apache/zeppelin/pull/1430
  
@r-kamath thanks for the inputs!


---
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 #1431: [ZEPPELIN-1446] Fix broken layout of `Create new inter...

2016-09-22 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1431
  
re-build CI


---
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-22 Thread vensant

Github user vensant commented on the issue:

https://github.com/apache/zeppelin/pull/1430
  
@corneadoug once the file is uploaded, it tries to process the json but 
because the file exceeds the max limit it tries to close the websocket event. 
so I am trying to broadcast and catch the error code and setting the error 
message in html. 
We can also implement it the way you have suggested by stopping the flow of 
execution by comparing the size with the max size in the client side itself


---
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 #1431: [ZEPPELIN-1446] Fix broken layout of `Create ne...

2016-09-22 Thread astroshim

GitHub user astroshim reopened a pull request:

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

[ZEPPELIN-1446] Fix broken layout of `Create new interpreter` UI.

### What is this PR for?
This PR fixes broken layout of `Create new interpreter` UI.


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


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


### How should this be tested?
- Try to create new interpreter on the Interpreters menu.


### Screenshots (if appropriate)
- before

![image](https://cloud.githubusercontent.com/assets/3348133/18592384/b3bfe120-7c71-11e6-80c2-31d0b4363009.png)


- after

![image](https://cloud.githubusercontent.com/assets/3348133/18592410/c777aef0-7c71-11e6-9379-9b424823fbed.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 ZEPPELIN-1446

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

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


commit 5f0a461a3ebf81a36cb126cf216f3ed9a8231f9f
Author: astroshim 
Date:   2016-09-16T15:53:21Z

fix align checkboxs.

commit 2d3ec76702c1127072f92489f5be333f7d85679f
Author: astroshim 
Date:   2016-09-18T13:58:21Z

fix space

commit fada36bc7076c040bbc903fbb1708d5b448204d2
Author: astroshim 
Date:   2016-09-19T07:42:47Z

fix spaces

commit 69cc1a076a725948a3c99e405a006cce78e81974
Author: astroshim 
Date:   2016-09-21T12:56:10Z

Merge branch 'master' into ZEPPELIN-1446

commit 729215b67757de84d2f10b7c65e971de777a4f87
Author: astroshim 
Date:   2016-09-22T05:41:03Z

Merge branch 'master' into ZEPPELIN-1446




---
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 #1431: [ZEPPELIN-1446] Fix broken layout of `Create ne...

2016-09-22 Thread astroshim

Github user astroshim closed the pull request at:

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


---
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 #1431: [ZEPPELIN-1446] Fix broken layout of `Create new inter...

2016-09-22 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1431
  
I rebased but 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.
---

48 matches

Mail list logo