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

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

https://github.com/apache/zeppelin/pull/1451
  
@astroshim Appreciate for your effort! Merge into master if there are no 
more discussions :)


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


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

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

https://github.com/apache/zeppelin/pull/1451
  
@AhyoungRyu Thank you so much for taking care of this!!


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


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

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

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

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

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

[INFO] BUILD FAILURE
[INFO] 

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

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

Looks good 👍 




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


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

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

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


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


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

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

https://github.com/apache/zeppelin/pull/1451
  
@astroshim Yeah that would be better than all contents are together in one 
page actually. Thanks for your consideration :D


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


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

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

https://github.com/apache/zeppelin/pull/1451
  
@AhyoungRyu Thank you for your suggestion. I think creating new page for 
`Zeppelin on CDH` is better, 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 #1451: [ZEPPELIN-1279] Zeppelin with CDH5.x docker document.

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

https://github.com/apache/zeppelin/pull/1451
  
@astroshim Thanks for your contribution! This can be really helpful to 
`CDH` & `Spark` & `Zeppelin` users.

https://cloud.githubusercontent.com/assets/10060731/18774378/6c93931a-8193-11e6-8720-8e7f9bf09549.png";
 width="500px">

How about separating `Zeppelin on Spark Cluster Mode` page with `Zeppelin 
on CDH` instead of putting this contents in one page?  Then you need to create 
`docs/install/cdh.md` maybe.
I think it's more makes to have only `Spark Cluster Mode` contents in 
`docs/install/spark_cluster_mode.md`.

Or there is an another option. If you think the contents in `Zeppelin on 
CDH` is too short to have a separated page, you can just modify the 
title("Apache Zeppelin on Spark Cluster Mode") and the contents of "Overview" 
in `spark_cluster_mode.md`. And maybe the file name needs to be changed 
accordingly :)

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