GitHub user kavinkumarks opened a pull request:
https://github.com/apache/zeppelin/pull/1262
[ZEPPELIN-1268]
### What is this PR for?
This pull request is to add the below options so when someone develops a
custom UI it would ease them.
1. Add a new option "Get paragra
Github user kavinkumarks commented on the issue:
https://github.com/apache/zeppelin/pull/1262
@corneadoug thanks for the update.I will remove the changes done for
it.Could someone review the other change that I have done as part of this?
---
If your project is set up for it, you can
Github user kavinkumarks commented on the issue:
https://github.com/apache/zeppelin/pull/1262
@jongyoul Added the unit test and updated docs.Also reverted the UI
code.Could you please check them?
---
If your project is set up for it, you can reply to this email and have your
reply
Github user kavinkumarks commented on the issue:
https://github.com/apache/zeppelin/pull/1262
Have updated the title, please let me know if I am missing anything else.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user kavinkumarks commented on the issue:
https://github.com/apache/zeppelin/pull/1262
@jongyoul I dont see the option to restart the build in travis since I
don't have the write access to repo.And I didn't find any way other than doing
some hacks.Could you please l
GitHub user kavinkumarks reopened a pull request:
https://github.com/apache/zeppelin/pull/1262
[ZEPPELIN-1268] As an enduser, I would like to embed paragraph and refresh
the content of it in my custom UI
### What is this PR for?
This pull request is to add the below options so
Github user kavinkumarks commented on the issue:
https://github.com/apache/zeppelin/pull/1262
Reopened to trigger travis 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 user kavinkumarks closed the pull request at:
https://github.com/apache/zeppelin/pull/1262
---
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 user kavinkumarks closed the pull request at:
https://github.com/apache/zeppelin/pull/1262
---
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 user kavinkumarks reopened a pull request:
https://github.com/apache/zeppelin/pull/1262
[ZEPPELIN-1268] As an enduser, I would like to embed paragraph and refresh
the content of it in my custom UI
### What is this PR for?
This pull request is to add the below options so
Github user kavinkumarks commented on the issue:
https://github.com/apache/zeppelin/pull/1262
The latest CI build is green
https://travis-ci.org/apache/zeppelin/builds/150722171 and also rebased the
branch with the latest master. @jongyoul could you please merge and close this
Github user kavinkumarks commented on the issue:
https://github.com/apache/zeppelin/pull/1262
@jongyoul any updates 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 user kavinkumarks commented on the issue:
https://github.com/apache/zeppelin/pull/1262
@jongyoul could you please take merging 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 user kavinkumarks commented on the issue:
https://github.com/apache/zeppelin/pull/1262
Could someone take care of merging this? NOTE that the review was over.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user kavinkumarks commented on the issue:
https://github.com/apache/zeppelin/pull/1262
Thanks @jongyoul !
---
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
GitHub user kavinkumarks opened 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 re
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 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
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 re
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 re
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
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 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
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 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 re
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 and
for this
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 re
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
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 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
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
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 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
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 re
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 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
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
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 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 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
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 re
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
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 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 re
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
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
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
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 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. If
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 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 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 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 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, or if the
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 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 on
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 this
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 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 and
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 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 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 emai
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 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 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 user kavinkumarks commented on the issue:
https://github.com/apache/zeppelin/pull/1393
Could someone else have a look at this and merge if everything is okay?
Thanks,
Kavin
MailTo: kavin.ku...@imaginea.com
---
If your project is set up for it, you can reply to
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 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 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 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 on
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, or if the
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 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 and
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 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 i
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 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 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, or if the
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 on
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 and
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 on
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, or if the
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 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 on
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 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 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, or if the
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 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 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 and
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 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 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 and
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 executing t
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. If
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. If
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 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 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, or if the
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
1 - 100 of 128 matches
Mail list logo
