[jira] [Created] (ZEPPELIN-1275) Comment code of PySparkInterpreter is not highlighted

2016-08-02 Thread Jeff Zhang (JIRA)
Jeff Zhang created ZEPPELIN-1275:


 Summary: Comment code of PySparkInterpreter is not highlighted 
 Key: ZEPPELIN-1275
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1275
 Project: Zeppelin
  Issue Type: Improvement
Affects Versions: 0.6.0
Reporter: Jeff Zhang
Priority: Minor






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


[GitHub] zeppelin issue #1231: [Zeppelin - 683][WIP] BittorrentNotebookRepo

2016-08-02 Thread onkarshedge
Github user onkarshedge commented on the issue:

https://github.com/apache/zeppelin/pull/1231
  
And also a note on directories..
1] torrent
   notes/
   17fa195f6c85f72d197d63896b15f456e624/note.json
   282f928f9429828e5ef0a422f13d3263e671a2d3/note.json
   resume/
   torrents/
   settings.dat

The torrent directory is created which will contain the following 
directories. When the user wants to share a note via DHT he will give the 
noteId , the note.json file is copied to torrent/notes//
and then seeded by adding to TorrentEngine.
The reason to copy before seeding is if note is changed or updated after 
added to seeding, the pieces hash will change and hence corrupt. The torrent 
state will change from seeding --> downloading because it doesn't have that 
file.



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

2016-08-02 Thread onkarshedge
Github user onkarshedge commented on the issue:

https://github.com/apache/zeppelin/pull/1231
  
@bzz here is the diagram.
![torrent 
1](https://cloud.githubusercontent.com/assets/10570922/17356236/efedf040-5973-11e6-8a1c-06ffe4c57ff1.png)
[Link to drive] 
(https://drive.google.com/file/d/0Bwg4NhDtfdM9WFRQSGJVV09Ub1U/view?usp=sharing)


![torrentclass](https://cloud.githubusercontent.com/assets/10570922/17356237/eff419e8-5973-11e6-8b4d-c97abf76e86c.png)
[Link to 
drive](https://drive.google.com/file/d/0Bwg4NhDtfdM9UFhkMDItQlVFaHM/view?usp=sharing)



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1272: Remove empty user when put permission

2016-08-02 Thread zhangmq
GitHub user zhangmq opened a pull request:

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

Remove empty user when put permission

### What is this PR for?
check empty users and delete it.  

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

### Todos

### What is the Jira issue?

### How should this be tested?
clear note permissions then your note can't read.

### 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/zhangmq/zeppelin master

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

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


commit 61bd580d4285135855d6610b83b367f19c5426ff
Author: zhangmengqi 
Date:   2016-08-03T06:24:48Z

remve empty user when put permission




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1271: [ZEPPELIN-1274]Write "Spark SQL" in docs rather than "...

2016-08-02 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1271
  
@sarutak Looks good. Thanks! 


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


[GitHub] zeppelin issue #1271: [ZEPPELIN-1274]Write "Spark SQL" in docs rather than "...

2016-08-02 Thread sarutak
Github user sarutak commented on the issue:

https://github.com/apache/zeppelin/pull/1271
  
@AhyoungRyu Thanks for the notification. I'll fix them soon.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1271: [ZEPPELIN-1274]Write "Spark SQL" in docs rather than "...

2016-08-02 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1271
  
@sarutak Thanks for fixing this. I found more `SparkSQL` in the below 2 
files. 
 - 
[docs/manual/dynamicform.md#using-form-templates](https://github.com/apache/zeppelin/blob/master/docs/manual/dynamicform.md#using-form-templates)
 - 
[docs/interpreter/livy.md#configuration](https://github.com/apache/zeppelin/blob/master/docs/interpreter/livy.md#configuration)

Could you change them as well? :)


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

2016-08-02 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/1258
  
@cloverhearts Thanks for the changes and keeping this PR focused.
I saw some great improvements with big Notebook with those changes.
@Leemoonsoo Can you try it too and tell us if you also think its better?


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


[GitHub] zeppelin issue #1270: [ZEPPELIN-1273] Use Math.abs to determine if custom fo...

2016-08-02 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1270
  
Looks great to me, thank you @praagarw 
Merging to only to master, if there is no further discussion

Or do you think we should also apply this to upcoming 0.6.1 as well?


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

2016-08-02 Thread bzz
Github user bzz commented on the issue:

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


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


[GitHub] zeppelin pull request #1271: [Z

2016-08-02 Thread sarutak
GitHub user sarutak opened a pull request:

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

[Z

### What is this PR for?
Some of the doc files say "SparkSQL" but the collect spelling is "Spark 
SQL" (need a white space between "Spark" and "SQL").
Lets's replace it with the collect one.


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

### Todos
* [ ] - Task

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

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

### Screenshots (if appropriate)

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



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

$ git pull https://github.com/sarutak/zeppelin ZEPPELIN-1274

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

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


commit 14aa2b760c256a3a862342bdca365bcecbea7072
Author: Kousuke Saruta 
Date:   2016-08-03T05:37:28Z

Replaced 'SparkSQL' in docs into 'Spark SQL'




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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-1274) Write "Spark SQL" in docs rather than "SparkSQL"

2016-08-02 Thread Kousuke Saruta (JIRA)
Kousuke Saruta created ZEPPELIN-1274:


 Summary: Write "Spark SQL" in docs rather than "SparkSQL"
 Key: ZEPPELIN-1274
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1274
 Project: Zeppelin
  Issue Type: Improvement
  Components: documentation
Affects Versions: 0.7.0
Reporter: Kousuke Saruta
Priority: Minor


Some of the doc files say "SparkSQL" but the collect spelling is "Spark SQL" 
(need a white space between "Spark" and "SQL").

Lets's replace it with the collect one.




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


[GitHub] zeppelin issue #1263: Small cleanup of zeppelin-server tests

2016-08-02 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1263
  
CI failure

```
Results :

Failed tests: 
  ZeppelinSparkClusterTest.sparkRTest:116 expected:<[[1] 3]> but 
was:<[Error in getSparkSession(): SparkSession not initialized


Error in (function (classes, fdef, mtable) : unable to find an 
inherited method for function 'count' for signature '"function"'
]>

Tests run: 65, Failures: 1, Errors: 0, Skipped: 0


Fixed in #1259 so rebasing 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 #1258: [ ZEPPELIN-1266 ] Code editor Optimization

2016-08-02 Thread cloverhearts
Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/1258
  
@bzz 
I removed the parts which you say.
Thank you.


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


[GitHub] zeppelin issue #1258: [ ZEPPELIN-1266 ] Code editor Optimization

2016-08-02 Thread cloverhearts
Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/1258
  
@corneadoug 
I did modification.
Please review for 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 #1267: [ZEPPELIN-1261] Bug fix in z.show() for matplot...

2016-08-02 Thread asfgit
Github user asfgit closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1248: [ZEPPELIN-1069]Ignore implicit interpreter when user e...

2016-08-02 Thread mwkang
Github user mwkang commented on the issue:

https://github.com/apache/zeppelin/pull/1248
  
I refactor method name.


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

2016-08-02 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1266
  
@bzz Really appreciate for your all precise comments! Updated several docs 
as you suggested :)


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1270: [ZEPPELIN-1273] Use Math.abs to determine if cu...

2016-08-02 Thread prasadwagle
GitHub user prasadwagle opened a pull request:

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

[ZEPPELIN-1273] Use Math.abs to determine if custom formatter should be 
called for y axis values

### What is this PR for?
Use Math.abs to determine if custom formatter should be called for y axis 
values.


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

### Todos
* [ ] - Task

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

### How should this be tested?
Create simple table with two columns - date in one and large negative 
numbers like -1000 in other. Display line chart and see that the y axis 
values are formatted like -10M.

### 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/prasadwagle/incubator-zeppelin ZEPPELIN-1273

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

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


commit 06e12f88e3d39f6b090ad8c9fc346491f9e79908
Author: Prasad Wagle 
Date:   2016-08-03T03:57:13Z

Use Math.abs to determine if custom formatter should be called for y axis 
values




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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-1273) d3.format not called for large negative numbers

2016-08-02 Thread Prasad Wagle (JIRA)
Prasad Wagle created ZEPPELIN-1273:
--

 Summary: d3.format not called for large negative numbers
 Key: ZEPPELIN-1273
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1273
 Project: Zeppelin
  Issue Type: Bug
  Components: front-end
Reporter: Prasad Wagle
Assignee: Prasad Wagle
Priority: Minor






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


[GitHub] zeppelin issue #1207: [DOC][ZEPPELIN-1209] Remove a useless sentence about d...

2016-08-02 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1207
  
CI build failure is irrelevant with this PR i think. Can this be merged 
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 pull request #1268: minor doc fix for r.md

2016-08-02 Thread asfgit
Github user asfgit closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1259: [ZEPPELIN-1260] R interpreter doesn't work with...

2016-08-02 Thread asfgit
Github user asfgit closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1268: minor doc fix for r.md

2016-08-02 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1268
  
@zjffdu you have good eyes :) Looks good.


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


[GitHub] zeppelin issue #1267: [ZEPPELIN-1261] Bug fix in z.show() for matplotlib gra...

2016-08-02 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1267
  
 Sounds great! Apparently CI failure has been addressed in #1259 as well


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #932: Replace CXF with Jersey2 [ZEPPELIN-903]

2016-08-02 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/932
  
A quick search over JIRA suggests that the failure you reffer was taked 
care of in [ZEPPELIN-1264](https://issues.apache.org/jira/browse/ZEPPELIN-1264) 
and is resolved in latest master, please rebase.


In PR body 

> Does the licenses files need update? N

I think it does

@echarles all transitive dependencies lie `woodstox` and 
`org.glassfish.jersey.bundles` and `javax.validation`, etc  will be included in 
convenience binary for release so each of them must be as well included in 
`zeppelin-distribution/src/bin-license/LICENSE` according to their license. 
Could you please double check that?



>Jersey2 can better manage the JSON bodies and mappings.

Could you please elaborate on that part? People might not be very familiar 
with it very well, so if you could put the benefits of such change in PR 
description - that would help a lot.

Thank you!


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


[GitHub] zeppelin issue #1267: [ZEPPELIN-1261] Bug fix in z.show() for matplotlib gra...

2016-08-02 Thread bustios
Github user bustios commented on the issue:

https://github.com/apache/zeppelin/pull/1267
  
I see. Thanks for your help and information @bzz. I'll keep it in mind for 
subsequent PRs


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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-1272) Zeppelin spark innterpreters throws java.lang.StackOverflowError after running a while

2016-08-02 Thread lichenglin (JIRA)
lichenglin created ZEPPELIN-1272:


 Summary: Zeppelin spark innterpreters throws 
java.lang.StackOverflowError after running a while
 Key: ZEPPELIN-1272
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1272
 Project: Zeppelin
  Issue Type: Bug
  Components: Interpreters
Affects Versions: 0.6.0
Reporter: lichenglin


I'm using Zeppelin  to run some spark job with crontab.

But In the last days,I found Zeppelin's spark interpreter shut down almost 
every day once or more.

And here is the log 
{code}
java.lang.StackOverflowError
at scala.reflect.internal.Types$TypeMap.mapOver(Types.scala:4183)
at scala.reflect.internal.Types$AsSeenFromMap.apply(Types.scala:4638)
at scala.reflect.internal.Types$TypeMap.mapOver(Types.scala:4197)
at scala.reflect.internal.Types$AsSeenFromMap.apply(Types.scala:4638)
at scala.reflect.internal.Types$Type.asSeenFrom(Types.scala:754)
at scala.reflect.internal.Types$Type.computeMemberType(Types.scala:788)
at 
scala.reflect.internal.Symbols$MethodSymbol.typeAsMemberOf(Symbols.scala:2655)
at scala.reflect.internal.Types$Type.memberType(Types.scala:779)
at 
scala.reflect.internal.Types$class.defineUnderlyingOfSingleType(Types.scala:1534)
at 
scala.reflect.internal.SymbolTable.defineUnderlyingOfSingleType(SymbolTable.scala:13)
at scala.reflect.internal.Types$SingleType.underlying(Types.scala:1486)
at scala.reflect.internal.Types$SingletonType.widen(Types.scala:1340)
at 
scala.reflect.internal.Types$AsSeenFromMap.toPrefix$1(Types.scala:4541)
at scala.reflect.internal.Types$AsSeenFromMap.apply(Types.scala:4556)
at scala.reflect.internal.Types$TypeMap.mapOver(Types.scala:4183)
at scala.reflect.internal.Types$AsSeenFromMap.apply(Types.scala:4638)
at scala.reflect.internal.Types$TypeMap.mapOver(Types.scala:4197)
at scala.reflect.internal.Types$AsSeenFromMap.apply(Types.scala:4638)
at scala.reflect.internal.Types$Type.asSeenFrom(Types.scala:754)
at scala.reflect.internal.Types$Type.computeMemberType(Types.scala:788)
at 
scala.reflect.internal.Symbols$MethodSymbol.typeAsMemberOf(Symbols.scala:2655)
at scala.reflect.internal.Types$Type.memberType(Types.scala:779)
at 
scala.reflect.internal.Types$class.defineUnderlyingOfSingleType(Types.scala:1534)
at 
scala.reflect.internal.SymbolTable.defineUnderlyingOfSingleType(SymbolTable.scala:13)
at scala.reflect.internal.Types$SingleType.underlying(Types.scala:1486)
at scala.reflect.internal.Types$SingletonType.widen(Types.scala:1340)
at 
scala.reflect.internal.Types$AsSeenFromMap.toPrefix$1(Types.scala:4541)
at scala.reflect.internal.Types$AsSeenFromMap.apply(Types.scala:4556)
at scala.reflect.internal.Types$TypeMap.mapOver(Types.scala:4183)
at scala.reflect.internal.Types$AsSeenFromMap.apply(Types.scala:4638)
at scala.reflect.internal.Types$TypeMap.mapOver(Types.scala:4197)
at scala.reflect.internal.Types$AsSeenFromMap.apply(Types.scala:4638)
at scala.reflect.internal.Types$Type.asSeenFrom(Types.scala:754)
at scala.reflect.internal.Types$Type.computeMemberType(Types.scala:788)
at 
scala.reflect.internal.Symbols$MethodSymbol.typeAsMemberOf(Symbols.scala:2655)
at scala.reflect.internal.Types$Type.memberType(Types.scala:779)
at 
scala.reflect.internal.Types$class.defineUnderlyingOfSingleType(Types.scala:1534)
at 
scala.reflect.internal.SymbolTable.defineUnderlyingOfSingleType(SymbolTable.scala:13)
at scala.reflect.internal.Types$SingleType.underlying(Types.scala:1486)
at scala.reflect.internal.Types$SingletonType.widen(Types.scala:1340)
at 
scala.reflect.internal.Types$AsSeenFromMap.toPrefix$1(Types.scala:4541)
at scala.reflect.internal.Types$AsSeenFromMap.apply(Types.scala:4556)
at scala.reflect.internal.Types$TypeMap.mapOver(Types.scala:4183)

{code}

It was a very very long log, seems to be a loop

I have adjust the env like this
{code}
export JAVA_HOME=/home/hadoop/jdk
export MASTER=spark://namenode:7077 
export ZEPPELIN_PORT=10001
export SPARK_HOME=/home/hadoop/spark-1.6.2-bin-hadoop2.6
export SPARK_SUBMIT_OPTIONS="--driver-memory 2g --executor-memory 5g 
--driver-class-path 
/home/hadoop/spark-1.6.2-bin-hadoop2.6/extlib/oracle-driver.jar:/home/hadoop/spark-1.6.2-bin-hadoop2.6/extlib/phoenix-4.7.0-HBase-1.1-client-spark.jar:/home/hadoop/spark-1.6.2-bin-hadoop2.6/extlib/spark-csv_2.10-1.3.0.jar:/home/hadoop/zeppelin-0.6.0-bin-all/lib/*:/home/hadoop/zeppelin-0.6.0-bin-all/*::/home/hadoop/zeppelin-0.6.0-bin-all/conf:/home/hadoop/zeppelin-0.6.0-bin-all/interpreter/spark/*::/home/hadoop/zeppelin-0.6.0-bin-all/conf:/home/hadoop/zeppelin-0.6.0-bin-all/conf:/home/hadoop/zeppelin-0.6.0-bi

[GitHub] zeppelin issue #1266: [ZEPPELIN-1219] Add searching feature to Zeppelin docs...

2016-08-02 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1266
  
Great improvement @AhyoungRyu ! Posted few suggestions above.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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-1271) ZeppelinSparkClusterTest.sparkRTest test is flaky and fails often

2016-08-02 Thread Alexander Bezzubov (JIRA)
Alexander Bezzubov created ZEPPELIN-1271:


 Summary: ZeppelinSparkClusterTest.sparkRTest test is flaky and 
fails often
 Key: ZEPPELIN-1271
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1271
 Project: Zeppelin
  Issue Type: Bug
Affects Versions: 0.6.0, 0.7.0
Reporter: Alexander Bezzubov


ZeppelinSparkClusterTest.sparkRTest test is flaky

Same failures
 - https://github.com/apache/zeppelin/pull/1249
 - https://github.com/apache/zeppelin/pull/1160
 - https://github.com/apache/zeppelin/pull/1051
 - https://github.com/apache/zeppelin/pull/1043
 - https://github.com/apache/zeppelin/pull/943
 - https://github.com/apache/zeppelin/pull/835

{code}
[INFO] Zeppelin: Server ... FAILURE [03:05 min]

Results :

Failed tests: 
  ZeppelinSparkClusterTest.sparkRTest:116 expected:<[[1] 3]> but 
was:<[Error in getSparkSession(): SparkSession not initialized


Error in (function (classes, fdef, mtable) : unable to find an 
inherited method for function 'count' for signature '"function"'
]>

Tests run: 65, Failures: 1, Errors: 0, Skipped: 0
{code}



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


[GitHub] zeppelin issue #1263: Small cleanup of zeppelin-server tests

2016-08-02 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1263
  
CI failed due to 
[ZEPPELIN-1264](https://issues.apache.org/jira/browse/ZEPPELIN-1264)

Rebasing on latest master and merging, if there is no further discusison


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1267: [ZEPPELIN-1261] Bug fix in z.show() for matplotlib gra...

2016-08-02 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1267
  
Looks great to me, thank you @bustios !

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 #1269: ZEPPELIN-1270. Remove getting SQLContext from SparkSes...

2016-08-02 Thread zjffdu
Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1269
  
\cc @Leemoonsoo  Please help review, 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 #1269: ZEPPELIN-1270. Remove getting SQLContext from S...

2016-08-02 Thread zjffdu
GitHub user zjffdu opened a pull request:

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

ZEPPELIN-1270. Remove getting SQLContext from SparkSession.wrapped()

### What is this PR for?
SparkSession.wrapped is only for spark 2.0 preview, it is not supported in 
spark 2.0 release. So I think we can remove that piece of code. Otherwise we 
will get the following error in log which might be a little confusing.
```
08:05:44,946 - Thread(pool-2-thread-3) - (Utils.java:40) - 
org.apache.spark.sql.SparkSession.wrapped()
java.lang.NoSuchMethodException: org.apache.spark.sql.SparkSession.wrapped()
at java.lang.Class.getMethod(Class.java:1786)
at org.apache.zeppelin.spark.Utils.invokeMethod(Utils.java:38)
at org.apache.zeppelin.spark.Utils.invokeMethod(Utils.java:33)
```


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

### Todos
* [ ] - Task

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

### How should this be tested?
Check the log and the above error is gone. 

### Screenshots (if appropriate)

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



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

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

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

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


commit 4401561ca852654fecc0affa5617816517402641
Author: Jeff Zhang 
Date:   2016-08-03T00:20:03Z

ZEPPELIN-1270. Remove getting SQLContext from SparkSession.wrapped()




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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-1270) Remove getting SQLContext from SparkSession.wrapped()

2016-08-02 Thread Jeff Zhang (JIRA)
Jeff Zhang created ZEPPELIN-1270:


 Summary: Remove getting SQLContext from SparkSession.wrapped()
 Key: ZEPPELIN-1270
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1270
 Project: Zeppelin
  Issue Type: Improvement
Affects Versions: 0.7.0
Reporter: Jeff Zhang
Assignee: Jeff Zhang
Priority: Minor


SparkSession.wrapped is only for spark 2.0 preview, it is not supported in 
spark 2.0 release. So I think we can remove that piece of code. Otherwise we 
will get the following error in log which might be a little confusing.

{noformat}
08:05:44,946 - Thread(pool-2-thread-3) - (Utils.java:40) - 
org.apache.spark.sql.SparkSession.wrapped()
java.lang.NoSuchMethodException: org.apache.spark.sql.SparkSession.wrapped()
at java.lang.Class.getMethod(Class.java:1786)
at org.apache.zeppelin.spark.Utils.invokeMethod(Utils.java:38)
at org.apache.zeppelin.spark.Utils.invokeMethod(Utils.java:33)
at 
org.apache.zeppelin.spark.SparkInterpreter.getSQLContext_2(SparkInterpreter.java:241)
at 
org.apache.zeppelin.spark.SparkInterpreter.getSQLContext(SparkInterpreter.java:229)
at 
org.apache.zeppelin.spark.SparkInterpreter.open(SparkInterpreter.java:744)
at 
org.apache.zeppelin.interpreter.LazyOpenInterpreter.open(LazyOpenInterpreter.java:69)
at 
org.apache.zeppelin.spark.SparkRInterpreter.getSparkInterpreter(SparkRInterpreter.java:188)
at 
org.apache.zeppelin.spark.SparkRInterpreter.open(SparkRInterpreter.java:72)
at 
org.apache.zeppelin.interpreter.LazyOpenInterpreter.open(LazyOpenInterpreter.java:69)
at 
org.apache.zeppelin.interpreter.LazyOpenInterpreter.interpret(LazyOpenInterpreter.java:93)
at 
org.apache.zeppelin.interpreter.remote.RemoteInterpreterServer$InterpretJob.jobRun(RemoteInterpreterServer.java:383)
at org.apache.zeppelin.scheduler.Job.run(Job.java:176)
at org.apache.zeppelin.scheduler.FIFOScheduler$1.run(FIFOScheduler.java:139)
at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
at java.util.concurrent.FutureTask.run(FutureTask.java:266)
at 
java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$201(ScheduledThreadPoolExecutor.java:180)
at 
java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:293)
at 
java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
at 
java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
at java.lang.Thread.run(Thread.java:745)
{noformat}



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


[GitHub] zeppelin pull request #1268: minor doc fix for r.md

2016-08-02 Thread zjffdu
GitHub user zjffdu opened a pull request:

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

minor doc fix for r.md

### What is this PR for?
straightforward doc fix


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

### Todos
* [ ] - Task

### What is the Jira issue?
* No jira created as it is very trival. 


### Screenshots (if appropriate)

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



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

$ git pull https://github.com/zjffdu/zeppelin r_doc_fix

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

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


commit 105fead32d131ed466bc169f084a443556968b9d
Author: Jeff Zhang 
Date:   2016-08-02T23:51:07Z

minor doc fix for r.md




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1268: minor doc fix for r.md

2016-08-02 Thread zjffdu
Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1268
  
@AhyoungRyu  Please help review. Thanks


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


[GitHub] zeppelin issue #1232: ZEPPELIN-1197. Should print output directly without in...

2016-08-02 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1232
  
Merge into master if there're no more discussions.


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


[GitHub] zeppelin issue #1260: ZEPPELIN-1267. PySparkInterpreter doesn't work in spar...

2016-08-02 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1260
  
Looks good to me.


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


[GitHub] zeppelin pull request #1267: [ZEPPELIN-1261] Bug fix in z.show() for matplot...

2016-08-02 Thread bustios
GitHub user bustios opened a pull request:

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

[ZEPPELIN-1261] Bug fix in z.show() for matplotlib graphs

### What is this PR for?
Bug fix in z.show() for matplotlib graphs and code refactoring

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

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

### How should this be tested?
```
%python
import matplotlib.pyplot as plt

x = [1,2,3,4,5]
y = [6,7,8,9,0]

plt.plot(x, y, marker="o")
z.show(plt, height="20em")
plt.close()
```
```
%python
import matplotlib.pyplot as plt

x = [1,2,3,4,5]
y = [6,7,8,9,0]

plt.plot(x, y, marker="o")
z.show(plt, height="300px")
plt.close()
```

### Screenshots (if appropriate)

![plot](https://dl.dropboxusercontent.com/u/20947972/z.show.height.example.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/bustios/zeppelin ZEPPELIN-1261

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

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


commit 3e0ef22121d3858206f9af1787bf79108f9f2ed6
Author: Paul Bustios 
Date:   2016-08-02T20:55:56Z

Bug fix in show_matplotlib




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


[GitHub] zeppelin issue #1259: [ZEPPELIN-1260] R interpreter doesn't work with Spark ...

2016-08-02 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1259
  
Merge it into master and branch-0.6 if there're no more discussion


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


[GitHub] zeppelin pull request #1261: [ZEPPELIN-1264] [HOTFIX] Fix CI test failure wi...

2016-08-02 Thread asfgit
Github user asfgit closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1261: [ZEPPELIN-1264] [HOTFIX] Fix CI test failure with Fail...

2016-08-02 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1261
  
Merge it into master and branch-0.6 as a hotfix


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

2016-08-02 Thread AhyoungRyu
GitHub user AhyoungRyu opened a pull request:

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

[ZEPPELIN-1219] Add searching feature to Zeppelin docs site

### What is this PR for?
As more and more document pages are added, it's really hard to find 
specific pages. So I added searching feature to Zeppelin documentation 
site([jekyll](https://jekyllrb.com/) based site) using 
[lunr.js](http://lunrjs.com/).

 - **How it works?**
  
  I created 
[`search_data.json`](https://github.com/AhyoungRyu/zeppelin/blob/6e02423f541cc406e4e41031629609a276a9f481/docs/search_data.json)
 which is used for docs info template. `lunr.js` combines all of the text from 
all of the docs in `docs/` into `_site/search_data.json`. It looks like below.
![screen shot 2016-08-03 at 4 49 59 
am](https://cloud.githubusercontent.com/assets/10060731/17342828/f2908be8-5935-11e6-8eee-b189677c0531.png)
All the info are comes from [Jekyll YAML front 
matter](https://jekyllrb.com/docs/frontmatter/) variables. (i.e. title, group, 
description.. that's why I rewrote all docs' title and description.)
Using this data, 
[search.js](https://github.com/AhyoungRyu/zeppelin/blob/6e02423f541cc406e4e41031629609a276a9f481/docs/assets/themes/zeppelin/js/search.js)
 will do this job!

### What type of PR is it?
Improvement & Feature

### Todos
* [x] - Keep consistency for all docs pages' `Title`
* [x] - Add some overview sentences to all docs pages' `Description` 
section (this will be used as the result preview)
* [x] - Add apache license header to all docs page (some pages are missing 
the license header currently)
* [x] - Add LICENSE for `lunr.min.js`

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

### How should this be tested?
1. Build docs site -> please see 
[docs/README.md#build-documentation](https://github.com/apache/zeppelin/tree/master/docs#build-documentation)
2. Click `search` icon in navbar and go to `search.html` page
3. Type anything you want to search in the search bar (i.e. type `python`, 
`spark`, `dynamic` ... )

### Screenshots (if appropriate)
https://cloud.githubusercontent.com/assets/10060731/17343202/71185300-5937-11e6-9e57-687bf2839cfb.png";>


![search](https://cloud.githubusercontent.com/assets/10060731/17343154/3df052f2-5937-11e6-950a-ba922576914d.gif)

### Questions:
* Does the licenses files need update? Yes, for `lunr.min.js`
* 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 ZEPPELIN-1219

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

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


commit 881a6b036e70bfb05f12c8a633f10c873508de9d
Author: AhyoungRyu 
Date:   2016-08-02T19:11:27Z

Add js files: lunr.min.js & search.js

commit d261a63a7581ed0cc7ca3d78c46ae4b074a17ba7
Author: AhyoungRyu 
Date:   2016-08-02T19:13:25Z

Add template for storing docs info based on jekyll front matter

commit 495d6c420d975653f48926e055cc299e043daaa8
Author: AhyoungRyu 
Date:   2016-08-02T19:16:27Z

Keep consistency all docs' front matter for the right search result

commit 09e023d86cfc48794f3df53529266f8fd6e65832
Author: AhyoungRyu 
Date:   2016-08-02T19:16:51Z

Add search icon to navbar

commit d55ce76f310634298c5605f8b5a9ff2e1f3dcd6c
Author: AhyoungRyu 
Date:   2016-08-02T19:17:08Z

Add searching page

commit d24c536f67c597db77751cbb383d113916bf769e
Author: AhyoungRyu 
Date:   2016-08-02T19:19:02Z

Attach 'Apache Zeppelin ZEPPELIN_VERSION Documentation: ' to title

commit 6e02423f541cc406e4e41031629609a276a9f481
Author: AhyoungRyu 
Date:   2016-08-02T19:20:03Z

Apply css style for the search results

commit 0f3457d8c4aaebe4419f68c48a6422494d54a2d2
Author: AhyoungRyu 
Date:   2016-08-02T20:12:11Z

Add lunr.min.js license info

commit e6ab4b7a424f6681001eb14dffad9bdae4b2e03d
Author: AhyoungRyu 
Date:   2016-08-02T20:12:42Z

Exclude lunr.min.js




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1259: [ZEPPELIN-1260] R interpreter doesn't work with Spark ...

2016-08-02 Thread echarles
Github user echarles commented on the issue:

https://github.com/apache/zeppelin/pull/1259
  
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 #932: Replace CXF with Jersey2 [ZEPPELIN-903]

2016-08-02 Thread echarles
Github user echarles commented on the issue:

https://github.com/apache/zeppelin/pull/932
  
Failing with 

org.apache.zeppelin.interpreter.InterpreterException: 
org.apache.thrift.TApplicationException: Internal error processing 
createInterpreter
at 
org.apache.zeppelin.interpreter.remote.RemoteInterpreter.init(RemoteInterpreter.java:238)

Is this a WKZCIF (Well Know Zeppelin CI Failure) ?


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

https://github.com/apache/zeppelin/pull/1265
  
Quickly went through the changes, but not sure to get it.
In which way is this related to the various intepreter modes we already 
have?
In which way is this related to ZEPPELIN-1000 (Multiple simultaneous users 
on a single WEB server)?


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


[GitHub] zeppelin issue #1247: ZEPPELIN-1254 Make get and save Interpreter bindings c...

2016-08-02 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/1247
  
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 #928: [ZEPPELIN-116] Add Apache Mahout Interpreter

2016-08-02 Thread rawkintrevo
Github user rawkintrevo commented on the issue:

https://github.com/apache/zeppelin/pull/928
  
[Build 6281](https://travis-ci.org/apache/zeppelin/builds/149199571)

Profiles 
[1](https://s3.amazonaws.com/archive.travis-ci.org/jobs/149199572/log.txt), 
[2](https://s3.amazonaws.com/archive.travis-ci.org/jobs/149199574/log.txt), and 
[3](https://travis-ci.org/apache/zeppelin/jobs/149199575):

All stalled a la 
[ZEPPELIN-862](https://issues.apache.org/jira/browse/ZEPPELIN-862?jql=project%20%3D%20ZEPPELIN%20AND%20status%20in%20(Open%2C%20%22In%20Progress%22%2C%20Reopened)%20AND%20labels%20%3D%20flaky-test)



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


[GitHub] zeppelin pull request #1265: [ZEPPELIN-1210] Run interpreter per user

2016-08-02 Thread jongyoul
GitHub user jongyoul opened a pull request:

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

[ZEPPELIN-1210] Run interpreter per user

### What is this PR for?
Enabling each user to run same interpreter.

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

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

### How should this be tested?
1. Enable shiro to use authentication mode
1. Check `per user` in your interpreter tab
1. Run different paragraphs with different users
  1. Run `%spark sc.version`, you will see the two `res0: ...` in your 
paragraphs

### 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/jongyoul/zeppelin ZEPPELIN-1210

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

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


commit 8c5874756c7f2cbddc3d2a93b3ea60dbaf30b469
Author: Jongyoul Lee 
Date:   2016-07-21T15:34:25Z

Added new parameter in order to pass the user information into 
getInterpreter

commit 9fdfd1747a49b900fe952a1704eab35a5b4487bd
Author: Jongyoul Lee 
Date:   2016-07-22T10:29:07Z

WIP

commit 52e1cbac924964316368cf112284325706174f03
Author: Jongyoul Lee 
Date:   2016-07-28T13:21:23Z

WIP

commit 65a24415824055bee5691866654b9b16e9106c76
Author: Jongyoul Lee 
Date:   2016-08-02T15:10:18Z

Added option in UI




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

2016-08-02 Thread rawkintrevo
GitHub user rawkintrevo reopened a pull request:

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

[ZEPPELIN-116] Add Apache Mahout Interpreter

### What is this PR for?
This PR adds Mahout functionality for the Spark Interpreter.


### What type of PR is it?
Improvement 

### Todos
- [x] Implement Mahout Interpreter in Spark
- [x] Add Unit Tests
- [x] Add Documentation
- [x] Add Example Notebook

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

### How should this be tested?
Open a Spark Notebook with Mahout enabled and run a few simple commands 
using the R-Like DSL and Spark Distributed Context (Mahout Specific)

### Screenshots (if appropriate)

### 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/rawkintrevo/incubator-zeppelin mahout-terp

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

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


commit af935152ce0b70578317ad0f629fd7cf7043f9e0
Author: Trevor Grant 
Date:   2016-06-08T17:05:25Z

[ZEPPELIN-116] Add Mahout Interpreter




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

2016-08-02 Thread rawkintrevo
Github user rawkintrevo closed the pull request at:

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


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


[GitHub] zeppelin issue #928: [ZEPPELIN-116] Add Apache Mahout Interpreter

2016-08-02 Thread rawkintrevo
Github user rawkintrevo commented on the issue:

https://github.com/apache/zeppelin/pull/928
  
## CI Failures in This PR

 Profile 
[2](https://s3.amazonaws.com/archive.travis-ci.org/jobs/148308638/log.txt) 
Hangs a la 
[ZEPPELIN-862](https://issues.apache.org/jira/browse/ZEPPELIN-862?jql=project%20%3D%20ZEPPELIN%20AND%20status%20in%20(Open%2C%20%22In%20Progress%22%2C%20Reopened)%20AND%20labels%20%3D%20flaky-test)

`No output has been received in the last 10 minutes, this potentially 
indicates a stalled build or something wrong with the build itself.`

 Profile 
[6](https://s3.amazonaws.com/archive.travis-ci.org/jobs/148308642/log.txt), 
[8](https://s3.amazonaws.com/archive.travis-ci.org/jobs/148308647/log.txt) 
Network issues, can't download dependencies
`[ERROR] Failed to execute goal 
org.apache.maven.plugins:maven-remote-resources-plugin:1.4:process (default) on 
project zeppelin: Error downloading resources archive. Could not transfer 
artifact 
org.apache.apache.resources:apache-jar-resource-bundle:jar:1.5-SNAPSHOT from/to 
codehaus-snapshots (https://nexus.codehaus.org/snapshots/): nexus.codehaus.org: 
Name or service not known`

`ERROR] Failed to execute goal 
org.apache.maven.plugins:maven-remote-resources-plugin:1.4:process (default) on 
project zeppelin: Error downloading resources archive. Could not transfer 
artifact 
org.apache.apache.resources:apache-jar-resource-bundle:jar:1.5-SNAPSHOT from/to 
apache-snapshots (https://repository.apache.org/snapshots/): Connect to 
repository.apache.org:443 [repository.apache.org/207.244.88.143] failed: 
Connection timed out`

 Profile [7 
](https://s3.amazonaws.com/archive.travis-ci.org/jobs/148308645/log.txt) This 
is a [known flaky test with no coresponding 
JIRA](https://github.com/apache/zeppelin/pull/1201)
`31m- should provide onclick method *** FAILED ***
  The code passed to eventually never returned normally. Attempted 1 
times over 206.957211 milliseconds. Last failure message: 0 was not equal to 1. 
(AbstractAngularElemTest.scala:72)`


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1264: Updated path to point to latest Maven binaries

2016-08-02 Thread jojurajan
GitHub user jojurajan opened a pull request:

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

Updated path to point to latest Maven binaries

### What is this PR for?
To update link of Maven binary provided in the installation steps.

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

### Todos
NA

### What is the Jira issue?
NA

### How should this be tested?
By running the following commands
```
wget 
http://www.eu.apache.org/dist/maven/maven-3/3.3.9/binaries/apache-maven-3.3.9-bin.tar.gz
sudo tar -zxf apache-maven-3.3.9-bin.tar.gz -C /usr/local/
sudo ln -s /usr/local/apache-maven-3.3.9/bin/mvn /usr/local/bin/mvn

# Build using latest mvn
mvn clean package -Pspark-1.6 -Phadoop-2.4 -Pyarn -Ppyspark -DskipTests
```

### 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/jojurajan/zeppelin jojurajan-maven-link-patch

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

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


commit 0a4853f1a56c1bf108ec5a9e19111708ae649194
Author: Joju Rajan 
Date:   2016-08-02T06:56:48Z

Updated path to latest Maven binaries




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1261: [ZEPPELIN-1264] [HOTFIX] Fix CI test failure with Fail...

2016-08-02 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1261
  
Have created 
[ZEPPELIN-1269](https://issues.apache.org/jira/browse/ZEPPELIN-1269)

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-1269) HeliumApplicationFactoryTest hangs on default build from master

2016-08-02 Thread Alexander Bezzubov (JIRA)
Alexander Bezzubov created ZEPPELIN-1269:


 Summary: HeliumApplicationFactoryTest hangs on default build from 
master
 Key: ZEPPELIN-1269
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1269
 Project: Zeppelin
  Issue Type: Bug
Reporter: Alexander Bezzubov



{code}
Running org.apache.zeppelin.helium.HeliumApplicationFactoryTest
20:53:26,641 ERROR org.apache.zeppelin.interpreter.Interpreter:341 - Static 
initialization is deprecated. You should change it to use 
interpreter-setting.json in your jar or 
interpreter/{interpreter}/interpreter-setting.json



20:53:28,711  INFO 
org.apache.zeppelin.interpreter.remote.RemoteInterpreterManagedProcess:116 - 
Run interpreter process 
[/Users/alex/floss/zeppelin-incubator/bin/interpreter.sh, -d, 
/Users/alex/floss/zeppelin-incubator/interpreter/spark, -p, 60434, -l, 
/Users/alex/floss/zeppelin-incubator/local-repo/2BUP7GVTH]
20:53:28,763  INFO 
org.apache.zeppelin.interpreter.remote.RemoteInterpreterManagedProcess:154 - 
Interpreter process exited 0
20:53:58,990 ERROR org.apache.zeppelin.scheduler.RemoteScheduler:256 - Can't 
get status information
org.apache.zeppelin.interpreter.InterpreterException: 
org.apache.thrift.transport.TTransportException: java.net.ConnectException: 
Connection refused
at 
org.apache.zeppelin.interpreter.remote.ClientFactory.create(ClientFactory.java:53)
at 
org.apache.zeppelin.interpreter.remote.ClientFactory.create(ClientFactory.java:1)
at 
org.apache.commons.pool2.BasePooledObjectFactory.makeObject(BasePooledObjectFactory.java:60)
at 
org.apache.commons.pool2.impl.GenericObjectPool.create(GenericObjectPool.java:861)
at 
org.apache.commons.pool2.impl.GenericObjectPool.borrowObject(GenericObjectPool.java:435)
at 
org.apache.commons.pool2.impl.GenericObjectPool.borrowObject(GenericObjectPool.java:363)
at 
org.apache.zeppelin.interpreter.remote.RemoteInterpreterProcess.getClient(RemoteInterpreterProcess.java:96)
at 
org.apache.zeppelin.scheduler.RemoteScheduler$JobStatusPoller.getStatus(RemoteScheduler.java:254)
at 
org.apache.zeppelin.scheduler.RemoteScheduler$JobStatusPoller.run(RemoteScheduler.java:212)
Caused by: org.apache.thrift.transport.TTransportException: 
java.net.ConnectException: Connection refused
at org.apache.thrift.transport.TSocket.open(TSocket.java:187)

..

20:56:59,101  INFO org.apache.zeppelin.notebook.Paragraph:282 - run paragraph 
20160802-205659_1961452200 using null 
org.apache.zeppelin.interpreter.LazyOpenInterpreter@747b96ad
20:56:59,105 ERROR org.apache.zeppelin.scheduler.Job:189 - Job failed
org.apache.zeppelin.interpreter.InterpreterException: 
java.lang.NullPointerException
at 
org.apache.zeppelin.interpreter.dev.DevInterpreter.interpret(DevInterpreter.java:88)
at 
org.apache.zeppelin.interpreter.ClassloaderInterpreter.interpret(ClassloaderInterpreter.java:58)
at 
org.apache.zeppelin.interpreter.LazyOpenInterpreter.interpret(LazyOpenInterpreter.java:94)
at org.apache.zeppelin.notebook.Paragraph.jobRun(Paragraph.java:309)
at org.apache.zeppelin.scheduler.Job.run(Job.java:176)
at org.apache.zeppelin.scheduler.FIFOScheduler$1.run(FIFOScheduler.java:139)
at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:471)
at java.util.concurrent.FutureTask.run(FutureTask.java:262)
at 
java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$201(ScheduledThreadPoolExecutor.java:178)
at 
java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:292)
at 
java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
at 
java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
at java.lang.Thread.run(Thread.java:744)
Caused by: java.lang.NullPointerException
at 
org.apache.zeppelin.interpreter.dev.DevInterpreter.interpret(DevInterpreter.java:86)
... 12 more
{code}



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


[GitHub] zeppelin issue #1231: [Zeppelin - 683][WIP] BittorrentNotebookRepo

2016-08-02 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1231
  
@onkarshedge 
>> Initial idea the we discussed was that `BittorrentNotebookRepo` 
encapsulates all the things that are required in order to get notebooks.

> Could you please elaborate this point.

Sure. The idea with P2P notebook storage implementation, be that Bittorrent 
or Dat or any other protocol is to have:
 - a self-contained implementation (that possibly can be extracted to maven 
submodule later) 
 - of a single class, that `implements NotebookRepo` protocol
 - and is pluggable, instead or on the side with current `VFSNotebookRepo` 
(for now - pluggable though a simple configuration in `zeppelin-env.sh`, later 
I can imagine that would be also possible though GUI)

In case only some parts of `VFSNotebookRepo` are required (not sure if that 
is the case or not) - we can extract them to the _skeletal implementation_ 
`AbstractFileSystemNotebookRepo` and make `BittorrentNotebookRepo extends 
AbstractFileSystemNotebookRepo`.  

For more details on this approach to design object oriented architecture 
please read infamous Joshua Bloch "Effective Java, 2nd edition" `Item18: Prefer 
interfaces to abstract classes` p.93 

---

Waiting for the diagram of current architecture and code improvements 
according to styleguide - please ping me and I'll be happy to make another pass 
tomorrow.




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1263: Small cleanup of zeppelin-server tests

2016-08-02 Thread bzz
GitHub user bzz opened a pull request:

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

Small cleanup of zeppelin-server tests

### What is this PR for?
Small cleanup of zeppelin-server tests

### What type of PR is it?
Refactoring

### How should this be tested?

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



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

$ git pull https://github.com/bzz/incubator-zeppelin 
cleanup/zeppelin-server-tests

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

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


commit 95462d6dedb834b8d8d63865083cf215894f951c
Author: Alexander Bezzubov 
Date:   2016-08-02T11:05:36Z

Small cleanup of org.apache.zeppelin.* tests




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1259: [ZEPPELIN-1260] R interpreter doesn't work with Spark ...

2016-08-02 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1259
  
Thanks @echarles for the review. I have exposed sqlContext as well


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

2016-08-02 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/1262
  
@kavinkumarks Thanks for the contribution, the Paragraph ID is actually 
already shown in the UI

https://github.com/apache/zeppelin/blob/master/zeppelin-web/src/app/notebook/paragraph/paragraph-control.html#L43



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

2016-08-02 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1239
  
@Leemoonsoo @minahlee Could you please test 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 pull request #1262: [ZEPPELIN-1268]

2016-08-02 Thread kavinkumarks
GitHub user kavinkumarks opened a pull request:

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

[ZEPPELIN-1268]

### What is this PR for?
This pull request is to add the below options so when someone develops a 
custom UI it would ease them.
1. Add a new option "Get paragraph id" in the notebook UI
2. Add a new REST API to get the status of a single paragraph

### What type of PR is it?
Improvement

### Todos
* Need to update the REST APIs documentation 
http://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/rest-api/rest-notebook.html#get-the-status-of-all-paragraphs

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

### How should this be tested?
* Navigate to the notebook UI and a new option "Get paragraph id" will be 
shown for each paragraph and on clicking on it, the paragraph id would be 
alerted
* The new REST API 
http://%5Bzeppelin-server%5D:%5Bzeppelin-port%5D/api/notebook/job/%5BnotebookId%5D/%5BparagraphId%5D
 could be accessed to get the status of a single paragraph.

### Screenshots (if appropriate)

### Questions:

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

$ git pull https://github.com/kavinkumarks/zeppelin zeppelin-1268-custom-UI

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

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


commit a449992256d8badf72a412ccc0e3134873d5db31
Author: Kavin 
Date:   2016-08-02T11:07:35Z

Changes to show new option to get paragraph id in notebook UI and new REST 
API to get status of a single paragraph




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1259: [ZEPPELIN-1260] R interpreter doesn't work with Spark ...

2016-08-02 Thread echarles
Github user echarles commented on the issue:

https://github.com/apache/zeppelin/pull/1259
  
@minahlee Just saw you comment on ZEPPELIN-1260, but @Leemoonsoo has been 
fast...

I have tested the changes, and now sparkr dataframes are working well (with 
the `spark` variable which referts to the new Spark2 session).

The R tutorial note refers to sqlContext which is no more present

```
Error in dispatchFunc("createDataFrame(data, schema = NULL, samplingRatio = 
1.0)", : object 'sqlContext' not found
```

Patch could also expose sqlContext.



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1261: [ZEPPELIN-1264] [HOTFIX] Fix CI test failure with Fail...

2016-08-02 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1261
  
@bzz I see the same error on my local test. However somehow travis does not 
have the error.
Anyway, do you mind file a new issue for the new error?


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


[GitHub] zeppelin issue #1258: [ ZEPPELIN-1266 ] Code editor Optimization

2016-08-02 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/1258
  
I would limit this PR to the first commit, for me the rest is out of scope.
Also, in the first PR, there is no need to make a if before calling 
`setParagraphMode`, the conditions you added is already at the beginning of the 
function.

`$scope.setParagraphMode` can also be changed to `var setParagraphMode`


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

2016-08-02 Thread SachinJanani
Github user SachinJanani commented on the issue:

https://github.com/apache/zeppelin/pull/1234
  
Travis checks are failing but are not related to this changes.@jongyoul Can 
you please have a look. 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1261: [ZEPPELIN-1264] [HOTFIX] Fix CI test failure with Fail...

2016-08-02 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1261
  
Have updated the comment, you might need to reload the page


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1261: [ZEPPELIN-1264] [HOTFIX] Fix CI test failure with Fail...

2016-08-02 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1261
  
Local build results are - it hangs for last 4 minutes with the error below

```
20:56:59,101  INFO org.apache.zeppelin.notebook.Paragraph:282 - run 
paragraph 20160802-205659_1961452200 using null 
org.apache.zeppelin.interpreter.LazyOpenInterpreter@747b96ad
20:56:59,105 ERROR org.apache.zeppelin.scheduler.Job:189 - Job failed
org.apache.zeppelin.interpreter.InterpreterException: 
java.lang.NullPointerException
at 
org.apache.zeppelin.interpreter.dev.DevInterpreter.interpret(DevInterpreter.java:88)
at 
org.apache.zeppelin.interpreter.ClassloaderInterpreter.interpret(ClassloaderInterpreter.java:58)
at 
org.apache.zeppelin.interpreter.LazyOpenInterpreter.interpret(LazyOpenInterpreter.java:94)
at org.apache.zeppelin.notebook.Paragraph.jobRun(Paragraph.java:309)
at org.apache.zeppelin.scheduler.Job.run(Job.java:176)
at 
org.apache.zeppelin.scheduler.FIFOScheduler$1.run(FIFOScheduler.java:139)
at 
java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:471)
at java.util.concurrent.FutureTask.run(FutureTask.java:262)
at 
java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$201(ScheduledThreadPoolExecutor.java:178)
at 
java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:292)
at 
java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
at 
java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
at java.lang.Thread.run(Thread.java:744)
Caused by: java.lang.NullPointerException
at 
org.apache.zeppelin.interpreter.dev.DevInterpreter.interpret(DevInterpreter.java:86)
... 12 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 #1261: [ZEPPELIN-1264] [HOTFIX] Fix CI test failure with Fail...

2016-08-02 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1261
  
Not sure why CI is not triggered in this PR.
Here's CI result with my travis account 
https://travis-ci.org/Leemoonsoo/zeppelin/builds/149153003

CI is not green yet, but at least ZEPPELIN-1264 is resolved.



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #932: Replace CXF with Jersey2 [ZEPPELIN-903]

2016-08-02 Thread echarles
Github user echarles commented on the issue:

https://github.com/apache/zeppelin/pull/932
  
@Leemoonsoo  Just pushed the asked changes regarding licenses.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1261: [ZEPPELIN-1264] [HOTFIX] Fix CI test failure wi...

2016-08-02 Thread Leemoonsoo
Github user Leemoonsoo closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1261: [ZEPPELIN-1264] [HOTFIX] Fix CI test failure wi...

2016-08-02 Thread Leemoonsoo
GitHub user Leemoonsoo reopened a pull request:

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

[ZEPPELIN-1264] [HOTFIX] Fix CI test failure with Failed to create 
interpreter: org.apache.zeppelin.interpreter.remote.mock.MockInterpreterA

### What is this PR for?
Fix CI test failure with error 

```
14:05:27,226 ERROR 
org.apache.zeppelin.interpreter.remote.RemoteInterpreter:237 - Failed to create 
interpreter: org.apache.zeppelin.interpreter.remote.mock.MockInterpreterA
14:05:27,227 ERROR 
org.apache.zeppelin.interpreter.remote.RemoteInterpreter:264 - Failed to 
initialize interpreter: 
org.apache.zeppelin.interpreter.remote.mock.MockInterpreterA. Remove it from 
interpreterGroup
14:05:27,240  INFO org.apache.zeppelin.scheduler.SchedulerFactory:131 - Job 
jobName1 started by scheduler test
14:05:27,240  INFO 
org.apache.zeppelin.interpreter.remote.RemoteInterpreter:223 - Create remote 
interpreter org.apache.zeppelin.interpreter.remote.mock.MockInterpreterA
14:05:27,242 ERROR 
org.apache.zeppelin.interpreter.remote.RemoteInterpreter:237 - Failed to create 
interpreter: org.apache.zeppelin.interpreter.remote.mock.MockInterpreterA
14:05:27,243 ERROR org.apache.zeppelin.scheduler.Job:189 - Job failed
org.apache.zeppelin.interpreter.InterpreterException: 
org.apache.thrift.TApplicationException: Internal error processing 
createInterpreter
at 
org.apache.zeppelin.interpreter.remote.RemoteInterpreter.init(RemoteInterpreter.java:238)
at 
org.apache.zeppelin.interpreter.remote.RemoteInterpreter.getFormType(RemoteInterpreter.java:383)
at 
org.apache.zeppelin.interpreter.remote.RemoteInterpreter.interpret(RemoteInterpreter.java:299)
at 
org.apache.zeppelin.scheduler.RemoteSchedulerTest$2.jobRun(RemoteSchedulerTest.java:210)
at org.apache.zeppelin.scheduler.Job.run(Job.java:176)
at 
org.apache.zeppelin.scheduler.RemoteScheduler$JobRunner.run(RemoteScheduler.java:329)
at 
java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:471)
at java.util.concurrent.FutureTask.run(FutureTask.java:262)
at 
java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$201(ScheduledThreadPoolExecutor.java:178)
at 
java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:292)
at 
java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
at 
java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
at java.lang.Thread.run(Thread.java:745)
Caused by: org.apache.thrift.TApplicationException: Internal error 
processing createInterpreter
at 
org.apache.thrift.TApplicationException.read(TApplicationException.java:111)
at org.apache.thrift.TServiceClient.receiveBase(TServiceClient.java:71)
at 
org.apache.zeppelin.interpreter.thrift.RemoteInterpreterService$Client.recv_createInterpreter(RemoteInterpreterService.java:196)
at 
org.apache.zeppelin.interpreter.thrift.RemoteInterpreterService$Client.createInterpreter(RemoteInterpreterService.java:180)
at 
org.apache.zeppelin.interpreter.remote.RemoteInterpreter.init(RemoteInterpreter.java:227)
... 12 more
```

Some unittest launches remote interpreter process for the test with some 
mock interpreter implementation. So mock interpreter class in the test should 
be available for interpreter's classpath for the test.


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

### Todos
* [x] - Add necessary test-classes directory in interpreter process's 
classpath

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

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


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

$ git pull https://github.com/Leemoonsoo/zeppelin ZEPPELIN-1264

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

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


commit 10ad928c79bc2ade26e6cdee58aa4e35ff4fda61
Author: Lee moon soo 
Date:   2016-08-02T11:38:45Z

Add zeppelin-interpreter/target/test-classes, 
zeppelin-zengine/target/test-classes in classpath of interpreter




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1261: [ZEPPELIN-1264] [HOTFIX] Fix CI test failure with Fail...

2016-08-02 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1261
  
CI seems to not beeing triggered on this branch :\

Trying locally with `time mvn package -pl "\!zeppelin-distribution"`


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1248: [ZEPPELIN-1069]Ignore implicit interpreter when user e...

2016-08-02 Thread astroshim
Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1248
  
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 pull request #1261: [ZEPPELIN-1264] [HOTFIX] Fix CI test failure wi...

2016-08-02 Thread Leemoonsoo
GitHub user Leemoonsoo opened a pull request:

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

[ZEPPELIN-1264] [HOTFIX] Fix CI test failure with Failed to create 
interpreter: org.apache.zeppelin.interpreter.remote.mock.MockInterpreterA

### What is this PR for?
Fix CI test failure with error 

```
14:05:27,226 ERROR 
org.apache.zeppelin.interpreter.remote.RemoteInterpreter:237 - Failed to create 
interpreter: org.apache.zeppelin.interpreter.remote.mock.MockInterpreterA
14:05:27,227 ERROR 
org.apache.zeppelin.interpreter.remote.RemoteInterpreter:264 - Failed to 
initialize interpreter: 
org.apache.zeppelin.interpreter.remote.mock.MockInterpreterA. Remove it from 
interpreterGroup
14:05:27,240  INFO org.apache.zeppelin.scheduler.SchedulerFactory:131 - Job 
jobName1 started by scheduler test
14:05:27,240  INFO 
org.apache.zeppelin.interpreter.remote.RemoteInterpreter:223 - Create remote 
interpreter org.apache.zeppelin.interpreter.remote.mock.MockInterpreterA
14:05:27,242 ERROR 
org.apache.zeppelin.interpreter.remote.RemoteInterpreter:237 - Failed to create 
interpreter: org.apache.zeppelin.interpreter.remote.mock.MockInterpreterA
14:05:27,243 ERROR org.apache.zeppelin.scheduler.Job:189 - Job failed
org.apache.zeppelin.interpreter.InterpreterException: 
org.apache.thrift.TApplicationException: Internal error processing 
createInterpreter
at 
org.apache.zeppelin.interpreter.remote.RemoteInterpreter.init(RemoteInterpreter.java:238)
at 
org.apache.zeppelin.interpreter.remote.RemoteInterpreter.getFormType(RemoteInterpreter.java:383)
at 
org.apache.zeppelin.interpreter.remote.RemoteInterpreter.interpret(RemoteInterpreter.java:299)
at 
org.apache.zeppelin.scheduler.RemoteSchedulerTest$2.jobRun(RemoteSchedulerTest.java:210)
at org.apache.zeppelin.scheduler.Job.run(Job.java:176)
at 
org.apache.zeppelin.scheduler.RemoteScheduler$JobRunner.run(RemoteScheduler.java:329)
at 
java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:471)
at java.util.concurrent.FutureTask.run(FutureTask.java:262)
at 
java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$201(ScheduledThreadPoolExecutor.java:178)
at 
java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:292)
at 
java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
at 
java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
at java.lang.Thread.run(Thread.java:745)
Caused by: org.apache.thrift.TApplicationException: Internal error 
processing createInterpreter
at 
org.apache.thrift.TApplicationException.read(TApplicationException.java:111)
at org.apache.thrift.TServiceClient.receiveBase(TServiceClient.java:71)
at 
org.apache.zeppelin.interpreter.thrift.RemoteInterpreterService$Client.recv_createInterpreter(RemoteInterpreterService.java:196)
at 
org.apache.zeppelin.interpreter.thrift.RemoteInterpreterService$Client.createInterpreter(RemoteInterpreterService.java:180)
at 
org.apache.zeppelin.interpreter.remote.RemoteInterpreter.init(RemoteInterpreter.java:227)
... 12 more
```

Some unittest launches remote interpreter process for the test with some 
mock interpreter implementation. So mock interpreter class in the test should 
be available for interpreter's classpath for the test.


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

### Todos
* [x] - Add necessary test-classes directory in interpreter process's 
classpath

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

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

…/test-classes in classpath of interpreter

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

$ git pull https://github.com/Leemoonsoo/zeppelin ZEPPELIN-1264

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

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


commit 10ad928c79bc2ade26e6cdee58aa4e35ff4fda61
Author: Lee moon soo 
Date:   2016-08-02T11:38:45Z

Add zeppelin-interpreter/target/test-classes, 
zeppelin-zengine/target/test-classes in classpath of interpreter




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1259: [ZEPPELIN-1260] R interpreter doesn't work with Spark ...

2016-08-02 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1259
  
@bzz Thanks for quick review. Updated the indent.
This should be merged to branch-0.6 as well


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1227: [ZEPPELIN-1198][Spark Standalone] Documents for runnin...

2016-08-02 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1227
  
Looks great to me! Merging if there is no further discussion.

> Does this needs documentation? no

I think it does ;)


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1255: Adding extra libraries to livy

2016-08-02 Thread zjffdu
Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1255
  
You can use --conf spark.jars= and --conf 
spark.jars.packages= instead 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1255: Adding extra libraries to livy

2016-08-02 Thread mfelgamal
Github user mfelgamal commented on the issue:

https://github.com/apache/zeppelin/pull/1255
  
I think that the livy itself doesn't support add  (--jars/--packages 
property in spark-submit)


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

2016-08-02 Thread onkarshedge
Github user onkarshedge commented on the issue:

https://github.com/apache/zeppelin/pull/1231
  
@bzz . I will post a diagram of communication for clear view.
`BittorrentNotebookRepo` does not implement/override any `NotebookRepo` 
methods. It uses all VFSNotebookRepo methods.
```java
Note get(String noteId, AuthenticationInfo subject);
List list(AuthenticationInfo subject);
void save(Note note, AuthenticationInfo subject);
```
Only two key methods are
```java
public String shareNote(String noteId); // take the note.json file and 
start seeding, return magnet link
public void download(String magnetLink); // download the files in the 
torrent directory.
```

```
Initial idea the we discussed was that BittorrentNotebookRepo encapsulates 
all the things that are required in order to get notebooks.
```
Could you please elaborate this point. 
Once the design and goal is clear, it will help me implement faster. 
Give me some time I will post a diagram..


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1254: [ZEPPELIN-1257] storage - fix get note revision api

2016-08-02 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue:

https://github.com/apache/zeppelin/pull/1254
  
@bzz i believe we decided not to merge it for now. I'll revise it and let 
you know


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

2016-08-02 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1231
  
@onkarshedge on the architecture side - could you please elaborate on the 
design that you choose, especially regarding `TorrentServlet` - why do you need 
a new servlet? What are the parts that communicate to each other? A diagram 
(event ASCI one) would help here.

On the code side - please read the project style guide for Java language 
carefully and make sure you follow the conventions described there.


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

2016-08-02 Thread onkarshedge
Github user onkarshedge commented on the issue:

https://github.com/apache/zeppelin/pull/1231
  
@khalidhuseynov  thanks for helping.

To actually see `TorrentSocket` in action I have to create 
a`WebSocketCreator` and a `Servlet`, then add this to jetty server , [at this 
line](https://github.com/apache/zeppelin/blob/6f867ceb0c2b93ecca3be8a16c155547182eb0dc/zeppelin-server/src/main/java/org/apache/zeppelin/server/ZeppelinServer.java#L223)
```java
webApp.addServlet(new ServletHolder("ws-events",new 
TorrentServlet()),"/download");
```
So then should it be like this ...
```java
public class BittorrentNotebookRepo extends VFSNotebookRepo implements 
TorrentEngineListener,
WebSocketCreator {
@Override
public Object createWebSocket(ServletUpgradeRequest servletUpgradeRequest,
 ServletUpgradeResponse servletUpgradeResponse) {
return new TorrentSocket(this);
  }
}
```
And the `TorrentServlet` class
```java
public class TorrentServlet extends WebSocketServlet {
  @Override
  public void configure(WebSocketServletFactory webSocketServletFactory) {
webSocketServletFactory.setCreator(new BittorrentNotebookRepo());
  }
}
```
Here is the problem as I cannot instantiate the BittorrentNotebookRepo here 
, it requires `conf` parameter. Also the NotebookRepos' are instantiated in 
`NotebookRepoSync`. 



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1254: [ZEPPELIN-1257] storage - fix get note revision api

2016-08-02 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1254
  
@khalidhuseynov thank you! 
Do you suggest merging this guy now and taking care of Revision separation 
later?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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-1268) As an enduser, I would like to embed paragraph and refresh the content of it in my custom UI

2016-08-02 Thread Kavin Kumar (JIRA)
Kavin Kumar created ZEPPELIN-1268:
-

 Summary: As an enduser, I would like to embed paragraph and 
refresh the content of it in my custom UI
 Key: ZEPPELIN-1268
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1268
 Project: Zeppelin
  Issue Type: Improvement
  Components: front-end, zeppelin-server, zeppelin-zengine
Reporter: Kavin Kumar


Being an end user of Apache Zeppelin, I would like to add the following 
enhancements to the codebase so that I can embed paragraph and also refresh the 
paragraph content from the custom UI.Note that the custom UI is different from 
the notebook UI and served by a simple js application.Embedding the paragraph 
content and refreshing the content of it in the custom UI are done using the 
Zeppelin REST APIs.

When we analyzed the existing APIs and the notebook UI, we thought that the 
following would be helpful when someone wants to do the same as mentioned above:

* Expose paragraph id as a new option for each paragraph in the notebook UI say 
"Get the paragraph id" so that the end user can easily find it out
* Create a new REST API to find out the status of a single paragraph



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


[GitHub] zeppelin issue #1260: ZEPPELIN-1267. PySparkInterpreter doesn't work in spar...

2016-08-02 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1260
  
\cc @jongyoul 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.
---


[GitHub] zeppelin issue #1248: [ZEPPELIN-1069]Ignore implicit interpreter when user e...

2016-08-02 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1248
  
👍  for having a test!

Looks good to me, modulo the issue raised above.


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

2016-08-02 Thread cloverhearts
Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/1258
  
@bzz Thank you for your feedback.
i will check to your opinion.
Thank you


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


[GitHub] zeppelin pull request #1260: ZEPPELIN-1267. PySparkInterpreter doesn't work ...

2016-08-02 Thread zjffdu
GitHub user zjffdu opened a pull request:

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

ZEPPELIN-1267. PySparkInterpreter doesn't work in spark 2.0

### What is this PR for?
PySparkInterpreter doesn't work in spark 2.0 because pyspark and py4j is 
not distributed to executors.  This PR extract the setup staff for pyspark 
interpreter into method setupConfForPySpark and use it for both spark1 and 
spark2. But this is just a short term solution, as I think this should be 
handled by spark rather than zeppelin, here zeppelin duplicate part of spark's 
work. In the long term, I'd like to resolve it in `ZEPPELIN-1263`.


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

### Todos
* https://issues.apache.org/jira/browse/ZEPPELIN-1263

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

### How should this be tested?
Verify it manually. 

### Screenshots (if appropriate)

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



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

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

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

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


commit 81d1d56c94934dd6c4b32144d810626356ce3c0e
Author: Jeff Zhang 
Date:   2016-08-02T09:41:21Z

ZEPPELIN-1267. PySparkInterpreter doesn't work in spark 2.0




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


[GitHub] zeppelin issue #1259: [ZEPPELIN-1260] R interpreter doesn't work with Spark ...

2016-08-02 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1259
  
Looks great to me, modulo question above.

Thank you for prompt fix @Leemoonsoo ! 
Shall this be merged to master or branch-0.6 as well? Asking as JIRA fix 
version is not set for this one.


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


[GitHub] zeppelin issue #1258: [ ZEPPELIN-1266 ] Code editor Optimization

2016-08-02 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1258
  
Would it make sense to change `isParagraphSaveAlready` => 
`wasParagraphAlreadySaved` ?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1259: [ZEPPELIN-1260] R interpreter doesn't work with...

2016-08-02 Thread Leemoonsoo
GitHub user Leemoonsoo opened a pull request:

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

[ZEPPELIN-1260] R interpreter doesn't work with Spark 2.0

### What is this PR for?
This PR fixes R interpreter doesn't work with Spark 2.0


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

### Todos
* [x] - Create and inject SparkSession into SparkRInterpreter

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

### How should this be tested?
Run Zeppelin with Spark 2.0 and run following codes and see if it returns 
`[1] 3`
```
%r 
localDF <- data.frame(name=c("a", "b", "c"), age=c(19, 23, 18))
df <- createDataFrame(spark, localDF)
count(df)
```

### Screenshots (if appropriate)

![image](https://cloud.githubusercontent.com/assets/1540981/17324196/fbf3df6c-586a-11e6-896f-68fd86e630e8.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/Leemoonsoo/zeppelin ZEPPELIN-1260

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

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


commit 6bd112811d81aa871b077c655fa0980802e3bc71
Author: Lee moon soo 
Date:   2016-08-02T09:30:23Z

Create and inject spark session into sparkr interpreter




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

2016-08-02 Thread cloverhearts
Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/1258
  
please, remove to browser cache


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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-1267) PySparkInterpreter doesn't work in spark 2.0

2016-08-02 Thread Jeff Zhang (JIRA)
Jeff Zhang created ZEPPELIN-1267:


 Summary: PySparkInterpreter doesn't work in spark 2.0
 Key: ZEPPELIN-1267
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1267
 Project: Zeppelin
  Issue Type: Bug
  Components: Interpreters
Affects Versions: 0.7.0
Reporter: Jeff Zhang
Assignee: Jeff Zhang


py4j is not distributed to executors. 
{noformat}
Error from python worker:
  /usr/bin/python: No module named pyspark
PYTHONPATH was:
  
/Users/jzhang/Temp/hadoop_tmp/nm-local-dir/usercache/jzhang/filecache/27/__spark_libs__3944184934921201174.zip/spark-core_2.11-2.0.0.jar
{noformat}



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


[GitHub] zeppelin pull request #1258: [ ZEPPELIN-1266 ] Code editor Optimization

2016-08-02 Thread cloverhearts
Github user cloverhearts closed the pull request at:

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


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

2016-08-02 Thread cloverhearts
Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/1258
  
ci trigger.


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

2016-08-02 Thread cloverhearts
GitHub user cloverhearts reopened a pull request:

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

[ ZEPPELIN-1266 ] Code editor Optimization

### What is this PR for?
If the result of many para- graph, the code editor is very slow.

### What type of PR is it?
Improvement

### Todos
- [x] removed call to function in template.
- [x] changed event (aceChange event to ace input event)  

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

### How should this be tested?
1. going to r tutorial notebook.
2. write to paragraph on anyway. (many many.. fast.. fast typing.)

### Screenshots (if appropriate)
 Before optimization.

![performance02](https://cloud.githubusercontent.com/assets/10525473/17323111/a1cd9b2e-58db-11e6-8d61-7ab98ea96b3a.gif)

 After optimization.

![performance01](https://cloud.githubusercontent.com/assets/10525473/17323107/a02b5338-58db-11e6-95c8-543aab7131dd.gif)

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


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

$ git pull https://github.com/cloverhearts/zeppelin dev/aceeditorperformance

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

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


commit 86ba5c42a6eb9ea9ba22d90c11dbb823e5b81ce2
Author: CloverHearts 
Date:   2016-08-02T07:19:30Z

ace editor performance up

commit 979e2f3a05bdeba14f297d0bcb4e85d1e593424a
Author: CloverHearts 
Date:   2016-08-02T08:20:28Z

ace editor performance up - remove call to template function

commit af5e18f275ebe0850649e7956c1be3673214aaf0
Author: CloverHearts 
Date:   2016-08-02T08:38:09Z

ace editor optimize - get isChange paragraph text value in template




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

2016-08-02 Thread cloverhearts
GitHub user cloverhearts opened a pull request:

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

[ ZEPPELIN-1266 ] Code editor Optimization

### What is this PR for?
If the result of many para- graph, the code editor is very slow.

### What type of PR is it?
Improvement

### Todos
- [x] removed call to function in template.
- [x] changed event (aceChange event to ace input event)  

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

### How should this be tested?
1. going to r tutorial notebook.
2. write to paragraph on anyway. (many many.. fast.. fast typing.)

### Screenshots (if appropriate)
 Before optimization.

![performance02](https://cloud.githubusercontent.com/assets/10525473/17323111/a1cd9b2e-58db-11e6-8d61-7ab98ea96b3a.gif)

 After optimization.

![performance01](https://cloud.githubusercontent.com/assets/10525473/17323107/a02b5338-58db-11e6-95c8-543aab7131dd.gif)

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


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

$ git pull https://github.com/cloverhearts/zeppelin dev/aceeditorperformance

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

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


commit 86ba5c42a6eb9ea9ba22d90c11dbb823e5b81ce2
Author: CloverHearts 
Date:   2016-08-02T07:19:30Z

ace editor performance up

commit 979e2f3a05bdeba14f297d0bcb4e85d1e593424a
Author: CloverHearts 
Date:   2016-08-02T08:20:28Z

ace editor performance up - remove call to template function

commit af5e18f275ebe0850649e7956c1be3673214aaf0
Author: CloverHearts 
Date:   2016-08-02T08:38:09Z

ace editor optimize - get isChange paragraph text value in template




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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-1266) If the result of many paragraph, the code editor is very slow

2016-08-02 Thread CloverHearts (JIRA)
CloverHearts created ZEPPELIN-1266:
--

 Summary: If the result of many paragraph, the code editor is very 
slow
 Key: ZEPPELIN-1266
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1266
 Project: Zeppelin
  Issue Type: Bug
Reporter: CloverHearts
Assignee: CloverHearts


If the result of many paragraph, the code editor is very slow.
for example, writing to r tutorial notebook



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


[GitHub] zeppelin issue #1210: [ZEPPELIN-1220] Add geographical map as visualization ...

2016-08-02 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/1210
  
@mwpenny 

Alright, so after taking a look, here are my feedbacks:

 Overall PR
* License of angular-esri-map will need to be added in [this 
file](https://github.com/apache/zeppelin/blob/master/zeppelin-distribution/src/bin_license/LICENSE)
 (That folder also contains single files if license needs to be copied)
* There is 2 files to commit: `index.html` and `karma.conf.js` which have 
changes after we build.
* I think it could be nice to have a page in the documentation, but let's 
make that in a different PR.

 The Map
* The mouse scroll to zoom in the map should be deactivated, if you scroll 
down the notebook you might zoom out and then the map vanishes.
* Could we have the map unzoomed to show the world when loaded (Its fine to 
keep it centered on Canada ;) )
* The Map height do not scale when we make the paragraph bigger
* The Map is going out of the paragraph instead of being contained or 
fitting the size:
https://cloud.githubusercontent.com/assets/710411/17321366/31e0d1a8-58d3-11e6-8a17-e383f8912b3b.png";>
* When I click on a Pin it moves me somewhere else

![mappin](https://cloud.githubusercontent.com/assets/710411/17321637/90e4d054-58d4-11e6-95ed-f8d2e63f78c7.gif)


 Options & Pivot
* Let's remove the `height:36px` style for the pivot area, I know it was 
copy pasted from other place in the code, but its better to have all of them 
with the same height.
* The BaseMap option isn't removed when we switch graphs, seems to be a 
mistake 
[here](https://github.com/mwpenny/zeppelin-esri/blob/esri/zeppelin-web/src/app/notebook/paragraph/paragraph-graphOptions.html#L31)
 with ng-if inside a class


I will spend more time on checking the code, and would leave inline 
comments if there is


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


  1   2   >