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

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

https://github.com/apache/zeppelin/pull/1302
  
@astroshim Looks good!
I think it would be better the activated and deactivated interpreter 
buttons can be in line like below images.

 - Before (with this patch)

https://cloud.githubusercontent.com/assets/10060731/18116842/d2a64028-6f85-11e6-93d0-ea613abf12ad.png;>

 -  After 

https://cloud.githubusercontent.com/assets/10060731/18117512/3cba0ec8-6f8a-11e6-84f5-c6f1bf5ef089.png;>

Or I'd like to suggest another option. How about simply changing `Restart` 
button color to grey for the deactivated interpreter button?(usually the grey 
color stands for `deactivated` I guess)
https://cloud.githubusercontent.com/assets/10060731/18117095/6bd23a4e-6f87-11e6-94d5-d08abffb6561.png;>


What do you think? :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.
---


GSoC zeppelin-683

2016-08-30 Thread onkar shedge
Hello,

Thank you all for passing me and their support throughout my GSoC project.
Learned a lot. It was a great experience for me.
Alex , corneau please review the ipfs PR. I have addressed the feedback.

Regards,
Onkar Shegde


[GitHub] zeppelin issue #1389: [ZEPPELIN-1279] Spark on Mesos Docker.

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

https://github.com/apache/zeppelin/pull/1389
  
@AhyoungRyu Thank you for quick review. I fixed what you pointed out.


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


[GitHub] zeppelin pull request #1389: [ZEPPELIN-1279] Spark on Mesos Docker.

2016-08-30 Thread astroshim
GitHub user astroshim opened a pull request:

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

[ZEPPELIN-1279] Spark on Mesos Docker.

### What is this PR for?
This PR is for the documentation of running zeppelin on production 
environments especially spark on mesos via Docker.
Related issue is https://github.com/apache/zeppelin/pull/1227 and 
https://github.com/apache/zeppelin/pull/1318 and I got a lot of hints from 
https://github.com/sequenceiq/hadoop-docker.
Tested on ubuntu.


### What type of PR is it?
Documentation


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


### How should this be tested?
You can refer to 
https://github.com/apache/zeppelin/blob/master/docs/README.md#build-documentation.

### 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/astroshim/zeppelin ZEPPELIN-1279

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

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


commit a3b0ded8ae1010f90e78f2178525e655c6e60e91
Author: astroshim 
Date:   2016-08-18T01:59:09Z

create dockerfile for mesos

commit c80835038917d2b3c7e554fc562d9713c57f7844
Author: astroshim 
Date:   2016-08-18T08:55:49Z

add image file and doc

commit d8c43b4aae9d0040eb9d3d437261c9d3376c8ba3
Author: astroshim 
Date:   2016-08-31T03:19:42Z

add navigation

commit 159bafc51196d50b8e4fc06ba18bec99c066d361
Author: astroshim 
Date:   2016-08-31T04:12:43Z

fix anchor




---
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 #1384: [ZEPPELIN-1387] Support table syntax in markdown inter...

2016-08-30 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/1384
  
@Leemoonsoo Thanks!

I'v added missing transitive deps and removed unused markdown4j dep. 

I also found that `org.ow2.asm:asm:jar:3.0.1` is described in 
`zeppelin-distribution/src/licence_bin/LICENSE` But already denoted license is 
different version (3.0.1) compared to pegdown's dependency (5.0.3). 

Should I updated `org.ow2.asm:asm:jar:3.0.1` to `org.ow2.asm:asm:jar:5.0.3`?


---
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 #1381: [ZEPPELIN-1390] SparkInterpreter does not work for Spa...

2016-08-30 Thread tae-jun
Github user tae-jun commented on the issue:

https://github.com/apache/zeppelin/pull/1381
  
@Leemoonsoo Thanks for comment!

I made some changes.

How does it look?

Thanks 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 #1302: [ZEPPELIN-1306] Interpreter restarts on a note.

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

https://github.com/apache/zeppelin/pull/1302
  
ping


---
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 #1358: [ZEPPELIN-1365] Error of Zeppelin Application in devel...

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

https://github.com/apache/zeppelin/pull/1358
  
ping


---
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 #1241: [ZEPPELIN-1217] Remove horizontal scrollbar in Zeppeli...

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

https://github.com/apache/zeppelin/pull/1241
  
@corneadoug I'm not sure I got your point correctly,
"You can configure Apache Zeppelin with both environment variables in 
`conf/zeppelin-env.sh` (`conf\zeppelin-env.cmd` for Windows) and Java 
properties in `conf/zeppelin-site.xml`. If both are defined, then the 
environment variables will take priority."

This short sentence is written in the above of the table. So do you mean 
this sentence is not enough? Or do you want me to add more description to the 
`Description` column?


---
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 #1371: [ZEPPELIN-1372]Automatically Detect the data type in t...

2016-08-30 Thread Peilin-Yang
Github user Peilin-Yang commented on the issue:

https://github.com/apache/zeppelin/pull/1371
  
@corneadoug thanks for all your feedbacks!


---
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 #1371: [ZEPPELIN-1372]Automatically Detect the data type in t...

2016-08-30 Thread Peilin-Yang
Github user Peilin-Yang commented on the issue:

https://github.com/apache/zeppelin/pull/1371
  
BTW, for the dates detection I made a pre-defined list of supported formats 
and used the strict mode for moment.js parsing. The format "x" or "X" which 
corresponds to the UNIX timestamp was intentionally excluded from this list as 
it can match almost every number.  


---
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 #1371: [ZEPPELIN-1372]Automatically Detect the data type in t...

2016-08-30 Thread Peilin-Yang
Github user Peilin-Yang commented on the issue:

https://github.com/apache/zeppelin/pull/1371
  
@corneadoug The ideal case is to fetch the type from back end. But that is 
pretty hard. There are several types of interpreters (Sql, Spark, AngularJS, 
Shell etc..) in the back end and create a unified interface for all of them 
needs a ton of work. 
A simple example is: if user submits a SQL query and in that query the user 
applies DATE function to a string field to make that column as type date then 
basically I do not think we can correctly get the type of the column.

To me, (again) the data type in front end table is just a utility for the 
user sorts/formats the data on-the-fly. Does this make more sense?

In the latest commit I added the date cell detection. Also, I make the 
detection at the time when data was loaded to the table and this would save 
some time (each cell will be processed and loaded anyway, see function 
$scope.loadTableData). I tried to follow the code convention to make the change 
as API but the current code seems do not like doing like that so I keep my 
changes with other parts of code that deals with table 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.
---


Re: improving matplotlib integration in zeppelin

2016-08-30 Thread Goodman, Alexander (398K)
Hi all,

I have recently finished implementing an inline plotting backend (for
static images, see ZEPPELIN-1345) on my local machine as well as on a spark
cluster. I will soon make a PR, but before that I have a few questions
where I would like some insight from more experienced developers on this
project:

1) What would be the best way to package the backend with the project?
Currently it exists as a lone python source file, and using it requires
that it can be imported through the user's local PYTHONPATH. Additionally,
I think it should be packaged as a standalone product from the interpreters
as it is something that can be used by both the pyspark and python
interpreters.

2) What is the best way to implement pre/post-execute callbacks?
Specifically in this case it would be useful to implement code that the
interpreter only executes after the paragraph is finished processing its
user-entered code into the interpreter, as then it would be possible to
display images after the last executed plotting command rather than waiting
for explicit calls to show(). For example in Jupyter, plt.plot(x) will
display the plot even without calling plt.show(), that's precisely what I
am aiming for here.

Thanks,
Alex

On Tue, Aug 23, 2016 at 7:28 PM, Alexander Bezzubov  wrote:

> Right, thank you Moon for explaining both approaches in detail.
>
> I will be happy to implement AngularDisplay support for Python interpreter
> later on, to enable it to interact with AngularObjects the way
> SparkInterpreter does. I have created ZEPPELIN-1361
>  to track the
> progress.
>
> This mechanism could then be used in more involved, interactive Matplotlib
> backend implementation.
>
> --
> Alex
>
> On Sat, Aug 20, 2016, 13:23 Goodman, Alexander (398K) <
> alexander.good...@jpl.nasa.gov> wrote:
>
> > Hi Moon,
> >
> > Thank you for the informative response. You are right, this is all in
> fact
> > explicitly stated in the most recent set of matplotlib release notes[1].
> > This won't really apply to the static inline plotting backend that I will
> > be tackling first which will mostly be pure python, but it will probably
> be
> > good to keep this in mind when we begin work on the interactive plotting.
> >
> > [1] - http://matplotlib.org/users/whats_new.html#interactive-oo-usage
> >
> > Thanks,
> > Alex
> >
> > On Thu, Aug 18, 2016 at 11:13 PM, moon soo Lee  wrote:
> >
> > > Hi,
> > >
> > > It's great to see improving matplotlib integration. Thanks a lot.
> > >
> > > In my understanding, in interactive mode, the graph supposed to be
> > updated
> > > even if some matplotlib methods are called in the other
> paragraph(cell).
> > > That means the result of a paragraph need to be updated by running
> > another
> > > paragraph.
> > >
> > > Currently, i think there're two different facilities in Zeppelin to do
> > > that.
> > >
> > > One possible way is using InterpreterContextRunner [1].
> > InterpreterContext
> > > provides InterpreterContextRunner[2] and it gives ability to run other
> > > paragraphs in the same note. However this approach does have some
> > > limitations. Like if paragraph (cell) is in the other notebook,
> > interactive
> > > update of graph will not work anymore. And because it's not only update
> > the
> > > result of the other paragraph, but also run the other paragraph, it'll
> be
> > > difficult to make interactive mode work correctly depends on user code
> in
> > > each paragraphs.
> > >
> > > Second possible approach is using AngularDisplay system.
> > > Which allows interpreter send/receive some data and event from/to
> > front-end
> > > side.
> > > So without rerun another paragraph, it's possible to update result of a
> > > paragraph from another.
> > > Any interpreter can get AngularObjectRegistry[3] from
> InterpreterContext
> > > [4], and AngularObjectRegistry allows create object / add event hook to
> > > communicate with front-end. I think this is more feasible approach.
> > >
> > > Thanks,
> > > moon
> > >
> > > [1]
> > > https://github.com/apache/zeppelin/blob/master/zeppelin-
> > > interpreter/src/main/java/org/apache/zeppelin/interpreter/
> > > InterpreterContextRunner.java
> > > [2]
> > > https://github.com/apache/zeppelin/blob/master/zeppelin-
> > > interpreter/src/main/java/org/apache/zeppelin/interpreter/
> > > InterpreterContext.java#L123
> > > [3]
> > > https://github.com/apache/zeppelin/blob/master/zeppelin-
> > > interpreter/src/main/java/org/apache/zeppelin/display/
> > > AngularObjectRegistry.java
> > > [4]
> > > https://github.com/apache/zeppelin/blob/master/zeppelin-
> > > interpreter/src/main/java/org/apache/zeppelin/interpreter/
> > > InterpreterContext.java#L115
> > >
> > >
> > > On Thu, Aug 18, 2016 at 11:57 AM Goodman, Alexander (398K) <
> > > alexander.good...@jpl.nasa.gov> wrote:
> > >
> > > > Hi all,
> > > >
> > > > As per previous discussion I had with Alex Bezzubov on the users
> > mailing
> > 

[GitHub] zeppelin issue #1387: [ZEPPELIN-1398] Use relative path for search_data.json

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

https://github.com/apache/zeppelin/pull/1387
  
Tested and 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 #1387: [ZEPPELIN-1398] Use relative path for search_data.json

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

https://github.com/apache/zeppelin/pull/1387
  
Thanks for checking @Leemoonsoo , I have followed your steps and provided 
an updated fix.
Here are the results on my local env.

https://cloud.githubusercontent.com/assets/382917/18104477/1e74a43a-6eb0-11e6-8244-7b73ff26509f.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 #1387: [ZEPPELIN-1398] Use relative path for search_data.json

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

https://github.com/apache/zeppelin/pull/1387
  
Thanks @lresende for contribution.
I have tested this patch, but i still get the same error.

How to test

1) build `gh-pages` (website) branch

```
bundle exec jekyll build --safe
mv _site /tmp/zeppelin_website
```

2) build this patch (docs) and copy it under docs/0.7.0-SNAPSHOT of website

```
cd docs
bundle exec jekyll build --safe
mkdir /tmp/zeppelin-website/docs
mv _site /tmp/zeppelin_website/0.7.0-SNAPSHOT
```

3) start httpserver 

```
cd /tmp/zeppelin_website
python -m SimpleHTTPServer
```

And browse localhost:8000, select 0.7.0-SNAPSHOT doc and try search



---
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 #1388: [ZEPPELIN-1399] Refactor automatic interpreter name in...

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

https://github.com/apache/zeppelin/pull/1388
  
Tested and LGTM.
Thanks @minahlee for the great improvement.


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

https://github.com/apache/zeppelin/pull/1334
  
@bzz  kindly be noted for point # 2:
Making user do minimal amount of code is good but the only cons is that the 
user will be given the default implementations and configurations and can not 
change it. This constraint is because we are using a very basic java REPL 
implementation. If this is fine, we can proceed and implement it!

What do you think about that?


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


[GitHub] zeppelin issue #1387: [ZEPPELIN-1398] Use relative path for search_data.json

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

https://github.com/apache/zeppelin/pull/1387
  
@lresende Wow super fast! That was my bad. I should've thought about the 
relative path. Thank you for the quick fix and 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 #1388: [ZEPPELIN-1399] Refactor automatic interpreter ...

2016-08-30 Thread minahlee
GitHub user minahlee opened a pull request:

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

[ZEPPELIN-1399] Refactor automatic interpreter name insert

### What is this PR for?
Refine automatic interpreter name insert. See below jira issue for detailed 
description

### What type of PR is it?
Improvement | Refactoring

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

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

### Screenshots (if appropriate)
- Do not insert interpreter name if previous interpreter name is not 
specified
**Before**
![aug-30-2016 
17-28-06](https://cloud.githubusercontent.com/assets/8503346/18095394/1f25c9b4-6ed7-11e6-818d-71099a96e305.gif)
**After**
![aug-30-2016 
17-24-23](https://cloud.githubusercontent.com/assets/8503346/18095264/b469cfc6-6ed6-11e6-8497-320e8f3f8f46.gif)


- Insert interpreter name of previous paragraph without running
**Before**
![aug-30-2016 
17-33-25](https://cloud.githubusercontent.com/assets/8503346/18095643/dfbc1eee-6ed7-11e6-8249-05c4d046da9d.gif)
**After**
![aug-30-2016 
17-24-32](https://cloud.githubusercontent.com/assets/8503346/18095249/a4936e90-6ed6-11e6-9223-6405f3e36f80.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/minahlee/zeppelin ZEPPELIN-1399

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

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


commit a67df081f9cc8178365c08a7eefe55f0850e59e0
Author: Mina Lee 
Date:   2016-08-23T13:40:26Z

Refactor automatic magic keyword insert

commit cb7e83c70f341b3bfec853583bbb2faf2c501030
Author: Mina Lee 
Date:   2016-08-24T07:36:28Z

modify 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 pull request #1387: [ZEPPELIN-1398] Use relative path for search_da...

2016-08-30 Thread lresende
GitHub user lresende opened a pull request:

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

[ZEPPELIN-1398] Use relative path for search_data.json

### What is this PR for?
Use relative path for search_data.json


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

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


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

$ git pull https://github.com/lresende/incubator-zeppelin search

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

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


commit 0b63cdc1e609494c80e7d6eb3e7c37fd66135ae6
Author: Luciano Resende 
Date:   2016-08-30T15:09:29Z

[ZEPPELIN-1398] Use relative path for search_data.json




---
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-1399) Refactor automatic interpreter name insert

2016-08-30 Thread Mina Lee (JIRA)
Mina Lee created ZEPPELIN-1399:
--

 Summary: Refactor automatic interpreter name insert
 Key: ZEPPELIN-1399
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1399
 Project: Zeppelin
  Issue Type: Improvement
Affects Versions: 0.7.0
Reporter: Mina Lee
Assignee: Mina Lee
 Fix For: 0.7.0


`lastReplName` is saved in note.json after paragraph running.
`%lastReplName` is added to any new paragraph automatically.

Some behavior feels unnatural for me with current implementation.

1.
Imagine your default interpreter name is `intp1`. If you type `%intp2` in first 
paragraph and add new paragraph in second place without running first 
paragraph, text of the paragraph would be `%intp1` not `%intp2`.

2.
It always insert %lastReplName text automatically even if you don't specify 
interpreter magic keyword in previous paragraph.

I would like to suggest not to save `lastReplName` info in note.json but just 
read text of previous paragraph and add %interpreterName to new paragraph if 
previous paragraph has one.



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


[GitHub] zeppelin pull request #1383: Merge pull request #1 from apache/master

2016-08-30 Thread demel-orange
Github user demel-orange closed the pull request at:

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


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


ApacheCon Seville CFP closes September 9th

2016-08-30 Thread Rich Bowen
It's traditional. We wait for the last minute to get our talk proposals
in for conferences.

Well, the last minute has arrived. The CFP for ApacheCon Seville closes
on September 9th, which is less than 2 weeks away. It's time to get your
talks in, so that we can make this the best ApacheCon yet.

It's also time to discuss with your developer and user community whether
there's a track of talks that you might want to propose, so that you
have more complete coverage of your project than a talk or two.

For Apache Big Data, the relevant URLs are:
Event details:
http://events.linuxfoundation.org/events/apache-big-data-europe
CFP:
http://events.linuxfoundation.org/events/apache-big-data-europe/program/cfp

For ApacheCon Europe, the relevant URLs are:
Event details: http://events.linuxfoundation.org/events/apachecon-europe
CFP: http://events.linuxfoundation.org/events/apachecon-europe/program/cfp

This year, we'll be reviewing papers "blind" - that is, looking at the
abstracts without knowing who the speaker is. This has been shown to
eliminate the "me and my buddies" nature of many tech conferences,
producing more diversity, and more new speakers. So make sure your
abstracts clearly explain what you'll be talking about.

For further updated about ApacheCon, follow us on Twitter, @ApacheCon,
or drop by our IRC channel, #apachecon on the Freenode IRC network.

-- 
Rich Bowen
WWW: http://apachecon.com/
Twitter: @ApacheCon


[GitHub] zeppelin issue #1383: Merge pull request #1 from apache/master

2016-08-30 Thread demel
Github user demel commented on the issue:

https://github.com/apache/zeppelin/pull/1383
  
My apologies. I had accidentally clicked the PR button, but thought I 
cancelled the request. 

Will do. Thanks for your help.


---
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-1398) Document search feature doesn't work

2016-08-30 Thread Ahyoung Ryu (JIRA)
Ahyoung Ryu created ZEPPELIN-1398:
-

 Summary: Document search feature doesn't work
 Key: ZEPPELIN-1398
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1398
 Project: Zeppelin
  Issue Type: Bug
Reporter: Ahyoung Ryu
Assignee: Ahyoung Ryu
 Fix For: 0.6.2


Currently 
[http://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/search.html|http://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/search.html]
 doesn't work because of the wrong path of {{search_data.json}}. So it needs to 
be fixed.



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


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

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

https://github.com/apache/zeppelin/pull/1334
  
Few questions after another pass:

 1. Right now Beam interpreter uses **Beam version 0.1.0-incubating**, but 
didn't Beam project release 0.2.0-incubating on 2016-08-09? Do you think it 
would be useful to update it right here, before the merge or do you plan to do 
that in a separate PR later on?

 2. AFAIK there are multiple "runners" for Beam, and right only be reading 
the doc you can see how to switch them. Do you think it would fit Zeppelin use 
case better if there are 3 different interpreters `%beam.flink`, `%beam.spark`, 
etc each with it's own appropriate configuration options and a user could just 
skip creating a class and defining `main() {}` in every paragraph?

 3. Here is the list of dependencies from `mvn dependency:tree`. All of 
them will become part of the convenience binary for the next Zeppelin release. 
What is needed to be done, in order for us to merge it is: 
  - make sure that ALL these dependencies 
  - and ALL their transitive dependencies 
  - have an appropriate licence, and are logged under the 
`./zeppelin-distribution/src/bin_license/LICENSE`
  - and for non-Apache license, full text of the license need to be added 
to ./zeppelin-distribution/src/license/`

  For your convenience I have listed all current dependencies of `beam` 
module below, please feel free to verify and mark those that satisfy the above 
criterias.

  ```
 - [ ] org.apache.zeppelin:zeppelin-beam:jar:0.7.0-SNAPSHOT
 - [ ] +- org.apache.spark:spark-core_2.10:jar:1.4.1:compile
 - [ ] |  +- com.twitter:chill_2.10:jar:0.5.0:compile
 - [ ] |  |  \- com.esotericsoftware.kryo:kryo:jar:2.21:compile
 - [ ] |  | +- 
com.esotericsoftware.reflectasm:reflectasm:jar:shaded:1.07:compile
 - [ ] |  | +- com.esotericsoftware.minlog:minlog:jar:1.2:compile
 - [ ] |  | \- org.objenesis:objenesis:jar:1.2:compile
 - [ ] |  +- com.twitter:chill-java:jar:0.5.0:compile
 - [ ] |  +- org.apache.spark:spark-launcher_2.10:jar:1.4.1:compile
 - [ ] |  +- org.apache.spark:spark-network-common_2.10:jar:1.4.1:compile
 - [ ] |  +- org.apache.spark:spark-network-shuffle_2.10:jar:1.4.1:compile
 - [ ] |  +- org.apache.spark:spark-unsafe_2.10:jar:1.4.1:compile
 - [ ] |  +- net.java.dev.jets3t:jets3t:jar:0.7.1:compile
 - [ ] |  +- org.apache.curator:curator-recipes:jar:2.4.0:compile
 - [ ] |  |  \- org.apache.curator:curator-framework:jar:2.4.0:compile
 - [ ] |  | \- org.apache.curator:curator-client:jar:2.4.0:compile
 - [ ] |  +- 
org.eclipse.jetty.orbit:javax.servlet:jar:3.0.0.v201112011016:compile
 - [ ] |  +- org.apache.commons:commons-lang3:jar:3.3.2:compile
 - [ ] |  +- org.apache.commons:commons-math3:jar:3.4.1:compile
 - [ ] |  +- com.google.code.findbugs:jsr305:jar:1.3.9:compile
 - [ ] |  +- org.slf4j:slf4j-api:jar:1.7.10:compile
 - [ ] |  +- org.slf4j:jul-to-slf4j:jar:1.7.10:compile
 - [ ] |  +- org.slf4j:jcl-over-slf4j:jar:1.7.10:compile
 - [ ] |  +- log4j:log4j:jar:1.2.17:compile
 - [ ] |  +- com.ning:compress-lzf:jar:1.0.3:compile
 - [ ] |  +- org.xerial.snappy:snappy-java:jar:1.1.1.7:compile
 - [ ] |  +- net.jpountz.lz4:lz4:jar:1.2.0:compile
 - [ ] |  +- org.roaringbitmap:RoaringBitmap:jar:0.4.5:compile
 - [ ] |  +- commons-net:commons-net:jar:2.2:compile
 - [ ] |  +- org.scala-lang:scala-library:jar:2.10.4:compile
 - [ ] |  +- org.json4s:json4s-jackson_2.10:jar:3.2.10:compile
 - [ ] |  |  \- org.json4s:json4s-core_2.10:jar:3.2.10:compile
 - [ ] |  | +- org.json4s:json4s-ast_2.10:jar:3.2.10:compile
 - [ ] |  | \- org.scala-lang:scalap:jar:2.10.0:compile
 - [ ] |  |\- org.scala-lang:scala-compiler:jar:2.10.0:compile
 - [ ] |  +- com.sun.jersey:jersey-server:jar:1.9:compile
 - [ ] |  |  \- asm:asm:jar:3.1:compile
 - [ ] |  +- com.sun.jersey:jersey-core:jar:1.9:compile
 - [ ] |  +- org.apache.mesos:mesos:jar:shaded-protobuf:0.21.1:compile
 - [ ] |  +- com.clearspring.analytics:stream:jar:2.7.0:compile
 - [ ] |  +- io.dropwizard.metrics:metrics-core:jar:3.1.0:compile
 - [ ] |  +- io.dropwizard.metrics:metrics-jvm:jar:3.1.0:compile
 - [ ] |  +- io.dropwizard.metrics:metrics-json:jar:3.1.0:compile
 - [ ] |  +- io.dropwizard.metrics:metrics-graphite:jar:3.1.0:compile
 - [ ] |  +- com.fasterxml.jackson.core:jackson-databind:jar:2.4.4:compile
 - [ ] |  +- 
com.fasterxml.jackson.module:jackson-module-scala_2.10:jar:2.4.4:compile
 - [ ] |  |  +- org.scala-lang:scala-reflect:jar:2.10.4:compile
 - [ ] |  |  \- com.thoughtworks.paranamer:paranamer:jar:2.6:compile
 - [ ] |  +- org.apache.ivy:ivy:jar:2.4.0:compile
 - [ ] |  +- oro:oro:jar:2.0.8:compile
 - [ ] |  +- org.tachyonproject:tachyon-client:jar:0.6.4:compile
 - [ ] |  |  \- 

[GitHub] zeppelin issue #1383: Merge pull request #1 from apache/master

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

https://github.com/apache/zeppelin/pull/1383
  
@demel-orange are you sure that this PR was made to the right 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.
---


[GitHub] zeppelin issue #1384: [ZEPPELIN-1387] Support table syntax in markdown inter...

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

https://github.com/apache/zeppelin/pull/1384
  
The change of library looks great to me, 👍  for having a test case!

Few things to take care of, before merging it:
 - for all the code, please make sure it follows project [styleguide\code 
conventions](https://github.com/apache/zeppelin/blob/master/CONTRIBUTING.md#code-convention)
 - as you add a new dependency - it will become a part of the next release 
convenience binary for Zeppelin and in order to be included, we need to verify 
it's licence compatibility. Can you please add it to 
`zeppelin-distribution/src/licence_bin/LICENSE`
 - also, applying some styles for such tables, as noted by @AhyoungRyu 
sounds like a good improvement 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 #1368: zeppelin-display dependency version built only for Sca...

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

https://github.com/apache/zeppelin/pull/1368
  
@lresende You are totally right - I got a bit mislead by this one 
https://github.com/apache/zeppelin/commit/6bb4b5ba8e7ecd6274dd437c6ec5c4a9b178e4d1#diff-d3b6c1f6610ca8471663da11e6225408L78
 which fixes server's dependency version to 2.10 one, but that's made from 
_zeppelin-distribution_ subproject, so this dependency has in fact different 
semantics that between _normal_ artifacts, right?


---
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-30 Thread WeichenXu123
GitHub user WeichenXu123 reopened 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 841b757690d41ef3bb162d8089cc0bad16333602
Author: WeichenXu 
Date:   2016-08-27T13:51:30Z

update.

commit 204a34c01ff8df61cc27833fc96cee6fb3ba81f1
Author: WeichenXu 
Date:   2016-08-28T07:44:26Z

improve code stype.




---
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 #1304: [ZEPPELIN-1190] [WIP] Visit Notebook Revision

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

https://github.com/apache/zeppelin/pull/1304
  
@corneadoug thanks for letting me know, i'll look into that and get back to 
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 #1241: [ZEPPELIN-1217] Remove horizontal scrollbar in Zeppeli...

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

https://github.com/apache/zeppelin/pull/1241
  
LGTM, could be good to try changing that Table by something more 
descriptive.
Although the fact that there is 2 different configuration files with 
possible exclusive configurations doesn't help.


---
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 #1304: [ZEPPELIN-1190] [WIP] Visit Notebook Revision

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

https://github.com/apache/zeppelin/pull/1304
  
@khalidhuseynov 
I found out that websocket events at Notebook level are already blocked in 
the revision (I don't receive anything, and my actions are not affecting the 
original note).

However there is some errors on Zeppelin server:
https://cloud.githubusercontent.com/assets/710411/18079516/2ddd7944-6ecc-11e6-9be3-632dc0a1a704.png;>

To reproduce, you can visit a revision form the dropdown. Then try running 
a paragraph (for example), this happen for all the Notebook websocket event 
(Run, Save etc...)

The backend seems to be using a NotebookSocket param that isn't initiated 
in our case (which is good). However since the function are not checking the 
parameter, it generates a NPE


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