You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zeppelin.apache.org by ggdupont <gi...@git.apache.org> on 2016/08/09 20:54:20 UTC
[GitHub] zeppelin pull request #1310: Apply new mechanism to HbaseInterpreter
GitHub user ggdupont opened a pull request:
https://github.com/apache/zeppelin/pull/1310
Apply new mechanism to HbaseInterpreter
### What is this PR for?
This PR applies the new interpreter registration mechanism to Hbase interpreter.
### What type of PR is it?
(minor) Improvement
### Todos
* [ ] - Task
### What is the Jira issue?
[ZEPPELIN-913]
### How should this be tested?
1. apply this patch
3. build source
4. bin/zeppelin-daemon.sh start
5.Run a paragraph with hbase commands
### Screenshots (if appropriate)
### Questions:
* Does the licenses files need update? => no
* Is there breaking changes for older versions? => well it's not compatible with older static interpreter declaration
* Does this needs documentation? => no
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ggdupont/zeppelin ZEPPELIN-913
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/zeppelin/pull/1310.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #1310
----
commit ca0f36c80e3857e7b5da5ea2f1a6428a10275e65
Author: gdupont <ge...@gmail.com>
Date: 2016-08-08T21:56:15Z
Apply new mechanism to HbaseInterpreter
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] zeppelin issue #1310: Apply new mechanism to HbaseInterpreter
Posted by ggdupont <gi...@git.apache.org>.
Github user ggdupont commented on the issue:
https://github.com/apache/zeppelin/pull/1310
Just realized I kept the old static declaration in comments... Not really clean and should probably be removed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] zeppelin issue #1310: Apply new mechanism to HbaseInterpreter
Posted by felixcheung <gi...@git.apache.org>.
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/1310
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] zeppelin pull request #1310: [ZEPPELIN-913] Apply new mechanism to HbaseInte...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1310
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] zeppelin issue #1310: Apply new mechanism to HbaseInterpreter
Posted by prabhjyotsingh <gi...@git.apache.org>.
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1310
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] zeppelin issue #1310: [ZEPPELIN-913] Apply new mechanism to HbaseInterpreter
Posted by jongyoul <gi...@git.apache.org>.
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1310
@ggdupont Could you tell me your jira ID? I'll update you as an assignee
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] zeppelin issue #1310: [ZEPPELIN-913] Apply new mechanism to HbaseInterpreter
Posted by ggdupont <gi...@git.apache.org>.
Github user ggdupont commented on the issue:
https://github.com/apache/zeppelin/pull/1310
@jongyoul simply gdupont
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---