[GitHub] zeppelin issue #1302: [ZEPPELIN-1306] Interpreter restarts on a note.

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

https://github.com/apache/zeppelin/pull/1302
  
@astroshim Definitely looks better! I like @Leemoonsoo 's idea 👍 
It's just my personal opinion, how about putting `refresh` icon to the left 
side of the interpreter name? I mean

 - Before
https://cloud.githubusercontent.com/assets/10060731/18031438/96ca6208-6d1b-11e6-9022-109d7e6fe85c.png;>

 - After 
https://cloud.githubusercontent.com/assets/10060731/18031439/9d8905cc-6d1b-11e6-987a-ec9dd1f8a4e9.png;>

And while we are familiar with the feature; "Restart Interpreter", some ppl 
may not. So what about showing some message with tooltip like this?
https://cloud.githubusercontent.com/assets/10060731/18031452/6acf5a40-6d1c-11e6-912e-4c5af35e68ed.png;>

What do you think?


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


[GitHub] zeppelin issue #1372: [ZEPPELIN-1383][ Interpreters][r-interpreter] remove S...

2016-08-27 Thread WeichenXu123
Github user WeichenXu123 commented on the issue:

https://github.com/apache/zeppelin/pull/1372
  
cc @jongyoul @Leemoonsoo 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 #1368: zeppelin-display dependency version built only for Sca...

2016-08-27 Thread lresende
Github user lresende commented on the issue:

https://github.com/apache/zeppelin/pull/1368
  
Currently build, by default, is configured to support Scala 2.10, thus just 
executing the build works. 

In order to build with Scala 2.11, you need to execute:
```
./dev/change_scala_version.sh 2.11
mvn . -Pscala2.11
```

Without that, it won't and should not work, because the dependencies will 
try to match the ones built for the scala version being used.


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

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

https://github.com/apache/zeppelin/pull/1366
  
Tested, LGTM. 
Merge to master and branch-0.6 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 #1283: Buffer append output results + fix extra incorrect res...

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

https://github.com/apache/zeppelin/pull/1283
  
According to the test results,

%sh interpreter output streaming might be other issue.
but it looks like %spark interpreter output streaming in this branch is not 
working as expected.

@beriaanirudh Could you take care of it?


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


[GitHub] zeppelin issue #1368: zeppelin-display dependency version built only for Sca...

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

https://github.com/apache/zeppelin/pull/1368
  
Thanks @krystiannowak for contribution.

I found that there is another issue for having `zeppelin-display` as a 
dependency of `spark` module. Here's related issue 
https://issues.apache.org/jira/browse/ZEPPELIN-1384.

To resolve ZEPPELIN-1384, i think zeppelin-display should be removed from 
the dependencies section and provide alternative way to load zeppelin-display.


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


[jira] [Created] (ZEPPELIN-1384) Spark interpreter binary compatibility to scala 2.10 / 2.11 broken

2016-08-27 Thread Lee moon soo (JIRA)
Lee moon soo created ZEPPELIN-1384:
--

 Summary: Spark interpreter binary compatibility to scala 2.10 / 
2.11 broken 
 Key: ZEPPELIN-1384
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1384
 Project: Zeppelin
  Issue Type: Bug
Affects Versions: 0.6.1
Reporter: Lee moon soo


Scala 2.11 support in SparkInterpreter designed to provide binary compatibility 
to scala 2.10 and scala 2.11.

i.e. changing SPARK_HOME should be enough to make Zeppelin work with any 
version of spark without rebuild. Spark 2.0.0 (with scala 2.11), Spark 1.6.2 
(with scala 2.10), and so on.

The reason is spark interpreter has zeppelin-display as a dependency. and 
zeppelin-display brings particular version of scala library.

Solution could be exclude zeppelin-display from spark interpreter's dependency, 
and let user load zeppelin-display from maven repository dynamically by using 
dependency loader, when it's needed.



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


[GitHub] zeppelin issue #1302: [ZEPPELIN-1306] Interpreter restarts on a note.

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

https://github.com/apache/zeppelin/pull/1302
  
\cc @Leemoonsoo I fixed UI like following screenshot and process. please 
review.

![o](https://cloud.githubusercontent.com/assets/3348133/18028272/33d20810-6cb5-11e6-9026-b51ddf8eb972.gif)


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

2016-08-27 Thread WeichenXu123
GitHub user WeichenXu123 opened a pull request:

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

[ZEPPELIN-1383][ Interpreters][r-interpreter] remove 
SparkInterpreter.getSystemDefault and update relative code

### What is this PR for?
clean some redundant code:
remove `SparkInterpreter.getSystemDefault` methods,
and replace it with `InterpreterProperty.getValue`

### What type of PR is it?
Improvement

### Todos
N/A

### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-1383
remove SparkInterpreter.getSystemDefault and update relative code

### How should this be tested?
Existing tests.

### 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/WeichenXu123/zeppelin 
remove_spark_interpreter_getSystemDefault

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

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


commit cdea57cf716cf2959273c7dc7780ebd5c7abee77
Author: WeichenXu 
Date:   2016-08-27T13:51:30Z

update.




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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-1383) remove SparkInterpreter.getSystemDefault and update relative code

2016-08-27 Thread Weichen Xu (JIRA)
Weichen Xu created ZEPPELIN-1383:


 Summary: remove SparkInterpreter.getSystemDefault and update 
relative code
 Key: ZEPPELIN-1383
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1383
 Project: Zeppelin
  Issue Type: Improvement
  Components: Interpreters, r-interpreter
Affects Versions: 0.7.0
Reporter: Weichen Xu


As the TODO in `InterpreterProperty` says,
we should remove `SparkInterpreter.getSystemDefault` methods,
because it can be replaced with `InterpreterProperty.getValue`
so that we can remove redundant code.



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


[GitHub] zeppelin issue #1371: [ZEPPELIN-1372]Automatically Detect the data type in t...

2016-08-27 Thread arunsoman
Github user arunsoman commented on the issue:

https://github.com/apache/zeppelin/pull/1371
  
It will be further beneficial if zeppelin could detect the column type, 
eg(date, number, string) and also if the field shows the property of measure or 
dimension; if zeppelin could dispatch such details then the display system can 
make out what charts to show to user, or user could make more informed choice  
when creating a report.  


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

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

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


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

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

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

ZEPPELIN-1342. Adding dependencies via SPARK_SUBMIT_OPTIONS doesn't work on 
Spark 2.0.0

### What is this PR for?

The root cause is due to the change of repl of scala-2.11. User needs to 
specify the jars in the repl setting explicitly.

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

### Todos
* [ ] - Task

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

### How should this be tested?
Tested manually as shown in the screenshot. 

### Screenshots (if appropriate)


![image](https://cloud.githubusercontent.com/assets/164491/17997416/f26c262c-6ba0-11e6-8586-22a6a633b21b.png)

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



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

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

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

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


commit 56925d61e937fd8b14590a011c1139d3a1b27e4a
Author: Jeff Zhang 
Date:   2016-08-26T07:19:51Z

ZEPPELIN-1342. Adding dependencies via SPARK_SUBMIT_OPTIONS doesn't work on 
Spark 2.0.0

commit 10f64e6264e9533209ab7d8b890945848672870a
Author: Jeff Zhang 
Date:   2016-08-27T02:34:50Z

fix test 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 #1365: ZEPPELIN-1374. Should prevent use dot in interpreter n...

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

https://github.com/apache/zeppelin/pull/1365
  
@corneadoug make sense, I will update the PR


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


[GitHub] zeppelin issue #1368: zeppelin-display dependency version built only for Sca...

2016-08-27 Thread krystiannowak
Github user krystiannowak commented on the issue:

https://github.com/apache/zeppelin/pull/1368
  
@lresende It seems that when just executing
```
mvn clean package -Pspark-1.6 -Phadoop-2.4 -Pyarn -Ppyspark -Pscala-2.11 
-DskipTests clean package
```
it fails
```
[INFO] 

[INFO] Reactor Summary:
[INFO]
[INFO] Zeppelin .. SUCCESS [  8.430 
s]
[INFO] Zeppelin: Interpreter . SUCCESS [ 54.976 
s]
[INFO] Zeppelin: Zengine . SUCCESS [ 36.888 
s]
[INFO] Zeppelin: Display system apis . SUCCESS [01:09 
min]
[INFO] Zeppelin: Spark dependencies .. SUCCESS [04:25 
min]
[INFO] Zeppelin: Spark ... FAILURE [  5.676 
s]
[INFO] Zeppelin: Markdown interpreter  SKIPPED
[INFO] Zeppelin: Angular interpreter . SKIPPED
[INFO] Zeppelin: Shell interpreter ... SKIPPED
[INFO] Zeppelin: Livy interpreter  SKIPPED
[INFO] Zeppelin: HBase interpreter ... SKIPPED
[INFO] Zeppelin: PostgreSQL interpreter .. SKIPPED
[INFO] Zeppelin: JDBC interpreter  SKIPPED
[INFO] Zeppelin: File System Interpreters  SKIPPED
[INFO] Zeppelin: Flink ... SKIPPED
[INFO] Zeppelin: Apache Ignite interpreter ... SKIPPED
[INFO] Zeppelin: Kylin interpreter ... SKIPPED
[INFO] Zeppelin: Python interpreter .. SKIPPED
[INFO] Zeppelin: Lens interpreter  SKIPPED
[INFO] Zeppelin: Apache Cassandra interpreter  SKIPPED
[INFO] Zeppelin: Elasticsearch interpreter ... SKIPPED
[INFO] Zeppelin: BigQuery interpreter  SKIPPED
[INFO] Zeppelin: Alluxio interpreter . SKIPPED
[INFO] Zeppelin: web Application . SKIPPED
[INFO] Zeppelin: Server .. SKIPPED
[INFO] Zeppelin: Packaging distribution .. SKIPPED
[INFO] 

[INFO] BUILD FAILURE
[INFO] 

[INFO] Total time: 07:21 min
[INFO] Finished at: 2016-08-27T12:30:32+01:00
[INFO] Final Memory: 63M/903M
[INFO] 

[ERROR] Failed to execute goal on project zeppelin-spark_2.10: Could not 
resolve dependencies for project 
org.apache.zeppelin:zeppelin-spark_2.10:jar:0.7.0-SNAPSHOT: Could not find
 artifact org.apache.zeppelin:zeppelin-display_2.11:jar:0.7.0-SNAPSHOT in 
apache.snapshots (http://repository.apache.org/snapshots) -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e 
switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, 
please read the following articles:
[ERROR] [Help 1] 
http://cwiki.apache.org/confluence/display/MAVEN/DependencyResolutionException
[ERROR]
[ERROR] After correcting the problems, you can resume the build with the 
command
[ERROR]   mvn  -rf :zeppelin-spark_2.10
```
and the PR fixes that.
I cannot currently use the bash script referenced (which just sed-s Scala 
versions in POM files), as currently I am using Windows, but hopefully after 
applying this fix it won't be necessary. I was also looking at the previous 
changes made by doing https://issues.apache.org/jira/browse/ZEPPELIN-1179 and 
it seems the fix is aligned with that approach. What do you think?


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


[jira] [Created] (ZEPPELIN-1382) Websocket reconnect button

2016-08-27 Thread chris snow (JIRA)
chris snow created ZEPPELIN-1382:


 Summary: Websocket reconnect button 
 Key: ZEPPELIN-1382
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1382
 Project: Zeppelin
  Issue Type: New Feature
  Components: Core
Reporter: chris snow


When using zeppelin via ssh tunnels [e.g. 1], the websocket will get 
disconnected and require reloading the browser to reconnect - this loses any 
changes you have performed since disconnected.

It would be good if there was functionality to allow the websocket to reconnect 
under these circumstances without reloading the browser.

[1] 
https://azure.microsoft.com/en-gb/documentation/articles/hdinsight-apache-spark-use-zeppelin-notebook/



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


[GitHub] zeppelin issue #1353: ZEPPELIN-1326: make profile to select dependency of ha...

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

https://github.com/apache/zeppelin/pull/1353
  
Merging this if no more discussion.


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


[GitHub] zeppelin pull request #1362: [ZEPPELIN-1178] Tooltip: Show chart type when h...

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

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1365: ZEPPELIN-1374. Should prevent use dot in interpreter n...

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

https://github.com/apache/zeppelin/pull/1365
  
We might also want to make sure that the Websocket API and basic API (if 
one exist), also check the name accordingly


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1368: zeppelin-display dependency version built only for Sca...

2016-08-27 Thread krystiannowak
Github user krystiannowak commented on the issue:

https://github.com/apache/zeppelin/pull/1368
  
Please try to build the whole project with 2.11 profile turned on. And
possibly clean your local Zeppelin artifacts.

Cheers,
Krystian

27.08.2016 04:43 "Luciano Resende"  napisał(a):
>
> This is also how it's documented on the readme.md
> https://github.com/apache/zeppelin#building-for-scala-211
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub, or mute the thread.



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