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

2016-08-23 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/989 I'm a bit lost with this feature, is it really considered as a NotebookRepo or does it only keep the revisions? Does importing the Hash mean that it import a Revision as Notebook? Why d

[jira] [Created] (ZEPPELIN-1362) Py4j depend on the version of the package declaration, and Spark - 1.6.2 provide inconsistent

2016-08-23 Thread Hefei Li (JIRA)
Hefei Li created ZEPPELIN-1362: -- Summary: Py4j depend on the version of the package declaration, and Spark - 1.6.2 provide inconsistent Key: ZEPPELIN-1362 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1362

[GitHub] zeppelin pull request #1353: ZEPPELIN-1326: make profile to select dependenc...

2016-08-23 Thread prabhjyotsingh
GitHub user prabhjyotsingh reopened a pull request: https://github.com/apache/zeppelin/pull/1353 ZEPPELIN-1326: make profile to select dependency of hadoop-common for JDBC interpreter ### What is this PR for? java.lang.ClassNotFoundException: org.apache.hadoop.security.UserGrou

[GitHub] zeppelin pull request #1353: ZEPPELIN-1326: make profile to select dependenc...

2016-08-23 Thread prabhjyotsingh
Github user prabhjyotsingh closed the pull request at: https://github.com/apache/zeppelin/pull/1353 --- 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 feat

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

2016-08-23 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1353 @jongyoul Sure, have made both changes. --- 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 fe

[GitHub] zeppelin pull request #1345: [ZEPPELIN-728] Can't POST interpreter setting (...

2016-08-23 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1345 --- 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 e

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

2016-08-23 Thread corneadoug
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. I

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

2016-08-23 Thread AhyoungRyu
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 pro

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

2016-08-23 Thread bzz
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 re

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

2016-08-23 Thread corneadoug
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 onl

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

2016-08-23 Thread bzz
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 ena

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

2016-08-23 Thread AhyoungRyu
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 th

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

2016-08-23 Thread bzz
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

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

[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

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

2016-08-23 Thread bzz
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 ha

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

2016-08-23 Thread bzz
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 y

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

[GitHub] zeppelin issue #1355: [ZEPPELIN-1359] Commit correctly ordered karma.conf fi...

2016-08-23 Thread corneadoug
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

[GitHub] zeppelin pull request #1351: [ZEPPELIN/1356] The graph legend truncates at t...

2016-08-23 Thread Peilin-Yang
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

[GitHub] zeppelin pull request #1351: [ZEPPELIN/1356] The graph legend truncates at t...

2016-08-23 Thread Peilin-Yang
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

[GitHub] zeppelin pull request #1351: [ZEPPELIN/1356] The graph legend truncates at t...

2016-08-23 Thread Peilin-Yang
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

[GitHub] zeppelin pull request #1351: [ZEPPELIN/1356] The graph legend truncates at t...

2016-08-23 Thread Peilin-Yang
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

[GitHub] zeppelin pull request #1351: [ZEPPELIN/1356] The graph legend truncates at t...

2016-08-23 Thread Peilin-Yang
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

[GitHub] zeppelin pull request #1351: [ZEPPELIN/1356] The graph legend truncates at t...

2016-08-23 Thread Peilin-Yang
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

[GitHub] zeppelin pull request #1351: [ZEPPELIN/1356] The graph legend truncates at t...

2016-08-23 Thread Peilin-Yang
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

[GitHub] zeppelin pull request #1351: [ZEPPELIN/1356] The graph legend truncates at t...

2016-08-23 Thread Peilin-Yang
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

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

[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 Thank you @agoodm. I committed a fix, could you try again please? --- 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 do

[jira] [Created] (ZEPPELIN-1360) Printing long strings is very slow in the python interpreter

2016-08-23 Thread Alex Goodman (JIRA)
Alex Goodman created ZEPPELIN-1360: -- Summary: Printing long strings is very slow in the python interpreter Key: ZEPPELIN-1360 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1360 Project: Zeppeli

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

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

[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

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

[GitHub] zeppelin issue #1355: [ZEPPELIN-1359] Commit correctly ordered karma.conf fi...

2016-08-23 Thread AhyoungRyu
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 i

[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

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

[GitHub] zeppelin issue #1348: [ZEPPELIN-1313] NullPointerException when using Clone ...

2016-08-23 Thread jongyoul
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

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

2016-08-23 Thread jongyoul
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. --

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

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

2016-08-23 Thread jongyoul
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

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

2016-08-23 Thread jongyoul
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

[GitHub] zeppelin issue #1315: [ZEPPELIN-530] Added changes for Credential Provider, ...

2016-08-23 Thread jongyoul
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

[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

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

2016-08-23 Thread corneadoug
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

[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: B

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

2016-08-23 Thread corneadoug
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 pag