[GitHub] zeppelin issue #1451: [ZEPPELIN-1279] Zeppelin with CDH5.x docker document.

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

https://github.com/apache/zeppelin/pull/1451
  
@astroshim still CI is not happy but it seems irrelevant to this change. As 
below result says, there is a test failure in `zeppelin-display`. 

```
36mTests: succeeded 17, failed 1, canceled 0, ignored 0, pending 0
*** 1 TEST FAILED ***
[INFO] 

[INFO] Reactor Summary:
[INFO] 
[INFO] Zeppelin ... SUCCESS [  
4.051 s]
[INFO] Zeppelin: Interpreter .. SUCCESS [02:13 
min]
[INFO] Zeppelin: Zengine .. SUCCESS [01:53 
min]
[INFO] Zeppelin: Display system apis .. FAILURE [ 
16.048 s]
[INFO] Zeppelin: Spark dependencies ... SKIPPED
[INFO] Zeppelin: Spark  SKIPPED
[INFO] Zeppelin: Markdown interpreter . SKIPPED
[INFO] Zeppelin: Angular interpreter .. SKIPPED
[INFO] Zeppelin: Shell interpreter  SKIPPED
[INFO] Zeppelin: Livy interpreter . SKIPPED
[INFO] Zeppelin: HBase interpreter  SKIPPED
[INFO] Zeppelin: PostgreSQL interpreter ... SKIPPED
[INFO] Zeppelin: JDBC interpreter . SKIPPED
[INFO] Zeppelin: File System Interpreters . SKIPPED
[INFO] Zeppelin: Flink  SKIPPED
[INFO] Zeppelin: Apache Ignite interpreter  SKIPPED
[INFO] Zeppelin: Kylin interpreter  SKIPPED
[INFO] Zeppelin: Python interpreter ... SKIPPED
[INFO] Zeppelin: Lens interpreter . SKIPPED
[INFO] Zeppelin: Apache Cassandra interpreter . SKIPPED
[INFO] Zeppelin: Elasticsearch interpreter  SKIPPED
[INFO] Zeppelin: BigQuery interpreter . SKIPPED
[INFO] Zeppelin: Alluxio interpreter .. SKIPPED
[INFO] Zeppelin: web Application .. SKIPPED
[INFO] Zeppelin: Server ... SKIPPED
[INFO] Zeppelin: Packaging distribution ... SKIPPED
[INFO] Zeppelin: R Interpreter  SKIPPED
[INFO] Zeppelin: Scalding interpreter . SKIPPED
[INFO] Zeppelin: Examples . SKIPPED
[INFO] Zeppelin: Example application - Clock .. SKIPPED
[INFO] Zeppelin: Example application - Horizontal Bar chart SKIPPED
[INFO] 

[INFO] BUILD FAILURE
[INFO] 

[INFO] Total time: 04:28 min
[INFO] Finished at: 2016-09-27T02:59:19+00:00
[INFO] Final Memory: 33M/421M
[INFO] 

[ERROR] Failed to execute goal 
org.scalatest:scalatest-maven-plugin:1.0:test (test) on project 
zeppelin-display_2.11: There are test failures -> [Help 1]
[ERROR] 
``` 

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

2016-09-26 Thread zjffdu
GitHub user zjffdu opened 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. Download livy 0.2.0 
from here 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




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1392: [ZEPPELIN-1437, 1438] Multi-user note management - use...

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

https://github.com/apache/zeppelin/pull/1392
  
i've addressed 
[zeppelin-1438](https://issues.apache.org/jira/browse/ZEPPELIN-1438) here as 
well in last commit a2ce26889117f364f9d2a5242f6c4daa1c25e265, so user workbench 
should be updating user lists without manual reload, ready for review


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


[jira] [Created] (ZEPPELIN-1495) Zeppelin-Web Good Practices #4 - Using ng-bind

2016-09-26 Thread CORNEAU Damien (JIRA)
CORNEAU Damien created ZEPPELIN-1495:


 Summary: Zeppelin-Web Good Practices #4 - Using ng-bind
 Key: ZEPPELIN-1495
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1495
 Project: Zeppelin
  Issue Type: Improvement
Reporter: CORNEAU Damien
Assignee: CORNEAU Damien


This Good Practive Guide will focus on using ng-bind instead of the {{}} 
syntax, to have a performance boost.



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


[jira] [Created] (ZEPPELIN-1494) Bind JDBC result to a dataset on the zeppelin context

2016-09-26 Thread Nathan Scully (JIRA)
Nathan Scully created ZEPPELIN-1494:
---

 Summary: Bind JDBC result to a dataset on the zeppelin context
 Key: ZEPPELIN-1494
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1494
 Project: Zeppelin
  Issue Type: Improvement
Reporter: Nathan Scully


There doesn't seem to be an easy way to leverage the JDBC interpreter to 
offload the work to a database then use the result in the following paragraph. 
I would like an easy way for a team to write SQL using a predefined connection 
against our database then use python to work with the results.

E.g.
{code}
%jdbc(dataset='mytableresult')
SELECT 1 FROM mytable;
{code}
Then access it:
{code}
%python
df =z.datasets('mytableresult)
{code}

The only other way I could see this working is using pyspark and the 
sqlc.read.jdbc but having to remember the URLs and driver etc is impractical 
unless they could be accessed as properties from the main context. Eg:
{code}
%pyspark
jdbcDF = sqlc.read.format("jdbc").options(z.interpreters.jdbc()).load()
{code}




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


[GitHub] zeppelin pull request #1461: [MINOR] Rearrange the contents in "About Zeppel...

2016-09-26 Thread AhyoungRyu
GitHub user AhyoungRyu opened a pull request:

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

[MINOR] Rearrange the contents in "About Zeppelin"

### What is this PR for?
Just rearranged the placement of Zeppelin logo, title and text in "About 
Zeppelin". In the mobile screen as well. 

### What type of PR is it?
Improvement

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

### How should this be tested?
To check this change in your local, build only `zeppelin-web` with `./grunt 
build` and then browse with `./grunt serve`.

### Screenshots (if appropriate)
- **Before** 
  1. big screen
https://cloud.githubusercontent.com/assets/10060731/18860571/8aa85aec-84b9-11e6-9112-b6869291f8f0.png;
 width="500px">

  2. mobile screen
https://cloud.githubusercontent.com/assets/10060731/18860576/91f623a6-84b9-11e6-96ba-e34c43221143.png;
 width="300px">

- **After** 
  1. big screen
https://cloud.githubusercontent.com/assets/10060731/18860581/9d4c6a4e-84b9-11e6-9b0f-0b3dac3cede3.png;
 width="500px">
  
  2. mobile screen
https://cloud.githubusercontent.com/assets/10060731/18860583/a599c4bc-84b9-11e6-9c67-4dcf2394d5fc.png;
 width="300px">



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



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

$ git pull https://github.com/AhyoungRyu/zeppelin improve/aboutZeppelin

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

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


commit f0209336c05c02a7134064113f3fd56c809e17cc
Author: AhyoungRyu 
Date:   2016-09-27T04:49:43Z

Rearrange 'About Zeppelin'




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1392: [ZEPPELIN-1437] Multi-user note management - us...

2016-09-26 Thread khalidhuseynov
GitHub user khalidhuseynov reopened a pull request:

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

[ZEPPELIN-1437] Multi-user note management - user aware reload broadcast

### What is this PR for?
This PR addresses part of multi-user note management in Zeppelin. One of 
the tasks namely listing notes per user on Zeppelin start was addressed in 
#1330. However that PR didn't solve all problems, and reloading notes was 
incomplete as well as socket broadcast was not user aware 
[ZEPPELIN-1437](https://issues.apache.org/jira/browse/ZEPPELIN-1437), 
[ZEPPELIN-1438](https://issues.apache.org/jira/browse/ZEPPELIN-1438). This PR 
addresses those issue.

### What type of PR is it?
Improvement

### Todos
* [x] - list notes per user on reload
* [x] - broadcast per user (multicast)
* [x] - tests
* [x] - use authorization module to filter notes on sync
* [ ] - discussion and review

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

### How should this be tested?
1. Start Zeppelin
2. Login as user1, and user2 on different windows
3. Each user should be able to see their own note workbench
4. If note changed to private (readers, writers not empty), that note 
should disappear from others note workbench.


### Screenshots (if appropriate)

![reload_broadcast](https://cloud.githubusercontent.com/assets/1642088/18679507/e4a0161c-7f9a-11e6-9d57-0930abf4b780.gif)


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



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

$ git pull https://github.com/khalidhuseynov/incubator-zeppelin 
feat/multi-user-notes

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

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


commit 17e2d4c0c5e5528031b36218339305300a69a543
Author: Khalid Huseynov 
Date:   2016-09-01T07:29:57Z

first draft

commit b7f19c918b015867aefce5bd5b3a08fd65d720ff
Author: Khalid Huseynov 
Date:   2016-09-17T06:11:40Z

separate getAllNotes() and getAllNotes(subject)

commit 92f37f50c3205d1a231693131d25e31b339d9deb
Author: Khalid Huseynov 
Date:   2016-09-17T07:36:02Z

substitute old getAllNotes(subject) with new implementation

commit d9c3bc98cb799526d343ac7151877089de5d2f54
Author: Khalid Huseynov 
Date:   2016-09-17T08:22:48Z

filter reload using predicates

commit 139940743f73692c4e3febff56a342960dd80185
Author: Khalid Huseynov 
Date:   2016-09-19T00:23:43Z

remove unused imports

commit 09e67234821823f1e5cf2f4acf2329b36abdbed9
Author: Khalid Huseynov 
Date:   2016-09-19T05:46:06Z

notebookAuthorization as singleton

commit 537cc0ebadac7bce9a9eb00fe76cd3d8da17f220
Author: Khalid Huseynov 
Date:   2016-09-19T16:21:03Z

apply filter from authorization in sync

that's to sync only user workbench notes

commit 6614e2bb5224a4f408473cf0cbbd1f5fcb5b119c
Author: Khalid Huseynov 
Date:   2016-09-20T04:14:13Z

improve tests

commit 9427e6260ab5ed3cf8a606152f5361d7ec79
Author: Khalid Huseynov 
Date:   2016-09-20T10:04:25Z

multicast fine grained note lists to users instead of broadcast

commit 781207e15b44dd90506910e8eceab2f805250a15
Author: Khalid Huseynov 
Date:   2016-09-20T16:25:55Z

bugfix: reload only once

commit 17eae84c0c259c6181df2ac49a63349adced2ed7
Author: Khalid Huseynov 
Date:   2016-09-21T04:20:13Z

bugfix: add precondition for NP

commit 9cf1d8848ee02b5fafda1afd538b71befd3eb412
Author: Khalid Huseynov 
Date:   2016-09-23T09:13:51Z

fix init not to initialize every time

commit 094de2e61f7f341b0b7699f7b2f10fafc87ee29f
Author: Khalid Huseynov 
Date:   2016-09-27T03:56:29Z

broadcast note list on perm update - zeppelin-1438




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1392: [ZEPPELIN-1437] Multi-user note management - us...

2016-09-26 Thread khalidhuseynov
Github user khalidhuseynov closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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-26 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/1429
  
Tested, 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 #1334: [ZEPPELIN-682] New interpreter for Apache Beam (incuba...

2016-09-26 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1334
  
Thank you so much @mfelgamal !


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1334: [ZEPPELIN-682] New interpreter for Apache Beam ...

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

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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-26 Thread corneadoug
Github user corneadoug commented on the issue:

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


[GitHub] zeppelin issue #1451: [ZEPPELIN-1279] Zeppelin with CDH5.x docker document.

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

https://github.com/apache/zeppelin/pull/1451
  
@astroshim To make CI happy, you might need to exclude `scripts` dir  from 
the `rat-plugin`. Please add this single line`scripts/**` to 
[here](https://github.com/apache/zeppelin/blob/master/pom.xml#L748).


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1460: [ZEPPELIN-1490] pass subject while persisting n...

2016-09-26 Thread khalidhuseynov
GitHub user khalidhuseynov reopened a pull request:

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

[ZEPPELIN-1490]  pass subject while persisting note on running status change

### What is this PR for?
This PR makes sure correct subject passed to `note.persist`. The subject is 
obtained from the paragraph level `authInfo` based on the job running.


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

### Todos
* [x] - pass correct subject

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

### How should this be tested?
CI is green

### Screenshots (if appropriate)

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


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

$ git pull https://github.com/khalidhuseynov/incubator-zeppelin 
storage/run-subject-npe

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

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


commit 3df9442593c4f4a66e80b787041f3f26b86c4e74
Author: Khalid Huseynov 
Date:   2016-09-24T17:19:34Z

get subject from the paragph and job correspondence

commit 2a827efae04f26f005fb44340ce28765d9455213
Author: Khalid Huseynov 
Date:   2016-09-26T07:34:29Z

change AuthenticationInfo to have ANONYMOUS field

commit 8e740ffd0dd9b05ad24b9092faa24b4af4c626f3
Author: Khalid Huseynov 
Date:   2016-09-27T00:55:46Z

fix checkstyle




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1460: [ZEPPELIN-1490] pass subject while persisting n...

2016-09-26 Thread khalidhuseynov
Github user khalidhuseynov closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1453: [ZEPPELIN-1482] Load updated dependency library...

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

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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-26 Thread asfgit
Github user asfgit closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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-1493) %sql charting errors for negative values when using groupby

2016-09-26 Thread Dustin DeMeo (JIRA)
Dustin DeMeo created ZEPPELIN-1493:
--

 Summary: %sql charting errors for negative values when using 
groupby
 Key: ZEPPELIN-1493
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1493
 Project: Zeppelin
  Issue Type: Bug
  Components: GUI
Affects Versions: 0.6.1
 Environment: Google Compute, Linux Ubuntu 16.04, Zeppelin 0.6.1 
(download), Spark 2.0.0, Scala 2.11
Reporter: Dustin DeMeo
Priority: Minor


Issues when using groupby:
1) "Histogram" chart displays negative values after positive values
2) "Line" charts do not display information for negative values

To reproduce:
1) Create spark zeppelin notebook with default spark interpreter and type code:
{quote}
{{
%spark
val df = Seq((-2), (-1), (-1), (0), (0), (0), (1), (1), (2)).toDF("value")
df.createOrReplaceTempView("df")

%sql
SELECT value, COUNT(1) AS value
FROM   df
GROUP BY value
ORDER BY value
}}
{quote}

2) Check outputs by choosing different chart types. Note that the "table" 
correctly shows the correct values in order. The "histogram" chart treats the 
vales as if they were unsigned. Both "line" charts fail to display information 
for negative values (but are ordered correctly).



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


[GitHub] zeppelin issue #1460: [ZEPPELIN-1490] pass subject while persisting note on ...

2016-09-26 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1460
  
Could you make CI 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 #1334: [ZEPPELIN-682] New interpreter for Apache Beam (incuba...

2016-09-26 Thread mfelgamal
Github user mfelgamal commented on the issue:

https://github.com/apache/zeppelin/pull/1334
  
@bzz For me, no further discussion. looking forward to seeing the PR merged.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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-26 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1334
  
Thank you @mfelgamal !
Merging to master, if there is no further 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 #704: Update NVD3 library for supporting new features.

2016-09-26 Thread camilosampedro
Github user camilosampedro commented on the issue:

https://github.com/apache/zeppelin/pull/704
  
Of course, I understand. I will close it now. :smile: 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #704: Update NVD3 library for supporting new features.

2016-09-26 Thread camilosampedro
Github user camilosampedro closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1454: [ZEPPELIN-1483] Zeppelin home page list notebooks does...

2016-09-26 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/1454
  
Thank you @corneadoug, have fixed 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 #530: ZEPPELIN-497 ] pyspark completion

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

https://github.com/apache/zeppelin/pull/530
  
hi @fanlu - it's best to report issue to dev@zeppelin.apache.org



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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-26 Thread kavinkumarks
Github user kavinkumarks commented on the issue:

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

@corneadoug thanks for the update! Could you please take care of merging 
this?

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


[GitHub] zeppelin issue #1136: [ZEPPELIN-1121] Bugfix pyspark autocompletion.

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

https://github.com/apache/zeppelin/pull/1136
  
@fanlu Thank you for share your problem. 
How did you build master? Did you add `-Ppyspark` to use pyspark on your 
build command?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1136: [ZEPPELIN-1121] Bugfix pyspark autocompletion.

2016-09-26 Thread fanlu
Github user fanlu commented on the issue:

https://github.com/apache/zeppelin/pull/1136
  
I still have this problem.
build master version
ERROR [2016-09-26 22:02:25,834] ({qtp532445947-596} 
NotebookServer.java[onMessage]:257) - Can't handle message
org.apache.zeppelin.interpreter.InterpreterException: 
org.apache.thrift.transport.TTransportException
at 
org.apache.zeppelin.interpreter.remote.RemoteInterpreter.completion(RemoteInterpreter.java:453)
at 
org.apache.zeppelin.interpreter.LazyOpenInterpreter.completion(LazyOpenInterpreter.java:122)
at org.apache.zeppelin.notebook.Paragraph.completion(Paragraph.java:242)
at org.apache.zeppelin.notebook.Note.completion(Note.java:520)
at 
org.apache.zeppelin.socket.NotebookServer.completion(NotebookServer.java:793)
at 
org.apache.zeppelin.socket.NotebookServer.onMessage(NotebookServer.java:216)
at 
org.apache.zeppelin.socket.NotebookSocket.onWebSocketText(NotebookSocket.java:56)
at 
org.eclipse.jetty.websocket.common.events.JettyListenerEventDriver.onTextMessage(JettyListenerEventDriver.java:128)
at 
org.eclipse.jetty.websocket.common.message.SimpleTextMessage.messageComplete(SimpleTextMessage.java:69)
at 
org.eclipse.jetty.websocket.common.events.AbstractEventDriver.appendMessage(AbstractEventDriver.java:65)
at 
org.eclipse.jetty.websocket.common.events.JettyListenerEventDriver.onTextFrame(JettyListenerEventDriver.java:122)
at 
org.eclipse.jetty.websocket.common.events.AbstractEventDriver.incomingFrame(AbstractEventDriver.java:161)
at 
org.eclipse.jetty.websocket.common.WebSocketSession.incomingFrame(WebSocketSession.java:309)
at 
org.eclipse.jetty.websocket.common.extensions.ExtensionStack.incomingFrame(ExtensionStack.java:214)
at 
org.eclipse.jetty.websocket.common.Parser.notifyFrame(Parser.java:220)
at org.eclipse.jetty.websocket.common.Parser.parse(Parser.java:258)
at 
org.eclipse.jetty.websocket.common.io.AbstractWebSocketConnection.readParse(AbstractWebSocketConnection.java:632)
at 
org.eclipse.jetty.websocket.common.io.AbstractWebSocketConnection.onFillable(AbstractWebSocketConnection.java:480)
at 
org.eclipse.jetty.io.AbstractConnection$2.run(AbstractConnection.java:544)
at 
org.eclipse.jetty.util.thread.QueuedThreadPool.runJob(QueuedThreadPool.java:635)
at 
org.eclipse.jetty.util.thread.QueuedThreadPool$3.run(QueuedThreadPool.java:555)
at java.lang.Thread.run(Thread.java:745)
Caused by: org.apache.thrift.transport.TTransportException
at 
org.apache.thrift.transport.TIOStreamTransport.read(TIOStreamTransport.java:132)
at org.apache.thrift.transport.TTransport.readAll(TTransport.java:86)
at 
org.apache.thrift.protocol.TBinaryProtocol.readAll(TBinaryProtocol.java:429)
at 
org.apache.thrift.protocol.TBinaryProtocol.readI32(TBinaryProtocol.java:318)
at 
org.apache.thrift.protocol.TBinaryProtocol.readMessageBegin(TBinaryProtocol.java:219)
at org.apache.thrift.TServiceClient.receiveBase(TServiceClient.java:69)
at 
org.apache.zeppelin.interpreter.thrift.RemoteInterpreterService$Client.recv_completion(RemoteInterpreterService.java:358)
at 
org.apache.zeppelin.interpreter.thrift.RemoteInterpreterService$Client.completion(RemoteInterpreterService.java:342)
at 
org.apache.zeppelin.interpreter.remote.RemoteInterpreter.completion(RemoteInterpreter.java:449)
... 21 more



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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-26 Thread fanlu
Github user fanlu commented on the issue:

https://github.com/apache/zeppelin/pull/530
  
version 0.7.0-SNAPSHOT
build by mvn clean package -Pbuild-distr -Pspark-2.0 -Dspark.version=2.0.0 
-Pscala-2.11 -Phadoop-2.6 -Dhadoop.version=2.6.0-cdh5.7.0 -Pyarn -Psparkr 
-Ppyspark -DskipTests -Drat.skip=true -U
when I use ctrl+. the stacktrace is
ERROR [2016-09-26 22:02:25,834] ({qtp532445947-596} 
NotebookServer.java[onMessage]:257) - Can't handle message
org.apache.zeppelin.interpreter.InterpreterException: 
org.apache.thrift.transport.TTransportException
at 
org.apache.zeppelin.interpreter.remote.RemoteInterpreter.completion(RemoteInterpreter.java:453)
at 
org.apache.zeppelin.interpreter.LazyOpenInterpreter.completion(LazyOpenInterpreter.java:122)
at org.apache.zeppelin.notebook.Paragraph.completion(Paragraph.java:242)
at org.apache.zeppelin.notebook.Note.completion(Note.java:520)
at 
org.apache.zeppelin.socket.NotebookServer.completion(NotebookServer.java:793)
at 
org.apache.zeppelin.socket.NotebookServer.onMessage(NotebookServer.java:216)
at 
org.apache.zeppelin.socket.NotebookSocket.onWebSocketText(NotebookSocket.java:56)
at 
org.eclipse.jetty.websocket.common.events.JettyListenerEventDriver.onTextMessage(JettyListenerEventDriver.java:128)
at 
org.eclipse.jetty.websocket.common.message.SimpleTextMessage.messageComplete(SimpleTextMessage.java:69)
at 
org.eclipse.jetty.websocket.common.events.AbstractEventDriver.appendMessage(AbstractEventDriver.java:65)
at 
org.eclipse.jetty.websocket.common.events.JettyListenerEventDriver.onTextFrame(JettyListenerEventDriver.java:122)
at 
org.eclipse.jetty.websocket.common.events.AbstractEventDriver.incomingFrame(AbstractEventDriver.java:161)
at 
org.eclipse.jetty.websocket.common.WebSocketSession.incomingFrame(WebSocketSession.java:309)
at 
org.eclipse.jetty.websocket.common.extensions.ExtensionStack.incomingFrame(ExtensionStack.java:214)
at 
org.eclipse.jetty.websocket.common.Parser.notifyFrame(Parser.java:220)
at org.eclipse.jetty.websocket.common.Parser.parse(Parser.java:258)
at 
org.eclipse.jetty.websocket.common.io.AbstractWebSocketConnection.readParse(AbstractWebSocketConnection.java:632)
at 
org.eclipse.jetty.websocket.common.io.AbstractWebSocketConnection.onFillable(AbstractWebSocketConnection.java:480)
at 
org.eclipse.jetty.io.AbstractConnection$2.run(AbstractConnection.java:544)
at 
org.eclipse.jetty.util.thread.QueuedThreadPool.runJob(QueuedThreadPool.java:635)
at 
org.eclipse.jetty.util.thread.QueuedThreadPool$3.run(QueuedThreadPool.java:555)
at java.lang.Thread.run(Thread.java:745)
Caused by: org.apache.thrift.transport.TTransportException
at 
org.apache.thrift.transport.TIOStreamTransport.read(TIOStreamTransport.java:132)
at org.apache.thrift.transport.TTransport.readAll(TTransport.java:86)
at 
org.apache.thrift.protocol.TBinaryProtocol.readAll(TBinaryProtocol.java:429)
at 
org.apache.thrift.protocol.TBinaryProtocol.readI32(TBinaryProtocol.java:318)
at 
org.apache.thrift.protocol.TBinaryProtocol.readMessageBegin(TBinaryProtocol.java:219)
at org.apache.thrift.TServiceClient.receiveBase(TServiceClient.java:69)
at 
org.apache.zeppelin.interpreter.thrift.RemoteInterpreterService$Client.recv_completion(RemoteInterpreterService.java:358)
at 
org.apache.zeppelin.interpreter.thrift.RemoteInterpreterService$Client.completion(RemoteInterpreterService.java:342)
at 
org.apache.zeppelin.interpreter.remote.RemoteInterpreter.completion(RemoteInterpreter.java:449)
... 21 more


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1390: [WIP] [ZEPPELIN-1339] [ZEPPELIN-1000] [ZEPPELIN-1338] ...

2016-09-26 Thread echarles
Github user echarles commented on the issue:

https://github.com/apache/zeppelin/pull/1390
  
I have a prototype that supports the 3 modes. Give me a few days to polish 
and commit.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1390: [WIP] [ZEPPELIN-1339] [ZEPPELIN-1000] [ZEPPELIN-1338] ...

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

https://github.com/apache/zeppelin/pull/1390
  
I didn't look through the PR, but according @jongyoul 's last comments. If 
only `isolated` mode is supported, then I don't think is a good idea.  We 
should not only consider isolatation of each user, but also need to take 
considerion of collobration. The most important thing is about how to share 
data between users. I think that is why we introduce `scoped` mode.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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-1492) Web Socket not updating notebook content correctly

2016-09-26 Thread Aleksandar Buha (JIRA)
Aleksandar Buha created ZEPPELIN-1492:
-

 Summary: Web Socket not updating notebook content correctly
 Key: ZEPPELIN-1492
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1492
 Project: Zeppelin
  Issue Type: Bug
 Environment: Google Chrome  53.0.2785.116 m
Zeppelin Version 0.6.1 
Reporter: Aleksandar Buha


While accessing and editing the same notebook via two web-browser windows, 
synchronization is failing if I do two consecutive updates from one of the 
browser windows.

This issue can be reproduced with the following two browser-windows and a 
single notebook. 

Browser A:
%pyspark
print "Original Zeppelin Notebook"

In Browser B) edit the notebook and the above command to: 

%pyspark
print "Notebook is updated"

If I run the notebook via browser A followed by browser B, everything is 
updated nicely. However, if I add the following line to the Notebook though 
Browser B:

print "once again"

and run the notebook through Browser B again, the content in Browser A is not 
updated.

Inside WebSocket object (Browser A) I can see that "op":"NOTE" has attribute 
"text":"%pyspark\nprint \"Notebook is updated\"\nprint \"once again\"". 
Unfortunately, I do not see the second print statement displayed on screen.



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


[GitHub] zeppelin issue #1230: [ZEPPELIN-606] Add a button to explicitly save a noteb...

2016-09-26 Thread rerngvit
Github user rerngvit commented on the issue:

https://github.com/apache/zeppelin/pull/1230
  
I agree with you @corneadoug 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1230: [ZEPPELIN-606] Add a button to explicitly save ...

2016-09-26 Thread rerngvit
Github user rerngvit closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1324: Update AWS S3 library to avoid old Jackson

2016-09-26 Thread iasandcb
Github user iasandcb commented on the issue:

https://github.com/apache/zeppelin/pull/1324
  
@corneadoug I see. I'll close this.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1324: Update AWS S3 library to avoid old Jackson

2016-09-26 Thread iasandcb
Github user iasandcb closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #262: ZEPPELIN-268 : Adding Data Validation Feature and Test ...

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

https://github.com/apache/zeppelin/pull/262
  
@Madhuka Should we 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.
---


[jira] [Created] (ZEPPELIN-1491) Notebook Emergency Mode가 있으면 좋겠습니다.

2016-09-26 Thread SanAh Kang (JIRA)
SanAh Kang created ZEPPELIN-1491:


 Summary: Notebook Emergency Mode가 있으면 좋겠습니다.
 Key: ZEPPELIN-1491
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1491
 Project: Zeppelin
  Issue Type: Bug
Affects Versions: 0.6.1
Reporter: SanAh Kang
Priority: Critical


paragraph setting을 잘못 건드리면 hang 현상에 빠지는데 이를 해결하기 위한 응급조치 모드가 있으면 좋을 것 같아요.

현재 해결 방법은
1. IE의 '스크립트 중지'기능이 제대로 작동하거나 (IE가 죽지 않는 가정 하에)
2. notebook의 note.json파일을 수정하거나,
3. data 처리 부분을 서버단에서 강제 수정

이를 해결할 수 있는 설정이 어딘가에 있으면 좋겠습니다.
해당 notebook의 모든 paragraph output이나 processing을 disable시키고 datagrid로 설정을 강제로 
바꾼다는 등의 응급조치 모드...

[~Ahyoung] [~moon]



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


[GitHub] zeppelin issue #576: ZEPPELIN-539 RemoteInterpreter Heartbeat (WIP)

2016-09-26 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue:

https://github.com/apache/zeppelin/pull/576
  
OK. This is already staled. I'll close this. 
@corneadoug Thanks for noticing!


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #576: ZEPPELIN-539 RemoteInterpreter Heartbeat (WIP)

2016-09-26 Thread HeartSaVioR
Github user HeartSaVioR closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #819: Fix compilation issues on branch 0.5.6 by backpo...

2016-09-26 Thread jersub
Github user jersub closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #819: Fix compilation issues on branch 0.5.6 by backporting f...

2016-09-26 Thread jersub
Github user jersub commented on the issue:

https://github.com/apache/zeppelin/pull/819
  
Fine, let's reopen it if anyone is interested.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1302: [ZEPPELIN-1306] Interpreter restarts on a note.

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

https://github.com/apache/zeppelin/pull/1302
  
@corneadoug I'm going to add test case soon. :laughing: 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #576: ZEPPELIN-539 RemoteInterpreter Heartbeat (WIP)

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

https://github.com/apache/zeppelin/pull/576
  
@HeartSaVioR It's been quite some time, should we 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 #704: Update NVD3 library for supporting new features.

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

https://github.com/apache/zeppelin/pull/704
  
@camilosampedro There is still no plan on upgrading nvd3 for now, can we 
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 pull request #1460: [ZEPPELIN-1490] pass subject while persisting n...

2016-09-26 Thread khalidhuseynov
GitHub user khalidhuseynov opened a pull request:

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

[ZEPPELIN-1490]  pass subject while persisting note on running status change

### What is this PR for?
This PR makes sure correct subject passed to `note.persist`. The subject is 
obtained from the paragraph level `authInfo` based on the job running.


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

### Todos
* [x] - pass correct subject

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

### How should this be tested?
CI is green

### Screenshots (if appropriate)

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


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

$ git pull https://github.com/khalidhuseynov/incubator-zeppelin 
storage/run-subject-npe

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

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


commit 3df9442593c4f4a66e80b787041f3f26b86c4e74
Author: Khalid Huseynov 
Date:   2016-09-24T17:19:34Z

get subject from the paragph and job correspondence

commit 2a827efae04f26f005fb44340ce28765d9455213
Author: Khalid Huseynov 
Date:   2016-09-26T07:34:29Z

change AuthenticationInfo to have ANONYMOUS field




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #819: Fix compilation issues on branch 0.5.6 by backporting f...

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

https://github.com/apache/zeppelin/pull/819
  
@jersub Can we 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 #1176: [Zeppelin 1165 : improve] Paragraph dynamic relationsh...

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

https://github.com/apache/zeppelin/pull/1176
  
@cloverhearts If I understand, the plan is to open a new PR for this 
feature?
Can it be closed 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 #1230: [ZEPPELIN-606] Add a button to explicitly save a noteb...

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

https://github.com/apache/zeppelin/pull/1230
  
@rerngvit If any more arguments for this feature, I'd love to hear it, 
otherwise we would better 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 #1324: Update AWS S3 library to avoid old Jackson

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

https://github.com/apache/zeppelin/pull/1324
  
@iasandcb Do you want to continue it later when you have some time, or just 
close this PR?
Otherwise you can also create a JIRA issue, and somebody can pick up the 
issue


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1309: [ZEPPELIN-1314] dump out the R command in debug mode

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

https://github.com/apache/zeppelin/pull/1309
  
@snowch Not sure how much dumping out the R command would help diagnose. Do 
you meet any case that failed to launch the R process ? If you want to diagnose 
the R process after it is launched. You can refer SparkOutputStream which will 
print output in debug level.  You just need add the following line in 
log4j.properties.
```
log4j.logger.org.apache.zeppelin.spark.SparkOutputStream=DEBUG
```




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1452: ZEPPELIN-1442. UDF can not be found due to 2 instances...

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

https://github.com/apache/zeppelin/pull/1452
  
@minahlee  PR is rebased, and the failed test is irrelevant.

```
- should provide onclick method *** FAILED ***
  The code passed to eventually never returned normally. Attempted 1 
times over 325.359079 milliseconds. Last failure message: 0 was not equal to 1. 
(AbstractAngularElemTest.scala:72)
AngularElem
```


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1324: Update AWS S3 library to avoid old Jackson

2016-09-26 Thread iasandcb
Github user iasandcb commented on the issue:

https://github.com/apache/zeppelin/pull/1324
  
@corneadoug no update since I have no time to spend on this issue.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1323: [HOTFIX] Add logins authentication for hdfs file inter...

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

https://github.com/apache/zeppelin/pull/1323
  
@cloverhearts @jongyoul Any way to try this one out?


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


[GitHub] zeppelin issue #1324: Update AWS S3 library to avoid old Jackson

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

https://github.com/apache/zeppelin/pull/1324
  
@iasandcb any update on this issue?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1357: ZEPPELIN-1364: Return only the ran paragraph details f...

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

https://github.com/apache/zeppelin/pull/1357
  
@vipul1409 Can you rebase this PR?
I will run a few tests to see if all the paragraph updates are handled here.


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


[jira] [Created] (ZEPPELIN-1490) Persisting note while changing status on paragraph run doesn't pass subject

2016-09-26 Thread Khalid Huseynov (JIRA)
Khalid Huseynov created ZEPPELIN-1490:
-

 Summary: Persisting note while changing status on paragraph run 
doesn't pass subject
 Key: ZEPPELIN-1490
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1490
 Project: Zeppelin
  Issue Type: Bug
Reporter: Khalid Huseynov


Zeppelin job listener persists note every time there's change on run status. 
However as you can see 
[here|https://github.com/apache/zeppelin/blob/master/zeppelin-server/src/main/java/org/apache/zeppelin/socket/NotebookServer.java#L1448]
 it doesn't pass correct subject resulting in possible NPEs. 



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


[GitHub] zeppelin issue #1375: [ZEPPELIN-1171] Only retrieve resources if client is s...

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

https://github.com/apache/zeppelin/pull/1375
  
@lresende @jongyoul Any news on this issue


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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-26 Thread meenakshisekar
Github user meenakshisekar commented on the issue:

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

I'm working towards that. Shall let you know once it is 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 #1429: ZEPPELIN-1440 Notebook clone: prefix name with "Copy o...

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

https://github.com/apache/zeppelin/pull/1429
  
@r-kamath Could you just rebase once more? I will make a final check after


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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-26 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/1445
  
@meenakshisekar Could you take care of 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 #1459: pass subject to get method in sync

2016-09-26 Thread khalidhuseynov
GitHub user khalidhuseynov opened a pull request:

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

pass subject to get method in sync

### What is this PR for?
This is to resolve possible NPEs in storage mechanism related to empty 
subject

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

### Todos
* [x] - pass `subject` instead of `null`

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

### How should this be tested?
storage tests pass, CI green

### Screenshots (if appropriate)

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



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

$ git pull https://github.com/khalidhuseynov/incubator-zeppelin 
fix/storage-npe

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

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


commit 4a59723ca3f50a9eced3f318159d8ce48a52b519
Author: Khalid Huseynov 
Date:   2016-09-22T08:13:31Z

pass subject to get method in sync




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1454: [ZEPPELIN-1483] Zeppelin home page list notebooks does...

2016-09-26 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/1454
  
Ready for review.
CI fails for #7346.9 with; which is unrelated

```
- should provide onclick method *** FAILED ***
  The code passed to eventually never returned normally. Attempted 1 
times over 310.82543 milliseconds. Last failure message: 0 was not equal to 1. 
(AbstractAngularElemTest.scala:72)
AngularElem
- should print angular display directive only once in a paragraph
AngularElem
- should bind angularObject to ng-model directive
AngularElem
- should able to disassociate AngularObjects
AngularElem
- should allow access to InterpreterContext inside of callback 
function
AngularElem
- should able to be created from implicit conversion
AngularElemTest:
AngularElem
- should provide onclick method
AngularElem
- should print angular display directive only once in a paragraph
AngularElem
- should bind angularObject to ng-model directive
AngularElem
- should able to disassociate AngularObjects
AngularElem
- should allow access to InterpreterContext inside of callback 
function
AngularElem
- should able to be created from implicit conversion
Run completed in 1 second, 221 milliseconds.
Total number of tests run: 18
Suites: completed 5, aborted 0
Tests: succeeded 17, failed 1, canceled 0, ignored 0, pending 0
*** 1 TEST FAILED ***
[INFO] 

[INFO] Reactor Summary:
[INFO] 
[INFO] Zeppelin: Interpreter .. SUCCESS [ 
12.937 s]
[INFO] Zeppelin: Zengine .. SUCCESS [  
5.803 s]
[INFO] Zeppelin: Display system apis .. FAILURE [  
2.906 s]
[INFO] Zeppelin: Spark dependencies ... SKIPPED
[INFO] Zeppelin: Spark  SKIPPED
[INFO] Zeppelin: Server ... SKIPPED
[INFO] 

[INFO] BUILD FAILURE
[INFO] 

[INFO] Total time: 22.748 s
[INFO] Finished at: 2016-09-25T05:00:13+00:00
[INFO] Final Memory: 30M/463M
[INFO] 

```



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1135: Add force layout visualization in paragraphs for SQL q...

2016-09-26 Thread davidtzoor
Github user davidtzoor commented on the issue:

https://github.com/apache/zeppelin/pull/1135
  
@corneadoug any update on my PR? looking forward to your feedback


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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-1489) Automatic prefix doesn't work with "%jdbc(prefix)"

2016-09-26 Thread Ahyoung Ryu (JIRA)
Ahyoung Ryu created ZEPPELIN-1489:
-

 Summary: Automatic prefix doesn't work with "%jdbc(prefix)"
 Key: ZEPPELIN-1489
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1489
 Project: Zeppelin
  Issue Type: Bug
Reporter: Ahyoung Ryu
 Attachments: hive.gif, jdbc(hive).gif

Currently when users run a paragraph, a new one will be automatically added to 
right below with "%prefix". But if I use "%jdbc(prefix)", the prefix doesn't 
show up in the new paragraph. Please see the attached gif images.



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


[GitHub] zeppelin issue #1453: [ZEPPELIN-1482] Load updated dependency library on int...

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

https://github.com/apache/zeppelin/pull/1453
  
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 #1452: ZEPPELIN-1442. UDF can not be found due to 2 instances...

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

https://github.com/apache/zeppelin/pull/1452
  
@zjffdu could you rebase and resolve conflicts?


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


[jira] [Created] (ZEPPELIN-1488) JDBC Intrepreter throws error while the interpreter is downloading dependencies

2016-09-26 Thread Rajarajan G (JIRA)
Rajarajan G created ZEPPELIN-1488:
-

 Summary: JDBC Intrepreter throws error while the interpreter is 
downloading dependencies
 Key: ZEPPELIN-1488
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1488
 Project: Zeppelin
  Issue Type: Improvement
  Components: Interpreters
Affects Versions: 0.6.0
Reporter: Rajarajan G






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


[jira] [Created] (ZEPPELIN-1487) Storage layer possible NPE exception because of empty subject

2016-09-26 Thread Khalid Huseynov (JIRA)
Khalid Huseynov created ZEPPELIN-1487:
-

 Summary: Storage layer possible NPE exception because of empty 
subject
 Key: ZEPPELIN-1487
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1487
 Project: Zeppelin
  Issue Type: Bug
Reporter: Khalid Huseynov
 Fix For: 0.7.0


Storage layer sync mechanism is passing subject as null 
[here|https://github.com/apache/zeppelin/blob/master/zeppelin-zengine/src/main/java/org/apache/zeppelin/notebook/repo/NotebookRepoSync.java#L271].
 That may cause NPE exception for multiuser environments



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