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

2016-08-22 Thread beriaanirudh
Github user beriaanirudh commented on the issue: https://github.com/apache/zeppelin/pull/1283 @corneadoug @jongyoul @bzz this is ready 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 h

[GitHub] zeppelin issue #1338: Zeppelin 1307 - Implement notebook revision in Zeppeli...

2016-08-22 Thread anthonycorbacho
Github user anthonycorbacho commented on the issue: https://github.com/apache/zeppelin/pull/1338 merging if 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 ena

[GitHub] zeppelin issue #1340: [MINOR][DOC] Update available interpreters' image in i...

2016-08-22 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1340 @anthonycorbacho Updated the image as you said. You can check `before` & `after` in the PR description. --- If your project is set up for it, you can reply to this email and have your reply a

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

2016-08-22 Thread kavinkumarks
Github user kavinkumarks 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 featur

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

2016-08-22 Thread kavinkumarks
GitHub user kavinkumarks reopened a pull request: https://github.com/apache/zeppelin/pull/1345 [ZEPPELIN-728] Can't POST interpreter setting (CorsFilter?) ### What is this PR for? This handles the NPE when the input json is empty for the interpreter setting POST request.

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

2016-08-22 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1345 Reopening to trigger the CI build. --- 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 e

[GitHub] zeppelin pull request #1348: [ZEPPELIN-1313] NullPointerException when using...

2016-08-22 Thread kavinkumarks
GitHub user kavinkumarks reopened a pull request: https://github.com/apache/zeppelin/pull/1348 [ZEPPELIN-1313] NullPointerException when using Clone notebook REST API ### What is this PR for? This fixes when the input json is empty for the clone notebook REST API and for this ca

[GitHub] zeppelin pull request #1348: [ZEPPELIN-1313] NullPointerException when using...

2016-08-22 Thread kavinkumarks
Github user kavinkumarks closed the pull request at: https://github.com/apache/zeppelin/pull/1348 --- 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 featur

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

2016-08-22 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1348 Reopening to trigger the CI build. --- 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 e

[GitHub] zeppelin pull request #1341: [ZEPPELIN-1162] Fix rawType in NotebookRestApi

2016-08-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1341 --- 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 #1345: [ZEPPELIN-728] Can't POST interpreter setting (CorsFil...

2016-08-22 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1345 Reopening to trigger the CI build. --- 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 e

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

2016-08-22 Thread kavinkumarks
Github user kavinkumarks 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 featur

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

2016-08-22 Thread kavinkumarks
GitHub user kavinkumarks reopened a pull request: https://github.com/apache/zeppelin/pull/1345 [ZEPPELIN-728] Can't POST interpreter setting (CorsFilter?) ### What is this PR for? This handles the NPE when the input json is empty for the interpreter setting POST request.

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

2016-08-22 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1348 The CI build is green refer to https://travis-ci.org/apache/zeppelin/builds/154093445 Could someone please review this? --- If your project is set up for it, you can reply to this e

[GitHub] zeppelin issue #1322: [ZEPPELIN-1320] Run zeppelin interpreter process as we...

2016-08-22 Thread echarles
Github user echarles commented on the issue: https://github.com/apache/zeppelin/pull/1322 Whatever `su` or `ssh` is used, I feel the main trick is the user provisioning on the host running the interpreter. Until now, the shiro authentication system had no impact on the user provisioni

[GitHub] zeppelin issue #1322: [ZEPPELIN-1320] Run zeppelin interpreter process as we...

2016-08-22 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1322 @echarles , Yes agreed, will need to update in doc, and a extra toolbar near the check box where user can enable User Impersonate. --- If your project is set up for it, you can reply to th

[GitHub] zeppelin issue #1284: [ZEPPELIN-1291]Tagging meta value on interpreter compl...

2016-08-22 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1284 re-trigger CI --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or

[GitHub] zeppelin pull request #1284: [ZEPPELIN-1291]Tagging meta value on interprete...

2016-08-22 Thread astroshim
GitHub user astroshim reopened a pull request: https://github.com/apache/zeppelin/pull/1284 [ZEPPELIN-1291]Tagging meta value on interpreter completion. ### What is this PR for? This PR is for showing the meta tag on auto completion. ### What type of PR is it? I

[GitHub] zeppelin pull request #1284: [ZEPPELIN-1291]Tagging meta value on interprete...

2016-08-22 Thread astroshim
Github user astroshim closed the pull request at: https://github.com/apache/zeppelin/pull/1284 --- 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 i

[GitHub] zeppelin issue #1322: [ZEPPELIN-1320] Run zeppelin interpreter process as we...

2016-08-22 Thread echarles
Github user echarles commented on the issue: https://github.com/apache/zeppelin/pull/1322 To make ZEPPELIN-1337 Umbrella for multiple user support for zeppelin more readable, should we rename the following: - ZEPPELIN-1340: "Run Hadoop-based interpreter process on Kerberos as

[GitHub] zeppelin issue #1322: [ZEPPELIN-1320] Run zeppelin interpreter process as we...

2016-08-22 Thread echarles
Github user echarles commented on the issue: https://github.com/apache/zeppelin/pull/1322 ... and make ZEPPELIN-1320 a subtask of ZEPPELIN-1337 ? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project do

RE: Issues in Zeppelin 0.6.0

2016-08-22 Thread Nagasravanthi, Valluri
Hi Jeff, We are still trying to sort out the issue that we highlighted before, so it’d be great if you could give your insight as soon as possible so that we can try and tackle it in a suitable manner. Thanks and Regards, [Descrip

[GitHub] zeppelin issue #1338: Zeppelin 1307 - Implement notebook revision in Zeppeli...

2016-08-22 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/1338 somehow i keep getting this error when doing checkpoint: ``` ERROR [2016-08-22 14:33:04,618] ({qtp787867107-76} NotebookServer.java[onMessage]:257) - Can't handle message java.l

[GitHub] zeppelin issue #1343: ZEPPELIN-1328 - z.show in python interpreter does not ...

2016-08-22 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/1343 LGTM. is it easy to add automated unit test for this? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not h

[GitHub] zeppelin issue #1347: ZEPPELIN-1284. Unable to run paragraph with default in...

2016-08-22 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/1347 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

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

2016-08-22 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1345 Reopening to trigger the CI build. --- 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 e

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

2016-08-22 Thread kavinkumarks
GitHub user kavinkumarks reopened a pull request: https://github.com/apache/zeppelin/pull/1345 [ZEPPELIN-728] Can't POST interpreter setting (CorsFilter?) ### What is this PR for? This handles the NPE when the input json is empty for the interpreter setting POST request.

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

2016-08-22 Thread kavinkumarks
Github user kavinkumarks 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 featur

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

2016-08-22 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/1348 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

[GitHub] zeppelin issue #1350: [ZEPPELIN-1351]Paragraph status should be ERROR if Par...

2016-08-22 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/1350 is it possible to add some unit test for this? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have thi

[GitHub] zeppelin issue #1350: [ZEPPELIN-1351]Paragraph status should be ERROR if Par...

2016-08-22 Thread mwkang
Github user mwkang commented on the issue: https://github.com/apache/zeppelin/pull/1350 @felixcheung Okay! I will add unit test. Thanks for your comment! --- 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 doe

[GitHub] zeppelin issue #1322: [ZEPPELIN-1320] Run zeppelin interpreter process as we...

2016-08-22 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1322 Yes, you are right, let me do it right away. --- 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 th

[jira] [Created] (ZEPPELIN-1353) java.lang.VerifyError for spark interpreter

2016-08-22 Thread hy5446 (JIRA)
hy5446 created ZEPPELIN-1353: Summary: java.lang.VerifyError for spark interpreter Key: ZEPPELIN-1353 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1353 Project: Zeppelin Issue Type: Bug

[GitHub] zeppelin issue #1284: [ZEPPELIN-1291]Tagging meta value on interpreter compl...

2016-08-22 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/1284 As before, I think it will be more intuitive if in a paragraph, the user types: ``` %hadoop1 spark. ``` And to autocomplete and the popups says `hadoop1`? Yes, I agree t

[GitHub] zeppelin issue #1343: ZEPPELIN-1328 - z.show in python interpreter does not ...

2016-08-22 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1343 There actually are tests, but we do not have CI profiles with both Python2 and Python3 yet. I think, I will take a look into creating one soon. Though @felixcheung is right, there are no Integrati

[GitHub] zeppelin pull request #1343: ZEPPELIN-1328 - z.show in python interpreter do...

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

[jira] [Created] (ZEPPELIN-1354) Inject Services

2016-08-22 Thread Eric Charles (JIRA)
Eric Charles created ZEPPELIN-1354: -- Summary: Inject Services Key: ZEPPELIN-1354 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1354 Project: Zeppelin Issue Type: Improvement

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

2016-08-22 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1345 The CI build is green refer to https://travis-ci.org/apache/zeppelin/builds/154134031 Could someone please review this? --- If your project is set up for it, you can reply to this e

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

2016-08-22 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1302 @astroshim I've also thought the interpreter `restart` button in separated page quite uncomfortable. This PR can be a good start for the better UX i think :) I tested your patch and it almo

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

2016-08-22 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1345 LGTM except minor comment. --- 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 w

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

2016-08-22 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1348 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 fe

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

2016-08-22 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1283 @beriaanirudh You need to change more where you don't use 'diamond operator'. Can you fix them? --- If your project is set up for it, you can reply to this email and have your reply appear on Gi

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

2016-08-22 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1315 @rconline Thanks for explaining it. I've researched the `jceks`. AFAIK, it doesn't need hadoop-common dependencies. It's included by java.security by default. Can you remove hadoop-common and sup

[GitHub] zeppelin issue #1322: [ZEPPELIN-1320] Run zeppelin interpreter process as we...

2016-08-22 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1322 @prabhjyotsingh I agree @echarles's idea. Interpreter tries to find hadoop dependencies first and if it passes, it uses `doAs`. Otherwise, let's talk about how to do it. How do you think of it?

[GitHub] zeppelin issue #1323: [HOTFIX] Add logins authentication for hdfs file inter...

2016-08-22 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1323 The codes look simple. @cloverhearts Can you help me review 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 doe

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

2016-08-22 Thread onkarshedge
Github user onkarshedge commented on the issue: https://github.com/apache/zeppelin/pull/989 @bzz I found this [documentation](http://docs.ipfs.apiary.io/#reference/version/version/version) and there would be no need for dependency. I have used apache-httpcomponents httpclient. ---

[GitHub] zeppelin pull request #1200: [ZEPPELIN-1191] Supported legacy way to run par...

2016-08-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1200 --- 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 #1315: [ZEPPELIN-530] Added changes for Credential Provider, ...

2016-08-22 Thread rconline
Github user rconline commented on the issue: https://github.com/apache/zeppelin/pull/1315 @jongyoul this work is currently solving for AD, but is not limited to it. Going forward users may choose to store their Hive, Hbase and other data system passwords. These passwords as convention

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

2016-08-22 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1315 @rconline It's not clear for me. Do you tell me that users store their Hive, Hbase passwords in HDFS? Or does this support reading `jceks` from HDFS? In your description, this PR is for encryptin

[GitHub] zeppelin issue #1301: [MINOR] Removed unused profiles from spark/pom.xml

2016-08-22 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1301 Merging if there's 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 e

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

2016-08-22 Thread rconline
Github user rconline commented on the issue: https://github.com/apache/zeppelin/pull/1315 @jongyoul let me try to take a step back and try to explain. Zeppelin is going to be used for various use cases, some of which will involve HDFS - Hive/Spark/Phoenix/Hbase etc, some use c

[jira] [Created] (ZEPPELIN-1355) Shiro authentication for LDAP SSO

2016-08-22 Thread Bhanuprakash Prathap (JIRA)
Bhanuprakash Prathap created ZEPPELIN-1355: -- Summary: Shiro authentication for LDAP SSO Key: ZEPPELIN-1355 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1355 Project: Zeppelin

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

2016-08-22 Thread Peilin Yang (JIRA)
Peilin Yang created ZEPPELIN-1356: - Summary: The graph legend truncates at the nearest period (.) in its grouping Key: ZEPPELIN-1356 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1356 Project: Z

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

2016-08-22 Thread Peilin-Yang
GitHub user Peilin-Yang opened 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 issue #928: [ZEPPELIN-116] Add Apache Mahout Interpreter

2016-08-22 Thread rawkintrevo
Github user rawkintrevo commented on the issue: https://github.com/apache/zeppelin/pull/928 for what it's worth (probably not much) this passed travis CI on my branch- [https://travis-ci.org/rawkintrevo/incubator-zeppelin/builds/154183905](https://travis-ci.org/rawkintrevo/incubat

[GitHub] zeppelin pull request #1352: Fix bug in z.show for Python interpreter

2016-08-22 Thread bustios
GitHub user bustios opened a pull request: https://github.com/apache/zeppelin/pull/1352 Fix bug in z.show for Python interpreter ### What is this PR for? Currently, height parameter for z.show implementation to display PNG images in Python interpreter is not working. This PR fix

[jira] [Created] (ZEPPELIN-1357) Category Text lables wrapping not done properly in charts

2016-08-22 Thread nirav patel (JIRA)
nirav patel created ZEPPELIN-1357: - Summary: Category Text lables wrapping not done properly in charts Key: ZEPPELIN-1357 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1357 Project: Zeppelin

[jira] [Created] (ZEPPELIN-1358) Displaying Pandas DataFrame index in table using z.show()

2016-08-22 Thread Paul Bustios (JIRA)
Paul Bustios created ZEPPELIN-1358: -- Summary: Displaying Pandas DataFrame index in table using z.show() Key: ZEPPELIN-1358 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1358 Project: Zeppelin

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

2016-08-22 Thread bustios
Github user bustios commented on the issue: https://github.com/apache/zeppelin/pull/1352 The [CI failure](https://s3.amazonaws.com/archive.travis-ci.org/jobs/154284590/log.txt) seems not to be related to this change. ``` INFO [22:02:24,946][] org.apache.zeppelin.cassandra.Ca

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

2016-08-22 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1352 @bustios thank you for the fix and followup on CI! Could you please also briefly explain, why Python3 compatibility facilities are not required here any more? --- If your project is set up f

[GitHub] zeppelin pull request #1328: ZEPPELIN-1324: Make paragraph code selectable w...

2016-08-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1328 --- 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 #1242: [ZEPPELIN-960] When there is no interpreter, paragraph...

2016-08-22 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1242 Thank you for your patience! Looks good to me, except minor style issue above? As soon as it is addressed - I think we should be good to merge it to master. --- If your project is set up

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

2016-08-22 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1334 @mfelgamal Thank you! Please let me look into CI 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 doe

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

2016-08-22 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1302 @astroshim Oh that was my bad. Please ignore my above comment. I only build `zeppelin-web` not backend side. Tested again and it works well as expected. Let's wait the other reviewers :)

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

2016-08-22 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1290 @mwkang Can you rebase 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 does not have this feature enabled

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

2016-08-22 Thread bustios
Github user bustios commented on the issue: https://github.com/apache/zeppelin/pull/1352 Yes @bzz, I just deleted `if self.py3:` but not `img_str = img_str.decode('ascii')` (it is in line 173), which means it will be executed for both Python 2 and 3. Decoding bytes to string it necess

[GitHub] zeppelin issue #1338: Zeppelin 1307 - Implement notebook revision in Zeppeli...

2016-08-22 Thread anthonycorbacho
Github user anthonycorbacho commented on the issue: https://github.com/apache/zeppelin/pull/1338 @khalidhuseynov fixed the issue, thank for testing --- 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

[GitHub] zeppelin issue #1338: Zeppelin 1307 - Implement notebook revision in Zeppeli...

2016-08-22 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/1338 tested and works well now! 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 en

[GitHub] zeppelin issue #89: enabled download data as csv button

2016-08-22 Thread qinzl1
Github user qinzl1 commented on the issue: https://github.com/apache/zeppelin/pull/89 only 1000 rows download, if i can config more rows for daowload --- 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 no

[GitHub] zeppelin issue #1008: [ZEPPELIN-997] Export data to csv

2016-08-22 Thread qinzl1
Github user qinzl1 commented on the issue: https://github.com/apache/zeppelin/pull/1008 if i can export more than 1000 rows --- 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 enabl

[GitHub] zeppelin issue #1008: [ZEPPELIN-997] Export data to csv

2016-08-22 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1008 @qinzl1 It might be independent from the download button. Queries are usually limited to 1000 rows (it is set in the interpreter's setting). Can you check that? --- If your project is

[GitHub] zeppelin pull request #1208: [gh-pages][ZEPPELIN-1216] Add a matrix table fo...

2016-08-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1208 --- 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 #1352: [ZEPPELIN-1327] Fix bug in z.show for Python interpret...

2016-08-22 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1352 Got it, makes perfect sense! Do you think we should add a comment in there, explaining that compatibility is the reason for such decoding? I feel like this might help future contributors to

[GitHub] zeppelin pull request #1338: Zeppelin 1307 - Implement notebook revision in ...

2016-08-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1338 --- 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 #1322: [ZEPPELIN-1320] Run zeppelin interpreter process as we...

2016-08-22 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1322 Sure, In this PR I was only thinking about the otherwise case i.e. in the environment where hadoop dependencies where not present, and hence start interpreter as end-web-user. --- If you

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

2016-08-22 Thread onkarshedge
GitHub user onkarshedge reopened a pull request: https://github.com/apache/zeppelin/pull/989 Added ipfsnotebookrepo [ Zeppelin-683 ] ### What is this PR for? IPFSNoteBookRepo IPFS storage and sharing zeppelin notes. ### What type of PR is it? Feature

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

2016-08-22 Thread onkarshedge
Github user onkarshedge closed the pull request at: https://github.com/apache/zeppelin/pull/989 --- 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 #1353: ZEPPELIN-1326: make profile to select dependenc...

2016-08-22 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened 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.UserGroupI

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

2016-08-22 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1302 Thank you very much about reviewing this PR. :smile: --- 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 hav

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

2016-08-22 Thread onkarshedge
Github user onkarshedge commented on the issue: https://github.com/apache/zeppelin/pull/989 @bzz CI is green please 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 enabl

[GitHub] zeppelin issue #1322: [ZEPPELIN-1320] Run zeppelin interpreter process as we...

2016-08-22 Thread echarles
Github user echarles commented on the issue: https://github.com/apache/zeppelin/pull/1322 Btw, for the hadoop case (or spark on yarn case), this PR may give an issue for `doAs`. Typically, you configure `hadoop.proxyuser.foo.hosts` and `hadoop.proxyuser.foo.group`, `foo` bein

[GitHub] zeppelin issue #1346: Update Utils.java

2016-08-22 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1346 @oeegee Thanks for the contribution. Looks good to me. Merge to master and branch-0.6 as a bugfix, if there're no more discussions. --- If your project is set up for it, you can reply t

[GitHub] zeppelin issue #1346: Update Utils.java

2016-08-22 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1346 @oeegee Yeah I saw your (unofficial?)issue report on FB. Thanks for fixing this! LGTM 👍 --- If your project is set up for it, you can reply to this email and have your reply appear on Git

[GitHub] zeppelin issue #1322: [ZEPPELIN-1320] Run zeppelin interpreter process as we...

2016-08-22 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