You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@zeppelin.apache.org by felixcheung <gi...@git.apache.org> on 2016/03/28 03:39:28 UTC

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-767] HBase interpreter ...

GitHub user felixcheung opened a pull request:

    https://github.com/apache/incubator-zeppelin/pull/799

    [ZEPPELIN-767] HBase interpreter does not work with HBase on a remote cluster

    ### What is this PR for?
    HBase interpreter fails with message "ERROR: KeeperErrorCode = ConnectionLoss for /hbase" when connecting to a remote HBASE (for instance, HBase running in CDH cluster)
    
    Initially it's thought that zkquoram setttings are not getting applied, but deeper investigations reveal that hbase-site.xml cannot be loaded.
    
    HBASE_HOME or HBASE_CONF_DIR is set by `hbase` script when running hbase shell - interpreter will need to at minimum replicate that behavior to add the directory with hbase-site.xml to CLASS_PATH in order to fix this issue.
    
    ### What type of PR is it?
    Bug Fix
    
    ### Todos
    * [x] - Bug fix
    * [x] - Update documentation
    
    
    ### What is the Jira issue?
    https://issues.apache.org/jira/browse/ZEPPELIN-767
    
    ### How should this be tested?
    (tested) Run HBase locally (standalone: https://hbase.apache.org/book.html#quickstart)
    (tested) Set HBASE_HOME in env and work with HBASE on a Hadoop cluster
    (tested) Set HBASE_CONF_DIR in env and work with HBASE on a Hadoop cluster
    
    ### Screenshots (if appropriate)
    N/A
    
    ### Questions:
    * Does the licenses files need update? No
    * Is there breaking changes for older versions? No
    * Does this needs documentation? Added


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/felixcheung/incubator-zeppelin hbaseconf

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-zeppelin/pull/799.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 #799
    
----
commit eeb341f40e2bfeb764c27d161eee99f3476b7c3b
Author: Felix Cheung <fe...@hotmail.com>
Date:   2016-03-21T21:11:58Z

    set hbase conf dir to classpath

commit a82c2a695666eb25c3f92f35b0711dbf201249fe
Author: Felix Cheung <fe...@hotmail.com>
Date:   2016-03-28T01:06:17Z

    fix bug, add doc, update text

----


---
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] incubator-zeppelin pull request: [ZEPPELIN-767] HBase interpreter ...

Posted by felixcheung <gi...@git.apache.org>.

Github user felixcheung commented on the pull request:

    https://github.com/apache/incubator-zeppelin/pull/799#issuecomment-202187238
  
    @vrajat @bbuild11



---
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] incubator-zeppelin pull request: [ZEPPELIN-767] HBase interpreter ...

Posted by felixcheung <gi...@git.apache.org>.

Github user felixcheung commented on the pull request:

    https://github.com/apache/incubator-zeppelin/pull/799#issuecomment-204068310
  
    test failure was:
    ```
    Failed tests: 
      ZeppelinSparkClusterTest.pySparkDepLoaderTest:167->getSparkVersionNumber:220 expected:<FINISHED> but was:<ERROR>
      ZeppelinSparkClusterTest.pySparkAutoConvertOptionTest:112->getSparkVersionNumber:220 expected:<FINISHED> but was:<ERROR>
      ZeppelinSparkClusterTest.basicRDDTransformationAndActionTest:81 expected:<FINISHED> but was:<ERROR>
      ZeppelinSparkClusterTest.pySparkTest:90->getSparkVersionNumber:220 expected:<FINISHED> but was:<ERROR>
      ZeppelinSparkClusterTest.zRunTest:152 expected:<FINISHED> but was:<ERROR>
    ```
    
    merging if no more 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 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] incubator-zeppelin pull request: [ZEPPELIN-767] HBase interpreter ...

Posted by asfgit <gi...@git.apache.org>.

Github user asfgit closed the pull request at:

    https://github.com/apache/incubator-zeppelin/pull/799


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