[GitHub] zeppelin issue #1808: [ZEPPELIN-1843] Error on invoking the REST API to run ...

2016-12-28 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1808 The travis build is green. --- 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 #1808: [ZEPPELIN-1843] Error on invoking the REST API to run ...

2016-12-28 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1808 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

[GitHub] zeppelin pull request #1808: [ZEPPELIN-1843] Error on invoking the REST API ...

2016-12-28 Thread kavinkumarks
Github user kavinkumarks closed the pull request at: https://github.com/apache/zeppelin/pull/1808 --- 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

[GitHub] zeppelin pull request #1808: [ZEPPELIN-1843] Error on invoking the REST API ...

2016-12-28 Thread kavinkumarks
GitHub user kavinkumarks reopened a pull request: https://github.com/apache/zeppelin/pull/1808 [ZEPPELIN-1843] Error on invoking the REST API to run paragraph synchronously ### What is this PR for? This fixes the validation check of paragraph's note id to match

[GitHub] zeppelin issue #1808: [ZEPPELIN-1843] Error on invoking the REST API to run ...

2016-12-28 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1808 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

[GitHub] zeppelin pull request #1808: [ZEPPELIN-1843] Error on invoking the REST API ...

2016-12-28 Thread kavinkumarks
GitHub user kavinkumarks reopened a pull request: https://github.com/apache/zeppelin/pull/1808 [ZEPPELIN-1843] Error on invoking the REST API to run paragraph synchronously ### What is this PR for? This fixes the validation check of paragraph's note id to match

[GitHub] zeppelin pull request #1808: [ZEPPELIN-1843] Error on invoking the REST API ...

2016-12-28 Thread kavinkumarks
Github user kavinkumarks closed the pull request at: https://github.com/apache/zeppelin/pull/1808 --- 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

[GitHub] zeppelin pull request #1808: [ZEPPELIN-1843] Error on invoking the REST API ...

2016-12-28 Thread kavinkumarks
GitHub user kavinkumarks reopened a pull request: https://github.com/apache/zeppelin/pull/1808 [ZEPPELIN-1843] Error on invoking the REST API to run paragraph synchronously ### What is this PR for? This fixes the validation check of paragraph's note id to match

[GitHub] zeppelin pull request #1808: [ZEPPELIN-1843] Error on invoking the REST API ...

2016-12-28 Thread kavinkumarks
Github user kavinkumarks closed the pull request at: https://github.com/apache/zeppelin/pull/1808 --- 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

[GitHub] zeppelin issue #1808: [ZEPPELIN-1843] Error on invoking the REST API to run ...

2016-12-28 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1808 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

[GitHub] zeppelin issue #1535: [Zeppelin-53] Broken newly created Notebook if there i...

2016-11-03 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1535 Resolved the conflicts and the build ran but there are failures related to spark for 2 consecutive times.Refer below for details: `Tests run: 1, Failures: 0, Errors: 1, Skipped: 0

[GitHub] zeppelin pull request #1535: [Zeppelin-53] Broken newly created Notebook if ...

2016-11-03 Thread kavinkumarks
Github user kavinkumarks closed the pull request at: https://github.com/apache/zeppelin/pull/1535 --- 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

[GitHub] zeppelin pull request #1535: [Zeppelin-53] Broken newly created Notebook if ...

2016-11-03 Thread kavinkumarks
GitHub user kavinkumarks reopened a pull request: https://github.com/apache/zeppelin/pull/1535 [Zeppelin-53] Broken newly created Notebook if there is no permissions to mkdir on disk ### What is this PR for? Error message should be shown to the user when

[GitHub] zeppelin issue #1535: [Zeppelin-53] Broken newly created Notebook if there i...

2016-10-26 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1535 The travis build is green again and rebased with the latest master. --- 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

[GitHub] zeppelin pull request #1535: [Zeppelin-53] Broken newly created Notebook if ...

2016-10-26 Thread kavinkumarks
Github user kavinkumarks closed the pull request at: https://github.com/apache/zeppelin/pull/1535 --- 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

[GitHub] zeppelin pull request #1535: [Zeppelin-53] Broken newly created Notebook if ...

2016-10-26 Thread kavinkumarks
GitHub user kavinkumarks reopened a pull request: https://github.com/apache/zeppelin/pull/1535 [Zeppelin-53] Broken newly created Notebook if there is no permissions to mkdir on disk ### What is this PR for? Error message should be shown to the user when there is permission

[GitHub] zeppelin issue #1535: [Zeppelin-53] Broken newly created Notebook if there i...

2016-10-26 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1535 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

[GitHub] zeppelin issue #1535: [Zeppelin-53] Broken newly created Notebook if there i...

2016-10-26 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1535 Can someone else review and merge 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

[GitHub] zeppelin issue #1520: [ZEPPELIN-1061] Select default interpreter while creat...

2016-10-26 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1520 The travis build is green after fixing the test case failure. Thanks, Kavin MailTo: kavin.ku...@imaginea.com --- If your project is set up for it, you can reply

[GitHub] zeppelin issue #1520: [ZEPPELIN-1061] Select default interpreter while creat...

2016-10-25 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1520 @AhyoungRyu I have resolved the conflicts and also fixed the issue "Can we reset the selected interpreter in the note creation page? Currently the dialog keeps the previously selecte

[GitHub] zeppelin issue #1520: [ZEPPELIN-1061] Select default interpreter while creat...

2016-10-23 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1520 @jongyoul & @AhyoungRyu thanks for reviewing it. @AhyoungRyu I will look into the issue. @corneadoug any updates on this? Thanks, Kavin MailTo: kavin.ku...@imaginea

[GitHub] zeppelin issue #1449: [MINOR] Change Markdown class name from MarkdownInterp...

2016-10-20 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1449 Thanks @Leemoonsoo ! - Kavin --- 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

[GitHub] zeppelin issue #1520: [ZEPPELIN-1061] Select default interpreter while creat...

2016-10-19 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1520 @jongyoul any udpates on 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 this feature

[GitHub] zeppelin issue #1449: [MINOR] Change Markdown class name from MarkdownInterp...

2016-10-19 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1449 @Leemoonsoo Yeah, it worked by changing it manually.I thought that it should have been taken care auto* when someone changes the relevant class name in the appropriate interpreter

[GitHub] zeppelin issue #1449: [MINOR] Change Markdown class name from MarkdownInterp...

2016-10-18 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1449 I am facing issues in my local on running paragraph with md interpreter since the markdown class name is 'MarkdownInterpreter' in conf/interpreter.json What should I do to resolve

[GitHub] zeppelin issue #1535: [Zeppelin-53] Broken newly created Notebook if there i...

2016-10-18 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1535 The CI build is green. Thanks, Kavin MailTo: kavin.ku...@imaginea.com --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] zeppelin issue #1535: [Zeppelin-53] Broken newly created Notebook if there i...

2016-10-18 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1535 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

[GitHub] zeppelin pull request #1535: [Zeppelin-53] Broken newly created Notebook if ...

2016-10-18 Thread kavinkumarks
Github user kavinkumarks closed the pull request at: https://github.com/apache/zeppelin/pull/1535 --- 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

[GitHub] zeppelin pull request #1535: [Zeppelin-53] Broken newly created Notebook if ...

2016-10-18 Thread kavinkumarks
GitHub user kavinkumarks reopened a pull request: https://github.com/apache/zeppelin/pull/1535 [Zeppelin-53] Broken newly created Notebook if there is no permissions to mkdir on disk ### What is this PR for? Error message should be shown to the user when there is permission

[GitHub] zeppelin pull request #1535: [Zeppelin-53] Broken newly created Notebook if ...

2016-10-18 Thread kavinkumarks
GitHub user kavinkumarks opened a pull request: https://github.com/apache/zeppelin/pull/1535 [Zeppelin-53] Broken newly created Notebook if there is no permissions to mkdir on disk ### What is this PR for? Error message should be shown to the user when there is permission issue

[GitHub] zeppelin issue #1524: [ZEPPELIN-1542] Cleanup of Note created during jUnit t...

2016-10-18 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1524 LGTM :+1: Thanks, Kavin MailTo: kavin.ku...@imaginea.com --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] zeppelin issue #1515: [ZEPPELIN-916] Apply new mechanism to KylinInterpreter

2016-10-17 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1515 LGTM :+1: Thanks, Kavin MailTo: kavin.ku...@imaginea.com --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] zeppelin issue #1497: [ZEPPELIN-1233] Support Redis interpreter

2016-10-14 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1497 I too checked the changes and they look good.The comment from @anthonycorbacho about usage of "Strategy pattern" for selecting the relevant redis command instance and exec

[GitHub] zeppelin issue #1520: [ZEPPELIN-1061] Select default interpreter while creat...

2016-10-14 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1520 The changes are done. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled

[GitHub] zeppelin pull request #1520: [ZEPPELIN-1061] Select default interpreter whil...

2016-10-14 Thread kavinkumarks
GitHub user kavinkumarks opened a pull request: https://github.com/apache/zeppelin/pull/1520 [ZEPPELIN-1061] Select default interpreter while creating note ### What is this PR for? This handles setting the default interpreter on creating a note through the zeppelin UI

[GitHub] zeppelin issue #1473: [ZEPPELIN-908] Apply new mechanism to CassandraInterpr...

2016-10-03 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1473 Thanks @doanduyhai and @jongyoul Kavin MailTo: kavin.ku...@imaginea.com --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] zeppelin issue #1473: [ZEPPELIN-908] Apply new mechanism to CassandraInterpr...

2016-09-29 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1473 Thanks @doanduyhai for reviewing it and the CI build is green! - Kavin MailTo: kavin.ku...@imaginea.com --- If your project is set up for it, you can reply to this email

[GitHub] zeppelin pull request #1473: [ZEPPELIN-908] Apply new mechanism to Cassandra...

2016-09-29 Thread kavinkumarks
GitHub user kavinkumarks reopened a pull request: https://github.com/apache/zeppelin/pull/1473 [ZEPPELIN-908] Apply new mechanism to CassandraInterpreter ### What is this PR for? This handles replacing the registration of interpreter with static block by the interpreter

[GitHub] zeppelin issue #1473: [ZEPPELIN-908] Apply new mechanism to CassandraInterpr...

2016-09-29 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1473 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

[GitHub] zeppelin pull request #1473: [ZEPPELIN-908] Apply new mechanism to Cassandra...

2016-09-29 Thread kavinkumarks
Github user kavinkumarks closed the pull request at: https://github.com/apache/zeppelin/pull/1473 --- 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

[GitHub] zeppelin pull request #1473: [ZEPPELIN-908] Apply new mechanism to Cassandra...

2016-09-29 Thread kavinkumarks
GitHub user kavinkumarks opened a pull request: https://github.com/apache/zeppelin/pull/1473 [ZEPPELIN-908] Apply new mechanism to CassandraInterpreter ### What is this PR for? This handles replacing the registration of interpreter with static block by the interpreter

[GitHub] zeppelin issue #1408: [ZEPPELIN-808] Changed Permissions on Notebooks do not...

2016-09-26 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1408 The CI build is green. @corneadoug thanks for the update! Could you please take care of merging this? Thanks, Kavin MailTo: kavin.ku...@imaginea.com --- If your

[GitHub] zeppelin pull request #1408: [ZEPPELIN-808] Changed Permissions on Notebooks...

2016-09-25 Thread kavinkumarks
GitHub user kavinkumarks reopened a pull request: https://github.com/apache/zeppelin/pull/1408 [ZEPPELIN-808] Changed Permissions on Notebooks do not provide helpful error to user ### What is this PR for? This is about showing information to the user when there are errors

[GitHub] zeppelin issue #1408: [ZEPPELIN-808] Changed Permissions on Notebooks do not...

2016-09-25 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1408 @corneadoug I checked the various combinations of file permissions for the notebook folder eg. permissions 444,400. The issue that you reported above that the notebook is not listed

[GitHub] zeppelin issue #1408: [ZEPPELIN-808] Changed Permissions on Notebooks do not...

2016-09-22 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1408 The CI build is green. --- 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 pull request #1408: [ZEPPELIN-808] Changed Permissions on Notebooks...

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

[GitHub] zeppelin pull request #1408: [ZEPPELIN-808] Changed Permissions on Notebooks...

2016-09-22 Thread kavinkumarks
GitHub user kavinkumarks reopened a pull request: https://github.com/apache/zeppelin/pull/1408 [ZEPPELIN-808] Changed Permissions on Notebooks do not provide helpful error to user ### What is this PR for? This is about showing information to the user when there are errors

[GitHub] zeppelin issue #1408: [ZEPPELIN-808] Changed Permissions on Notebooks do not...

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

[GitHub] zeppelin issue #1408: [ZEPPELIN-808] Changed Permissions on Notebooks do not...

2016-09-22 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1408 @corneadoug Made changes to exit the paragraph execution when there are errors with the notebook file system. Thanks, Kavin MailTo: kavin.ku...@imaginea.com --- If your

[GitHub] zeppelin issue #1408: [ZEPPELIN-808] Changed Permissions on Notebooks do not...

2016-09-19 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1408 @corneadoug Sorry, I wasn't clear about the steps given in the PR.This was about the permission issue with the actual notebook file system and not with the permissions feature

[GitHub] zeppelin issue #1408: [ZEPPELIN-808] Changed Permissions on Notebooks do not...

2016-09-14 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1408 Could someone else review and merge this? Thanks, Kavin MailTo: kavin.ku...@imaginea.com --- If your project is set up for it, you can reply to this email and have your

[GitHub] zeppelin issue #1408: [ZEPPELIN-808] Changed Permissions on Notebooks do not...

2016-09-14 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1408 The CI build is green. --- 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 #1408: [ZEPPELIN-808] Changed Permissions on Notebooks do not...

2016-09-14 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1408 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

[GitHub] zeppelin pull request #1408: [ZEPPELIN-808] Changed Permissions on Notebooks...

2016-09-14 Thread kavinkumarks
GitHub user kavinkumarks reopened a pull request: https://github.com/apache/zeppelin/pull/1408 [ZEPPELIN-808] Changed Permissions on Notebooks do not provide helpful error to user ### What is this PR for? This is about showing information to the user when there are errors

[GitHub] zeppelin issue #1408: [ZEPPELIN-808] Changed Permissions on Notebooks do not...

2016-09-14 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1408 @khalidhuseynov thanks for checking the changes.I have updated the screenshot and triggered the CI build again. Thanks, Kavin MailTo: kavin.ku...@imaginea.com --- If your

[GitHub] zeppelin issue #1393: [ZEPPELIN-1142] Zeppelin allows two users to simultane...

2016-09-13 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1393 Thanks @Leemoonsoo for reviewing this! Could you please merge this? - Kavin MailTo: kavin.ku...@imaginea.com --- If your project is set up for it, you can reply to this email

[GitHub] zeppelin issue #1408: [ZEPPELIN-808] Changed Permissions on Notebooks do not...

2016-09-12 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1408 @khalidhuseynov I have made the changes and committed them.I have handled the exception in runParagraph() instead of throwing custom exception from VFSNotebookRepo class since the exception

[GitHub] zeppelin issue #1420: [ZEPPELIN-1421] Fix dead link in docs/README.md

2016-09-12 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1420 @AhyoungRyu the changes look good! Thanks, Kavin MailTo: kavin.ku...@imaginea.com --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] zeppelin issue #1408: [ZEPPELIN-808] Changed Permissions on Notebooks do not...

2016-09-09 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1408 @khalidhuseynov thanks for sharing the inputs on this.Basically I like your idea of propagating the specific error to the user instead of the generic error message so the user would be more

[GitHub] zeppelin issue #1420: [ZEPPELIN-1421] Fix dead link in docs/README.md

2016-09-09 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1420 @AhyoungRyu thanks for the information but we can use the path https://github.com/apache/zeppelin/blob/master/docs/assets/themes/zeppelin/img/docs-img/shell-example.png to access the image

[GitHub] zeppelin issue #1420: [ZEPPELIN-1421] Fix dead link in docs/README.md

2016-09-09 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1420 LGTM with the dead link in README.md Also, shouldn't we need to fix the broken image in shell interpreter doc? --- If your project is set up for it, you can reply to this email

[GitHub] zeppelin issue #1393: [ZEPPELIN-1142] Zeppelin allows two users to simultane...

2016-09-09 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1393 Thanks @khalidhuseynov ! Can we get this merged? -Kavin MailTo: kavin.ku...@imaginea.com --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] zeppelin issue #1408: [ZEPPELIN-808] Changed Permissions on Notebooks do not...

2016-09-08 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1408 Could someone please review this? Thanks, Kavin MailTo: kavin.ku...@imaginea.com --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] zeppelin issue #1393: [ZEPPELIN-1142] Zeppelin allows two users to simultane...

2016-09-07 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1393 The CI build is green. --- 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 #1393: [ZEPPELIN-1142] Zeppelin allows two users to simultane...

2016-09-07 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1393 Fixed the build test failure and the CI build is in progress. --- 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

[GitHub] zeppelin issue #1393: [ZEPPELIN-1142] Zeppelin allows two users to simultane...

2016-09-06 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1393 Found an build test failure, working on 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

[GitHub] zeppelin pull request #1408: [ZEPPELIN-808] Changed Permissions on Notebooks...

2016-09-06 Thread kavinkumarks
GitHub user kavinkumarks opened a pull request: https://github.com/apache/zeppelin/pull/1408 [ZEPPELIN-808] Changed Permissions on Notebooks do not provide helpful error to user ### What is this PR for? This is about showing information to the user when there are errors

[GitHub] zeppelin issue #1393: [ZEPPELIN-1142] Zeppelin allows two users to simultane...

2016-09-06 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1393 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

[GitHub] zeppelin pull request #1393: [ZEPPELIN-1142] Zeppelin allows two users to si...

2016-09-06 Thread kavinkumarks
Github user kavinkumarks closed the pull request at: https://github.com/apache/zeppelin/pull/1393 --- 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

[GitHub] zeppelin pull request #1393: [ZEPPELIN-1142] Zeppelin allows two users to si...

2016-09-06 Thread kavinkumarks
GitHub user kavinkumarks reopened a pull request: https://github.com/apache/zeppelin/pull/1393 [ZEPPELIN-1142] Zeppelin allows two users to simultaneously edit the notebook permissions ### What is this PR for? This is about storing the owner information on creating a note so

[GitHub] zeppelin issue #1393: [ZEPPELIN-1142] Zeppelin allows two users to simultane...

2016-09-06 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1393 @khalidhuseynov thanks for the review! I have replied to the comments and the CI build is in progress. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] zeppelin pull request #1393: [ZEPPELIN-1142] Zeppelin allows two users to si...

2016-09-01 Thread kavinkumarks
GitHub user kavinkumarks opened a pull request: https://github.com/apache/zeppelin/pull/1393 [ZEPPELIN-1142] Zeppelin allows two users to simultaneously edit the notebook permissions ### What is this PR for? This is about storing the owner information on creating a note so when

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

2016-09-01 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1348 Thanks everyone! - Kavin --- 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

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

2016-08-29 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1348 @felixcheung / @jongyoul any updates on this? Thanks, Kavin --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

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

2016-08-25 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1348 @felixcheung / @jongyoul could you please take care of merging this? Thanks, Kavin --- If your project is set up for it, you can reply to this email and have your reply appear

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

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

2016-08-19 Thread kavinkumarks
GitHub user kavinkumarks opened 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

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

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

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

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

  1   2   >