date:20160823

Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/1354
  
@bzz I added a section with the Selenium CLI command, can you double check 
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 #1354: [ZEPPELIN-1346] Add contribution guidelines to the web...

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1354
  
@corneadoug Thank you for the explanation. Since it's a only one page, i've 
created a PR at your repo :) 
(Just combined `docs/README.md` & `docs/CONTRIBUTING.md` contents)


---
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 #989: Added ipfsnotebookrepo [ Zeppelin-683 ]

Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/989
  
Looks great, @onkarshedge! I posted a few comments above.

\cc @khalidhuseynov for extra back-end code review and @corneadoug for 
front-end one


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

[GitHub] zeppelin issue #1354: [ZEPPELIN-1346] Add contribution guidelines to the web...

Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/1354
  
@AhyoungRyu That's the goal of having the side menu, to be able to have 
contribution guide focused on specific components (Back-end, front-end, 
interpreters, website, docs etc..)

I only wanted to cover the structure in this PR, that's why there is web 
application as example.
So you can either do a PR after this one is merged, or directly in this PR, 
your choice!


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

Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1283
  
@beriaanirudh thank you for prompt fixes!


---
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 #1354: [ZEPPELIN-1346] Add contribution guidelines to the web...

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1354
  
@corneadoug  Great work! Definitely this way would be better than we put 
`CONTRIBUTING.md` under each component dir in Zeppelin github. I also left some 
minor comments. 
One question. If this PR can cover the other contributing guide(not only 
`web application`), can we put `docs/CONTRIBUTING.md` as well? What do you 
think? I can create a PR to your repo for that if you're positive :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 #1242: [ZEPPELIN-960] When there is no interpreter, paragraph...

Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1242
  
Thank you @cloverhearts !
Looks good to me, merging if there is no further discussion


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

Re: improving matplotlib integration in zeppelin

2016-08-23 Thread Alexander Bezzubov

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
> > > list, I have created two new JIRA issues ([1] and [2]) explaining in
> more
> > > detail what I think we should ultimately strive for in our ongoing work
> > to
> > > improve matplotlib integration in zeppelin. For now I think I will be
> > able
> > > to handle the bulk of the work for the static images backend issue
> > > [ZEPPELIN-1345] on my own, but more collaboration will be needed to get
> > > interactive plotting to work.  Please feel free to discuss any thoughts
> > or
> > > suggestions you may have here.
> > >
> > > [1] - https://issues.apache.org/jira/browse/ZEPPELIN-1344
> > > [2] - https://issues.apache.org/jira/browse/ZEPPELIN-1345
> > >
> > > Thanks,
> > > Alex
> > > --
> > > Alex Goodman
> > > Data Scientist I
> > > Science Data Modeling and Computing (398K)
> > > Jet Propulsion Laboratory
> > > California Institute of Technology
> > > Tel: +1-818-354-6012
> > >
> >
>
>
>
> --
> Alex Goodman
> Data Scientist I
> Science Data Modeling and Computing (398K)
> Jet Propulsion Laboratory
> California Institute of Technology
> Tel: +1-818-354-6012
>

[jira] [Created] (ZEPPELIN-1361) Python interpreter - add support for AngularDisplay system

2016-08-23 Thread Alexander Bezzubov (JIRA)

Alexander Bezzubov created ZEPPELIN-1361:


 Summary: Python interpreter - add support for AngularDisplay system
 Key: ZEPPELIN-1361
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1361
 Project: Zeppelin
  Issue Type: Improvement
  Components: python-interpreter
Reporter: Alexander Bezzubov
 Fix For: 0.7.0


As discussed on mailing list dev@z.a.o subj "improving matplotlib integration 
in zeppelin" - in order to complete ZEPPELIN-1344 and give ability for one 
Python paragraph to update content in another one - we can use AngularDisplay 
system, same as in Apache Spark interpreter.


{code}
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. 

[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
{code}




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

[GitHub] zeppelin issue #1352: [ZEPPELIN-1327] Fix bug in z.show for Python interpret...

Github user bzz commented on the issue:

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


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

[GitHub] zeppelin issue #1354: [ZEPPELIN-1346] Add contribution guidelines to the web...

Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1354
  
Overall looks great to me, thank you @corneadoug for taking care! Few minor 
comments above.
\cc @AhyoungRyu for review


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

[GitHub] zeppelin pull request #1355: [ZEPPELIN-1359] Commit correctly ordered karma....

2016-08-23 Thread asfgit

Github user asfgit closed the pull request at:

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


---
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 #1355: [ZEPPELIN-1359] Commit correctly ordered karma.conf fi...

Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/1355
  
Merging it fast since it doesn't bring any real code change


---
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 #1351: [ZEPPELIN/1356] The graph legend truncates at t...

GitHub user Peilin-Yang reopened a pull request:

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

[ZEPPELIN/1356] The graph legend truncates at the nearest period (.) in its 
grouping

### What is this PR for?
Fix the issue: in line graph if user uses the numbers that contains 
period(.), e.g. 3.14 in the groups the legend will only show 3 instead of 3.14.


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

### Todos

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

### How should this be tested?
Localhost test with screenshot

### Screenshots (if appropriate)

https://drive.google.com/a/twitter.com/file/d/0B9QN74Rceec8NG10UEt0R3VUWTg/view

### 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/Peilin-Yang/zeppelin ypeilin/ZEPPELIN-1356

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

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


commit 00527a5a0c35c0b292348a6b90110de22d8106c5
Author: Peilin Yang 
Date:   2016-08-22T19:40:24Z

fix ZEPPELIN/1356




---
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 #1351: [ZEPPELIN/1356] The graph legend truncates at t...

Github user Peilin-Yang closed the pull request at:

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


---
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 #1351: [ZEPPELIN/1356] The graph legend truncates at t...

GitHub user Peilin-Yang reopened a pull request:

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

[ZEPPELIN/1356] The graph legend truncates at the nearest period (.) in its 
grouping

### What is this PR for?
Fix the issue: in line graph if user uses the numbers that contains 
period(.), e.g. 3.14 in the groups the legend will only show 3 instead of 3.14.


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

### Todos

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

### How should this be tested?
Localhost test with screenshot

### Screenshots (if appropriate)

https://drive.google.com/a/twitter.com/file/d/0B9QN74Rceec8NG10UEt0R3VUWTg/view

### 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/Peilin-Yang/zeppelin ypeilin/ZEPPELIN-1356

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

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


commit 00527a5a0c35c0b292348a6b90110de22d8106c5
Author: Peilin Yang 
Date:   2016-08-22T19:40:24Z

fix ZEPPELIN/1356




---
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 #1351: [ZEPPELIN/1356] The graph legend truncates at t...

Github user Peilin-Yang closed the pull request at:

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


---
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 #1351: [ZEPPELIN/1356] The graph legend truncates at t...

GitHub user Peilin-Yang reopened a pull request:

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

[ZEPPELIN/1356] The graph legend truncates at the nearest period (.) in its 
grouping

### What is this PR for?
Fix the issue: in line graph if user uses the numbers that contains 
period(.), e.g. 3.14 in the groups the legend will only show 3 instead of 3.14.


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

### Todos

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

### How should this be tested?
Localhost test with screenshot

### Screenshots (if appropriate)

https://drive.google.com/a/twitter.com/file/d/0B9QN74Rceec8NG10UEt0R3VUWTg/view

### 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/Peilin-Yang/zeppelin ypeilin/ZEPPELIN-1356

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

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


commit 00527a5a0c35c0b292348a6b90110de22d8106c5
Author: Peilin Yang 
Date:   2016-08-22T19:40:24Z

fix ZEPPELIN/1356




---
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 #1351: [ZEPPELIN/1356] The graph legend truncates at t...

Github user Peilin-Yang closed the pull request at:

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


---
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 #1351: [ZEPPELIN/1356] The graph legend truncates at t...

GitHub user Peilin-Yang reopened a pull request:

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

[ZEPPELIN/1356] The graph legend truncates at the nearest period (.) in its 
grouping

### What is this PR for?
Fix the issue: in line graph if user uses the numbers that contains 
period(.), e.g. 3.14 in the groups the legend will only show 3 instead of 3.14.


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

### Todos

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

### How should this be tested?
Localhost test with screenshot

### Screenshots (if appropriate)

https://drive.google.com/a/twitter.com/file/d/0B9QN74Rceec8NG10UEt0R3VUWTg/view

### 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/Peilin-Yang/zeppelin ypeilin/ZEPPELIN-1356

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

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


commit 00527a5a0c35c0b292348a6b90110de22d8106c5
Author: Peilin Yang 
Date:   2016-08-22T19:40:24Z

fix ZEPPELIN/1356




---
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 #1351: [ZEPPELIN/1356] The graph legend truncates at t...

Github user Peilin-Yang closed the pull request at:

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


---
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 #1352: [ZEPPELIN-1327] Fix bug in z.show for Python interpret...

2016-08-23 Thread agoodm

Github user agoodm commented on the issue:

https://github.com/apache/zeppelin/pull/1352
  
@bustios Nice job, this works great now. ð 


---
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 #1352: [ZEPPELIN-1327] Fix bug in z.show for Python interpret...

2016-08-23 Thread agoodm

Github user agoodm commented on the issue:

https://github.com/apache/zeppelin/pull/1352
  
@bustios 
Nice job on removing the need to perform the python version check 
completely. This certainly helps make the code look cleaner.

However I cannot get your changes to work correctly with PNG images. 
Instead of resizing the image itself, it resizes the background containing the 
image:

![ex](https://puu.sh/qLGFi/6f22bd252d.png)

If I use width instead of height in this example, the image actually gets 
resized:

![ex](https://puu.sh/qLH80/e7d05ac8a0.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 #1323: [HOTFIX] Add logins authentication for hdfs file inter...

2016-08-23 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/1323
  
I have a question.
Webhdfs that will enable authentication through any set of Hadoop?
I've tested this pr.
but, this pr and not merged master branch both normalize. working well.

Please share your authentication settings.
It would be helpful to me to test.
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 #1352: [ZEPPELIN-1327] Fix bug in z.show for Python interpret...

2016-08-23 Thread bustios

Github user bustios commented on the issue:

https://github.com/apache/zeppelin/pull/1352
  
Done. I added a comment and put the decoding statement in a separate line 
for a better understanding.


---
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-23 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/1242
  
@bzz Thank you.
I had remove to try catch.
All requests have been completed.
Have a Nice Day--!


---
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 #1355: [ZEPPELIN-1359] Commit correctly ordered karma.conf fi...

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1355
  
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 #89: enabled download data as csv button

2016-08-23 Thread ZhijieWang

Github user ZhijieWang commented on the issue:

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

@qinzl1 , I don't think the limitation of how many rows to download is
specified by my download button. The download button only converts existing
JSON response retrieved from Zeppelin server and converted in to CSV format
with UI button. You should look at which specific interpreter you are using.
Most example set the max row to display for dataframe or result for 1000.
For example, [file]
(https://github.com/apache/zeppelin/blob/5afd7cacdb6ac35d19ebf63bac9f2d20298cfc5c/python/src/main/resources/interpreter-setting.json)
```
[
{
"group": "python",
"name": "python",
"className": "org.apache.zeppelin.python.PythonInterpreter",
"properties": {
"zeppelin.python": {
"envName": null,
"propertyName": "zeppelin.python",
"defaultValue": "python",
"description": "Python directory. It is set to python by
default.(assume python is in your $PATH)"
},
"zeppelin.python.maxResult": {
"envName": null,
"propertyName": "zeppelin.python.maxResult",
"defaultValue": "1000",
"description": "Max number of dataframe rows to display."
}
}
},
{
"group": "python",
"name": "sql",
"className": "org.apache.zeppelin.python.PythonInterpreterPandasSql",
"properties": { }
}
]```

However, be aware of how many rows you are requesting. The more rows you
request, the more memory it uses to build the JSON response and the longer it
takes for the browser to do the file format conversion.

I am not entirely sure how optimized is the JSON parsing and encoding
library used by Zeppelin or what library it is. Based on what I have seen, it
seems the default library and direct toString() conversion.

---
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 #1345: [ZEPPELIN-728] Can't POST interpreter setting (CorsFil...

2016-08-23 Thread kavinkumarks

Github user kavinkumarks commented on the issue:

https://github.com/apache/zeppelin/pull/1345
  
Updated the docs. @jongyoul could you please merge if everything is okay?


---
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 #1348: [ZEPPELIN-1313] NullPointerException when using Clone ...

Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1348
  
@kavinkumarks Thanks for the removing the method, but personally, I think 
we should handle it with another PR. @felixcheung What do you think of it? It 
looks irrelevant. 


---
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 #1353: ZEPPELIN-1326: make profile to select dependency of ha...

Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1353
  
I think your changed is that we already discussed before. `provided` is 
mandatory, and check the method those SparkInterpreter does. How about it? And 
some changes are going with another 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 #1340: [MINOR][DOC] Update available interpreters' image in i...

2016-08-23 Thread minahlee

Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1340
  
LGTM merge 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 #1265: [ZEPPELIN-1210] Run interpreter per user

GitHub user jongyoul reopened a pull request:

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

[ZEPPELIN-1210] Run interpreter per user

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

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

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

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

### Screenshots (if appropriate)

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


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

$ git pull https://github.com/jongyoul/zeppelin ZEPPELIN-1210

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

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

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1265


commit 01afc809e7f17c8991a78649d54852bdef8c318e
Author: Jongyoul Lee 
Date:   2016-07-21T15:34:25Z

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

commit 8b8074ad24be05d6ad75bd7f9f3387bedd7ca1dd
Author: Jongyoul Lee 
Date:   2016-07-22T10:29:07Z

WIP

commit 9edf4931e79a0d3c75e623ae82e3cf2002dc2003
Author: Jongyoul Lee 
Date:   2016-07-28T13:21:23Z

WIP

commit 5651259e9394aa0b9553d208bdf083418bff972f
Author: Jongyoul Lee 
Date:   2016-08-02T15:10:18Z

Added option in UI

commit 44e279b034ee23e23733ec21c882693b3b017d8b
Author: Jongyoul Lee 
Date:   2016-08-02T15:24:03Z

Reverted some value to default ones

commit 78bc2caa564cd4088f1e24fbaa6f6aec4d43a487
Author: Jongyoul Lee 
Date:   2016-08-08T15:09:14Z

Fixed some mismatch after rebase

commit 81874c651f4500519a2e8c0ae2e7be90e3605a9e
Author: Jongyoul Lee 
Date:   2016-08-09T03:50:50Z

Fixed tests to use AuthenticationInfo

commit bd29b1cb10ec97886dc50a56c7798a5482e4b2b7
Author: Jongyoul Lee 
Date:   2016-08-11T06:58:02Z

Fixed some tests

commit b53b05571326c1037a27e52e4bc785189e16a2a1
Author: Jongyoul Lee 
Date:   2016-08-11T07:28:35Z

Fixed some tests

commit f77edc9f723a7af19e8fa6c90c73b50dfc905e3c
Author: Jongyoul Lee 
Date:   2016-08-16T11:54:45Z

Fixed some codes after rebase




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

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

Github user jongyoul closed the pull request at:

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


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

Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1315
  
@rconline I know what you focus on and try to solve the problem. I, 
actually, cannot accept whether your solution is perfect or not, but I agree 
the problem you told. Thanks for the explaining it. your codes looks good to me.


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

[GitHub] zeppelin issue #1283: Buffer append output results + fix extra incorrect res...

2016-08-23 Thread beriaanirudh

Github user beriaanirudh commented on the issue:

https://github.com/apache/zeppelin/pull/1283
  
@jongyoul done. Used diamond operator at all places except for 1 where 
compiler wouldn't allow (AppendOutputRunner.java:78)


---
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 #1355: [ZEPPELIN-1359] Commit correctly ordered karma....

GitHub user corneadoug opened a pull request:

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

[ZEPPELIN-1359] Commit correctly ordered karma.conf file

### What is this PR for?
In the karma-conf.js file, dependencies are usually alphabetically ordered.
On a clean repo build, the file is changed to be alphabetically order those 
dependencies.
SInce one of those is not in the right order, it creates a modified file in 
the repository after build.

This PR is simply re-ordering the file so that there is no modifications on 
build.


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

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

### How should this be tested?
Build the source code from a clean repo, there shouldn't be any file 
changed if you type `git status`

### 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/corneadoug/incubator-zeppelin ZEPPELIN-1359

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

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


commit 6d9a8da1e4575eef33f0d6ee6a2fe61ad3d67e33
Author: Damien CORNEAU 
Date:   2016-08-23T07:27:58Z

Add correct karma.conf.js




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

[jira] [Created] (ZEPPELIN-1359) Re-order Zeppelin-web karma.conf

2016-08-23 Thread CORNEAU Damien (JIRA)

CORNEAU Damien created ZEPPELIN-1359:


 Summary: Re-order Zeppelin-web karma.conf
 Key: ZEPPELIN-1359
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1359
 Project: Zeppelin
  Issue Type: Bug
Reporter: CORNEAU Damien
Assignee: CORNEAU Damien
Priority: Minor


If you do a build from a clean repository, karma-conf will be modified to 
re-shuffled alphabetically (like it is supposed to be) the order of the 
dependencies.

Since it isn't a code modification we want after build, we should change the 
un-ordered dependency and commit it to the source code.



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

[GitHub] zeppelin pull request #1354: [ZEPPELIN-1346] Add contribution guidelines to ...

GitHub user corneadoug opened a pull request:

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

[ZEPPELIN-1346] Add contribution guidelines to the website

### What is this PR for?
This PR is bringing Contribution guidelines to the website.
We also introduce a sidemenu on those pages allowing to create additional 
content.
As an example, for the Web Application Guidelines, we added a page about 
"Defining Components"

In a different PR, we will remove the contributing markdown, and change the 
links in the repository to the website.


### What type of PR is it?
Documentation

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

### How should this be tested?
Build the website locally and visit the community page

### Screenshots (if appropriate)
![screen shot 2016-08-23 at 3 50 41 
pm](https://cloud.githubusercontent.com/assets/710411/17882774/8da96b86-6949-11e6-8789-fed606b64fd8.png)

![screen shot 2016-08-23 at 3 50 51 
pm](https://cloud.githubusercontent.com/assets/710411/17882782/99cbcc60-6949-11e6-84d4-937045c9cf55.png)

![screen shot 2016-08-23 at 3 50 33 
pm](https://cloud.githubusercontent.com/assets/710411/17882785/9c0315f6-6949-11e6-963b-3e26190bc72a.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/corneadoug/incubator-zeppelin 
add/ContributeCategory

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

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


commit 06238fbc9b3a65085db2ea1ad4aa53630b1660f3
Author: Damien CORNEAU 
Date:   2016-08-22T04:47:46Z

Modify the Community page

commit cb8760861b28b78e2ffc58e65bafd1ee154ce30b
Author: Damien CORNEAU 
Date:   2016-08-22T04:51:45Z

Add forgotten license headers

commit acc68b50d71e16752eb598235d5be32ad6fbf5b8
Author: Damien CORNEAU 
Date:   2016-08-22T05:38:19Z

Add contribution Guidelines to the website

commit 5f7e2bd47c2b027c9d63fb4d0ebc6c9e2b8b2e33
Author: Damien CORNEAU 
Date:   2016-08-22T08:13:14Z

Add a side menu template

commit c933c194b3190961e799b8c9a75071718290c6dd
Author: Damien CORNEAU 
Date:   2016-08-22T08:43:34Z

Add zeppelin-web contribution page

commit 306ae06724a870b2da454283afc851f20b65925f
Author: Damien CORNEAU 
Date:   2016-08-22T09:05:32Z

Add zeppelin-web GPG menu

commit 666f3ec95ed4f7d6aed5efefec3b40269d022bf0
Author: Damien CORNEAU 
Date:   2016-08-23T05:31:29Z

Add first Zeppelin-web best practices page

commit 81493abd9ecbf07cf864f5509e96c1bbabdc72ce
Author: Damien CORNEAU 
Date:   2016-08-23T06:45:29Z

Change Side Menu style




---
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 #1322: [ZEPPELIN-1320] Run zeppelin interpreter process as we...

2016-08-23 Thread prabhjyotsingh

Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/1322
  
Agreed @echarles, the `doAs` part will be a problem, until 
[ZEPPELIN-1340](https://issues.apache.org/jira/browse/ZEPPELIN-1340) is 
resolved. Until then for security we may have to run half interpreter with 
"User Impersonate" enable from UI (for example shell, python interpreter), and 
for others use the standard `doAs` already implemented (like livy, spark, jdbc)


---
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 #1346: Update Utils.java