[GitHub] zeppelin pull request #1262: [ZEPPELIN-1268] As an enduser, I would like to ...

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

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


---
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] As an enduser, I would like to embed p...

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

https://github.com/apache/zeppelin/pull/1262
  
@kavinkumarks So sorry for the late reply. I'll merge 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 #1336: [ZEPPELIN-1333] prevent calling runParagraph() ...

2016-08-17 Thread nazgul33
Github user nazgul33 closed the pull request at:

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


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


[GitHub] zeppelin issue #1290: [ZEPPELIN-708]shift positions automatically to ...

2016-08-17 Thread resec
Github user resec commented on the issue:

https://github.com/apache/zeppelin/pull/1290
  
@mwkang I am excited to see this enhancement implemented, great 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 #1262: [ZEPPELIN-1268] As an enduser, I would like to embed p...

2016-08-17 Thread kavinkumarks
Github user kavinkumarks commented on the issue:

https://github.com/apache/zeppelin/pull/1262
  
Could someone take care of merging this? NOTE that the review was over.


---
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 #1318: [ZEPPELIN-1280][Spark on Yarn] Documents for running z...

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

https://github.com/apache/zeppelin/pull/1318
  
Thanks @astroshim, I have no other concerns. 


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


Re: Zeppelin web development tools

2016-08-17 Thread onkar shedge
Thanks Corneau .

On Thu, Aug 18, 2016 at 4:56 AM, Corneau Damien 
wrote:

> Hi Onkar,
>
> `./grunt serve` is the best way to go, also everything in that mode should
> be the same as after build.
>
> I just tried and import dialog is showing, so if there is issues, you might
> want to clean your cache, or might
> have forgot to add the newly created .js file in the index.html (
> https://github.com/apache/zeppelin/blob/master/zeppelin-
> web/CONTRIBUTING.md#new-files-includes
> )
>
> Otherwise, Zeppelin is splitted in submodules, and you do not need to build
> the whole project for changes in one module.
> While you are running the server, you could just do: `./grunt build` (in
> zeppelin-web) in another term, and it would update the running web-app with
> new code compiled.
>
> On Wed, Aug 17, 2016 at 9:01 PM, onkar shedge 
> wrote:
>
> > Hello corneadoug,
> >
> > What tools should I use to quickly see runtime changes of my javascript
> and
> > html.
> > Right now I am changing the respective .js .html files and doing mvn
> > package on Zeppelin root project. This is very time consuming it takes 20
> > to 30 min for packaging.
> > Running ./grunt serve
> > serves pages on localhost:9000 but there is no import dialog in it.
> >
> > Regards,
> > Onkar Shedge
> >
>


[GitHub] zeppelin pull request #1325: Add an Apache Zeppelin Logo section to the webs...

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

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


---
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 #1318: [ZEPPELIN-1280][Spark on Yarn] Documents for running z...

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

https://github.com/apache/zeppelin/pull/1318
  
It would be better to change the title to reflect the docker.  I think we 
should mention docker is only for small experimental enviroment rather than 
production environment.  Besides that, I don't know how much complicated of 
using docker, I would be more conservative to bring extras dependencies, 
especially when it is complicated and not usually needed in real environment. 


---
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 #1318: [ZEPPELIN-1280][Spark on Yarn] Documents for running z...

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

https://github.com/apache/zeppelin/pull/1318
  
@zjffdu You're right, usually users don't make their production using 
docker.
but We have been received many question about running Zeppelin on their 
production environments so I think this PR gives hints users to solve their 
problems and making their production environments too.
Does this make sense?


---
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 #1210: [ZEPPELIN-1220] Add geographical map as visualization ...

2016-08-17 Thread mwpenny
Github user mwpenny commented on the issue:

https://github.com/apache/zeppelin/pull/1210
  
@corneadoug When I test it without the CSS, the tiles still load but 
they're not positioned properly (they extend outside of the 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 pull request #1339: [WIP][ZEPPELIN-1332] Remove spark-dependencies ...

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

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

[WIP][ZEPPELIN-1332] Remove spark-dependencies & suggest new way

### What is this PR for?
Currently, Zeppelin's embedded Spark is located under `interpreter/spark/`. 
For whom **builds Zeppelin from source**, this Spark is downloaded when they 
build the source with [build 
profiles](https://github.com/apache/zeppelin#spark-interpreter). I think this 
various build profiles are useful to customize the embedded Spark, but many 
Spark users are using their own Spark not Zeppelin's embedded one. Nowadays, 
only Spark beginners use this embedded Spark. For them, there are too 
many build profiles(it's so complicated i think). In case of **Zeppelin binary 
package**, it's included by default under `interpreter/spark/`. That's why 
Zeppelin package size is so huge. 

This PR will change the embedded Spark binary downloading mechanism as like 
below.

1. If user didn't set their own `SPARK_HOME`, 
[bin/download-spark.sh](https://github.com/AhyoungRyu/zeppelin/blob/5703fbf27fedda9ec7dd142e275b8654c9bc6296/bin/download-spark.sh)
 will be run when they start Zeppelin server using `bin/zeppelin-daemon.sh` or 
`bin/zeppelin.sh`.
2. 
[bin/download-spark.sh](https://github.com/AhyoungRyu/zeppelin/blob/5703fbf27fedda9ec7dd142e275b8654c9bc6296/bin/download-spark.sh)
 : Download `spark-2.0.0-bin-hadoop2.7.tgz` from mirror site to 
`$ZEPPELIN_HOME/.spark-dist/` and untar -> Set `SPARK_HOME` as 
`$ZEPPELIN_HOME/.spark-dist/spark-2.0.0-bin-hadoop2.7` -> add this `SPARK_HOME` 
to `conf/zeppelin-env.sh`

With this new mechanism, we can not only reduce Zeppelin overall binary 
package size but also user doesn't need to type complicating build profiles 
when they build Zeppelin source.

### What type of PR is it?
Improvement

### Todos
* [ ] - update 
[README.md](https://github.com/apache/zeppelin/blob/master/README.md)
* [ ] - add `download-spark.cmd` for Window users 

### What is the Jira issue?
See [ZEPPELIN-1332](https://issues.apache.org/jira/browse/ZEPPELIN-1332)'s 
description for the details about **Why we need to remove spark-dependencies** 
& **New suggestion for Zeppelin's embedded Spark binary**.


### How should this be tested?
After apply this patch, build with `mvn clean package -DskipTests`. Please 
note that you need to check `spark-dependencies` is removed well or not.
 - Without prespecified `SPARK_HOME` 
  1. Start Zeppelin daemon
  https://cloud.githubusercontent.com/assets/10060731/17759836/e3c16022-6535-11e6-8576-43975c3293c3.png;>
  2. Check `conf/zeppelin-env.sh` line 46. `SPARK_HOME` will be set like 
below 
  ```
  export 
SPARK_HOME="/YOUR_ZEPPELIN_HOME/.spark-dist/spark-2.0.0-bin-hadoop2.7"
  ```
  3. Go to Zeppelin website and run `sc.version` with Spark interpreter & 
`echo $SPARK_HOME` with sh interpreter.
  https://cloud.githubusercontent.com/assets/10060731/17759937/a7bcc584-6536-11e6-9664-cffdc6e5bdf8.png;>

 - With prespecified `SPARK_HOME`
Nothing happened. Zeppelin will be started as like before.
 
### Screenshots (if appropriate)

### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? need to update 
[README.md](https://github.com/apache/zeppelin/blob/master/README.md)


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

$ git pull https://github.com/AhyoungRyu/zeppelin ZEPPELIN-1332

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

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


commit ae74e90f8409b7396eeebf34c103a6db071b1771
Author: AhyoungRyu 
Date:   2016-08-16T15:08:19Z

Fix typo comment in interpreter.sh

commit ada6f37d1df60f37740d63c913cdd89f7b919269
Author: AhyoungRyu 
Date:   2016-08-17T01:52:06Z

Remove spark-dependencies

commit 87b929d7d38e447306796cec44b35cb7317b9bb3
Author: AhyoungRyu 
Date:   2016-08-17T07:14:35Z

Add spark-2.*-bin-hadoop* to .gitignore

commit 5703fbf27fedda9ec7dd142e275b8654c9bc6296
Author: AhyoungRyu 
Date:   2016-08-17T15:22:25Z

Add download-spark.sh file

commit 35350bb9990436cd7ede1e611f0b94a56ed24793
Author: AhyoungRyu 
Date:   2016-08-17T15:28:51Z

Remove useless comment line in common.sh




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

[GitHub] zeppelin issue #1318: [ZEPPELIN-1280][Spark on Yarn] Documents for running z...

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

https://github.com/apache/zeppelin/pull/1318
  
The jira title seems a little confusing to me. The PR is for running spark 
on yarn by docker, but I don't think users will use docker for production. 


---
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 #1326: [ZEPPELIN-1323] Add contribution guide for Zeppelin do...

2016-08-17 Thread felixcheung
Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1326
  
Merging if there is no more discussion.



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


[GitHub] zeppelin pull request #1338: [WIP ] Zeppelin 1307 - Implement notebook revis...

2016-08-17 Thread anthonycorbacho
GitHub user anthonycorbacho reopened a pull request:

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

[WIP ] Zeppelin 1307 - Implement notebook revision in Zeppelinhub repo

### What is this PR for?
Implement versioning in ZeppelinHub notebook storage.

### What type of PR is it?
Improvement

### Todos
* [ ] - Implement Versioning API

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

### How should this be tested?
Edit `zeppelin-env.sh` and add 
`org.apache.zeppelin.notebook.repo.zeppelinhub.ZeppelinHubRepo` in 
`ZEPPELIN_NOTEBOOK_STORAGE`.

### 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/anthonycorbacho/incubator-zeppelin 
ZEPPELIN-1307

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

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


commit 9bd0946233f8b8ffd0a66effe46bb66303cc676b
Author: Anthony Corbacho 
Date:   2016-08-17T03:54:07Z

Close InputStream in asyncGet

commit 3942a781b55c780de5aa11038f62d6494b620bdd
Author: Anthony Corbacho 
Date:   2016-08-17T03:58:44Z

Implement get revision

commit 5d4b54b3901a23eb396bded45a2f95565afa7610
Author: Anthony Corbacho 
Date:   2016-08-17T05:45:52Z

Implement checkpoint method

commit 2395a6ea2a70f39fa95b7b26bc461518e3653326
Author: Anthony Corbacho 
Date:   2016-08-17T06:56:54Z

Light refactor of ZeppelinHubRestapiHandler and extract api call to a 
single method

commit 74a0cdbb92908b7b7c9bc104e2541d32a22dacbe
Author: Anthony Corbacho 
Date:   2016-08-17T06:59:17Z

change asyncPutWithResponseBody to accpet url instead of noteId




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


Re: Zeppelin web development tools

2016-08-17 Thread Corneau Damien
Hi Onkar,

`./grunt serve` is the best way to go, also everything in that mode should
be the same as after build.

I just tried and import dialog is showing, so if there is issues, you might
want to clean your cache, or might
have forgot to add the newly created .js file in the index.html (
https://github.com/apache/zeppelin/blob/master/zeppelin-web/CONTRIBUTING.md#new-files-includes
)

Otherwise, Zeppelin is splitted in submodules, and you do not need to build
the whole project for changes in one module.
While you are running the server, you could just do: `./grunt build` (in
zeppelin-web) in another term, and it would update the running web-app with
new code compiled.

On Wed, Aug 17, 2016 at 9:01 PM, onkar shedge 
wrote:

> Hello corneadoug,
>
> What tools should I use to quickly see runtime changes of my javascript and
> html.
> Right now I am changing the respective .js .html files and doing mvn
> package on Zeppelin root project. This is very time consuming it takes 20
> to 30 min for packaging.
> Running ./grunt serve
> serves pages on localhost:9000 but there is no import dialog in it.
>
> Regards,
> Onkar Shedge
>


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

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

https://github.com/apache/zeppelin/pull/1334
  
@felixcheung the requied changes is done.


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


[jira] [Created] (ZEPPELIN-1342) Adding dependencies via SPARK_SUBMIT_OPTIONS doesn't work on Spark 2.0.0

2016-08-17 Thread Mike Sells (JIRA)
Mike Sells created ZEPPELIN-1342:


 Summary: Adding dependencies via SPARK_SUBMIT_OPTIONS doesn't work 
on Spark 2.0.0
 Key: ZEPPELIN-1342
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1342
 Project: Zeppelin
  Issue Type: Bug
  Components: Interpreters
Affects Versions: 0.6.1, 0.7.0
Reporter: Mike Sells


Passing dependencies via the SPARK_SUBMIT_OPTIONS seems to be broken in 0.6.1 
and 0.7.0 when using Spark 2.0. To replicate pass in a package to 
SPARK_SUBMIT_OPTIONS like below:

SPARK_SUBMIT_OPTIONS="--packages com.databricks:spark-avro_2.11:3.0.0"

Now try to import it with:
import com.databricks.spark.avro._

It will error out:
:25: error: object databricks is not a member of package com
   import com.databricks.spark.avro._

I checked the logs are there is no error retrieving the package. So it seems to 
be something with the classpath.



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


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

2016-08-17 Thread mwpenny
Github user mwpenny commented on the issue:

https://github.com/apache/zeppelin/pull/1210
  
@karup1990 This is very strange. I cannot reproduce the issue using the 
same browser version and your network doesn't seem to be the problem since it 
is working for you in Safari.

The only other thing I can think of is a browser setting or extension 
interfering somehow. Try running Chrome with the `--disable-extensions` 
argument. If that doesn't work, try creating a new Chrome user profile and 
using that temporarily.


---
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 #1210: [ZEPPELIN-1220] Add geographical map as visualization ...

2016-08-17 Thread mwpenny
Github user mwpenny commented on the issue:

https://github.com/apache/zeppelin/pull/1210
  
@karup1990 Can you send a screenshot of the map? If you look at the network 
requests in your browser's developer tools, are the requests to the ArcGIS 
servers successful (i.e., retrieval of the map JavaScript and CSS)? Here's a 
screenshot of what happens when I load the map (I'm using Firefox):

![resload](https://cloud.githubusercontent.com/assets/9356790/17742603/afcf633a-646e-11e6-9162-fa1aa399d904.png)

Also check to make sure the map tile requests are succeeding. Screenshot 
from my browser:

![tileload](https://cloud.githubusercontent.com/assets/9356790/17742634/d480a37e-646e-11e6-9f08-6076652458ca.png)


---
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 #1210: [ZEPPELIN-1220] Add geographical map as visualization ...

2016-08-17 Thread karup1990
Github user karup1990 commented on the issue:

https://github.com/apache/zeppelin/pull/1210
  
@mwpenny Thanks for the awesome work.
I was trying out the visualizations.
I can see only the pins, the maps seems to not load.
Any idea what could be wrong?


---
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 #1313: [ZEPPELIN-1245] Focus first paragraph after notebook c...

2016-08-17 Thread rajarajan-g
Github user rajarajan-g commented on the issue:

https://github.com/apache/zeppelin/pull/1313
  
@Leemoonsoo : Thanks for reviewing the code

I did check the existing behaviour , for existing note it doesn't focus to 
any paragraph.

So,I did code changes now in such way, if we create notebook/open an 
existing notebook, it will focus on the first paragraph.

For existing notebook, i do focus to end of the paragraph.I hope those 
changes are fine.  


---
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-17 Thread echarles
Github user echarles commented on the issue:

https://github.com/apache/zeppelin/pull/932
  
@bzz I have moved  `LICENSE-jersery-2` to 
`zeppelin-distribution/src/bin_license`.


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


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

2016-08-17 Thread felixcheung
Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1334
  
Let's make sure the new doc is added to the list in _navigation.html?

https://zeppelin.apache.org/docs/0.6.1/development/writingzeppelininterpreter.html#contributing-a-new-interpreter-to-zeppelin-releases



---
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 #1315: [ZEPPELIN-530] Added changes for Credential Provider, ...

2016-08-17 Thread rconline
Github user rconline commented on the issue:

https://github.com/apache/zeppelin/pull/1315
  
@jongyoul It does not matter, because we are just depending upon hadoop 2.6 
credential api's. So if the user is on Hadoop 1.x, then he just needs to add 
Hadoop common - 2.6 jars to the classpath to create the jceks file. At the time 
of authentication, this jar is already present inside the zeppelin artifact, so 
we are good 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 #1334: [ZEPPELIN-682] New interpreter for Apache Beam (incuba...

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

https://github.com/apache/zeppelin/pull/1334
  
@bzz Kindly be noted that there is no maven Scala 2.11 build for beam 
runner Flink. The only available build is beam-runners-flink_2.10 .

https://mvnrepository.com/artifact/org.apache.beam/beam-runners-flink_2.10/0.1.0-incubating
Our pom file :
```org.apache.beam
beam-runners-flink_2.10
${beam.beam.version}
```
That is why we hardcode it.
While the failure logs claims that maven can not find 
`beam-runners-flink_2.11`
It seems that your builder enforces the usage of Scala 2.11


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


[GitHub] zeppelin issue #1290: [ZEPPELIN-708]shift positions automatically to ...

2016-08-17 Thread resec
Github user resec commented on the issue:

https://github.com/apache/zeppelin/pull/1290
  
@mwkang  Just a minor suggestion, when presses  and focus 
moves to a new cell, is it better that the focused line is next to the [%xxx]? 
so users can start typing new code right away without a . Personally I 
keep the same [%] within a notebook far more than changing it in most of 
the cases.


---
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 #1242: [ZEPPELIN-960] When there is no interpreter, paragraph...

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

https://github.com/apache/zeppelin/pull/1242
  
Great work @cloverhearts ! Few minor comments above.

Also, looks like CI is failing, and if you could post here the reason - 
other people could help determining if that is relevant to your change, or can 
be ignored.


---
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 #1336: [ZEPPELIN-1333] prevent calling runParagraph() on shif...

2016-08-17 Thread nazgul33
Github user nazgul33 commented on the issue:

https://github.com/apache/zeppelin/pull/1336
  
> However, @nazgul33 next time it would be better to make a separate branch 
instead of making the PR from master.

can't agree more. lol


---
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 #1256: [ ZEPPELIN-1259 ] create ace.editor configuration mana...

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

https://github.com/apache/zeppelin/pull/1256
  
@corneadoug You're welcome, 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 #1135: Add force layout visualization in paragraphs for SQL q...

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

https://github.com/apache/zeppelin/pull/1135
  
@davidtzoor sorry for the long wait, I was trying to finish a few things 
before looking closer into this PR.

There was another visualization that got merged, so you will have to rebase.

Can you also provide a few things to make it easier for me to review:
* Some code (paragraph) that create or import a data set necessary to use 
that graph, you can update the **How should this be tested** part of the PR's 
description with it.
* The data structure used by the graph (in PR description)

Then I should be able to test it out, and give some feedback to have it 
merged


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


[GitHub] zeppelin issue #1242: [ZEPPELIN-960] When there is no interpreter, paragraph...

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

https://github.com/apache/zeppelin/pull/1242
  
retry ci


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



[GitHub] zeppelin pull request #1242: [ZEPPELIN-960] When there is no interpreter, pa...

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

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


---
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 #1337: [ZEPPELIN-1333] Fix Double Run when Shift + Ent...

2016-08-17 Thread corneadoug
Github user corneadoug closed the pull request at:

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


---
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 #1325: Add an Apache Zeppelin Logo section to the website

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

https://github.com/apache/zeppelin/pull/1325
  
Merging if there is no more discussion


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


[GitHub] zeppelin issue #1336: [ZEPPELIN-1333] prevent calling runParagraph() on shif...

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

https://github.com/apache/zeppelin/pull/1336
  
Tested, I like this way better.
However, @nazgul33 next time it would be better to make a separate branch 
instead of making the PR from master.

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 #1336: [ZEPPELIN-1333] prevent calling runParagraph() on shif...

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

https://github.com/apache/zeppelin/pull/1336
  
@corneadoug Can you review this PR first? @nazgul33 and you seem to resolve 
same issue.


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


Re: [ANNOUNCE] Apache Zeppelin 0.6.1 released

2016-08-17 Thread DuyHai Doan
Congratulations to the whole team for the great job

On Tue, Aug 16, 2016 at 6:59 PM, Mohit Jaggi  wrote:

> awesome!
>
> Mohit Jaggi
> Founder,
> Data Orchard LLC
> www.dataorchardllc.com
>
>
>
>
> On Aug 16, 2016, at 6:56 AM, Ahyoung Ryu  wrote:
>
> Congratulations, the Apache Zeppelin team & all community members! [image:
> ]
>
> 
>
>> Congratulations !!!
>>
>> Great turn around with required spark 2.0 support and fix for SparkR with
>> Spark2.0.
>>
>> Regards,
>> Sourav
>>
>> On Tue, Aug 16, 2016 at 2:02 AM, Hyung Sung Shim 
>> wrote:
>>
>>> Congratulations for 0.6.1 release!
>>> Thank you for everyone especially mina.
>>>
>>> 2016-08-16 16:57 GMT+09:00 Mina Lee :
>>>
 The Apache Zeppelin community is pleased to announce the availability of
 the 0.6.1 release.

 Zeppelin is a collaborative data analytics and visualization tool for
 distributed, general-purpose data processing system such as Apache
 Spark,
 Apache Flink, etc.

 The community put significant effort into improving Apache Zeppelin
 since
 the last release, focusing on supporting Scala 2.11 and Spark 2.0.
 22 contributors provided 80+ patches for new features, improvements
 and bug fixes. More than 60+ issues have been resolved.

 We encourage download the latest release from
 http://zeppelin.apache.org/download.html

 Release note is available at
 http://zeppelin.apache.org/releases/zeppelin-release-0.6.1.html

 We welcome your help and feedback. For more information on the project
 and
 how to get involved, visit our website at http://zeppelin.apache.org/

 Thanks to all users and contributors who have helped to improve Apache
 Zeppelin.

 Regards,
 The Apache Zeppelin community

>>>
>>>
>>
>


[GitHub] zeppelin issue #989: Added ipfsnotebookrepo [ Zeppelin-683 WIP ]

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

https://github.com/apache/zeppelin/pull/989
  
@bzz there are no transitive dependencies, just Junit and hamcrest
The reason of CI failure is 

>[ERROR] Failed to execute goal on project zeppelin-zengine: Could not 
resolve dependencies for project 
org.apache.zeppelin:zeppelin-zengine:jar:0.7.0-SNAPSHOT: Failed to collect 
dependencies at org.ipfs:api:jar:0.0.1-SNAPSHOT: Failed to read artifact 
descriptor for org.ipfs:api:jar:0.0.1->SNAPSHOT: Could not transfer artifact 
org.ipfs:api:pom:0.0.1-SNAPSHOT from/to codehaus-snapshots 
(https://nexus.codehaus.org/snapshots/): nexus.codehaus.org: Name or service 
not known: Unknown host nexus.codehaus.org: Name or service not known -> [Help 
1]

It has to be added  in local .m2 repository 


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