[GitHub] incubator-zeppelin pull request: HBase Shell Interpreter

2016-02-02 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/278 --- 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 fea

[GitHub] incubator-zeppelin pull request: HBase Shell Interpreter

2016-02-02 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/278#issuecomment-178437156 Cool! --- 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 f

[GitHub] incubator-zeppelin pull request: HBase Shell Interpreter

2016-02-01 Thread vrajat
Github user vrajat commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/278#issuecomment-178390216 w00t! On Tue, Feb 2, 2016 at 11:25 AM Felix Cheung wrote: > Great! It works (tested with hbase-1.0.2). > Merging if no more discussio

[GitHub] incubator-zeppelin pull request: HBase Shell Interpreter

2016-02-01 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/278#issuecomment-178390124 Great! It works (tested with hbase-1.0.2). Merging if no more discussion (I will help fix the doc format). --- If your project is set up for it, you

[GitHub] incubator-zeppelin pull request: HBase Shell Interpreter

2016-02-01 Thread vrajat
Github user vrajat commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/278#issuecomment-177969630 @felixcheung Please take a look now. I have fixed all bugs and I can use it on my HBase setup (HBase 1.0.0 and Hadoop 2.6.0) --- If your project is set up f

[GitHub] incubator-zeppelin pull request: HBase Shell Interpreter

2016-01-31 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/278#issuecomment-177817602 cool, let me know when I should try again. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as wel

[GitHub] incubator-zeppelin pull request: HBase Shell Interpreter

2016-01-31 Thread vrajat
Github user vrajat commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/278#issuecomment-177816952 @felixcheung I've got past that error. I am hitting ``` java.lang.VerifyError: class org.apache.hadoop.hbase.protobuf.generated.HBaseProtos$Column

[GitHub] incubator-zeppelin pull request: HBase Shell Interpreter

2016-01-31 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/278#issuecomment-177814834 @vrajat I see this error when I run it ``` INFO [2016-02-01 06:55:28,887] ({pool-1-thread-3} HbaseInterpreter.java[open]:88) - Home:/opt/hba

[GitHub] incubator-zeppelin pull request: HBase Shell Interpreter

2016-01-31 Thread vrajat
Github user vrajat commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/278#issuecomment-177487227 Ah ha! Now I remember why I had my own copy of hirb.rb. I couldnt get this piece of code in the original hirb.rb to work. So I had deleted it and it had no ba

[GitHub] incubator-zeppelin pull request: HBase Shell Interpreter

2016-01-29 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/278#issuecomment-177057862 Sure - let's get it working, and [here](https://github.com/apache/incubator-zeppelin/blob/master/zeppelin-zengine/src/main/java/org/apache/zeppelin/conf/Z

[GitHub] incubator-zeppelin pull request: HBase Shell Interpreter

2016-01-29 Thread vrajat
Github user vrajat commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/278#issuecomment-177045808 Sorry. I've been busy with work. I'll try to get it next week. I'll look at the bug. It must have been caused by one of the clean up commits. With respe

[GitHub] incubator-zeppelin pull request: HBase Shell Interpreter

2016-01-29 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/278#issuecomment-176933999 @vrajat ? --- 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 t

[GitHub] incubator-zeppelin pull request: HBase Shell Interpreter

2016-01-21 Thread AhyoungRyu
Github user AhyoungRyu commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/278#issuecomment-173790541 @vrajat Hi! It's really good to add HBase interpreter!! I saw a documentation `hbase.md`you added. But currently, there is no link for this

[GitHub] incubator-zeppelin pull request: HBase Shell Interpreter

2016-01-21 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/278#issuecomment-173760087 `hbase shell` is calling `${HBASE_HOME}/bin/hirb.rb` - maybe this needs to do the same? --- If your project is set up for it, you can reply to this emai

[GitHub] incubator-zeppelin pull request: HBase Shell Interpreter

2016-01-20 Thread vrajat
Github user vrajat commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/278#issuecomment-173466556 Surprising. It seems like the hbase ruby scripts arent loaded. I used the interpreter two days back. I'll try to reproduce the problem. On Thu, Jan

[GitHub] incubator-zeppelin pull request: HBase Shell Interpreter

2016-01-20 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/278#issuecomment-173465856 @vrajat I am having problem getting this to work. It's like this is my setup but I wonder if you have any idea to fix this - HBase bin release from Ap

[GitHub] incubator-zeppelin pull request: HBase Shell Interpreter

2016-01-20 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/278#issuecomment-173454486 also, I realize when testing this, you should add to the list in [ZeppelinConfiguration.java](https://github.com/apache/incubator-zeppelin/blob/master/zep

[GitHub] incubator-zeppelin pull request: HBase Shell Interpreter

2016-01-20 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/278#issuecomment-173451402 thanks, trying to test this now. btw, this [doc style update](https://github.com/apache/incubator-zeppelin/commit/0d157aebd2199cc6badc36810fd4518fa539

[GitHub] incubator-zeppelin pull request: HBase Shell Interpreter

2016-01-20 Thread vrajat
Github user vrajat commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/278#issuecomment-173447413 @felixcheung 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 hav

[GitHub] incubator-zeppelin pull request: HBase Shell Interpreter

2016-01-20 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/278#issuecomment-173427176 For these new dependencies in the pom file ``` org.apache.hbase + hbase-client + ${hbase.hbase.version} + +

[GitHub] incubator-zeppelin pull request: HBase Shell Interpreter

2016-01-20 Thread vrajat
Github user vrajat commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/278#issuecomment-173420899 License of HBase ? On Thu, 21 Jan 2016 at 05:31, Felix Cheung wrote: > @vrajat could you check the licenses then?

[GitHub] incubator-zeppelin pull request: HBase Shell Interpreter

2016-01-20 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/278#issuecomment-173407376 @vrajat could you check the licenses then? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] incubator-zeppelin pull request: HBase Shell Interpreter

2016-01-20 Thread vrajat
Github user vrajat commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/278#issuecomment-173201439 @bzz I have added a doc. Its quite simple. HBase docs are really awesome. So I have pointed to their docs for the bulk of information. I've only focused on th

[GitHub] incubator-zeppelin pull request: HBase Shell Interpreter

2016-01-11 Thread bzz
Github user bzz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/278#issuecomment-170483684 You can also read more about how to update the docs here http://zeppelin.incubator.apache.org/docs/0.6.0-incubating-SNAPSHOT/development/howtocontributewebsite.ht

[GitHub] incubator-zeppelin pull request: HBase Shell Interpreter

2016-01-11 Thread bzz
Github user bzz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/278#issuecomment-170482882 Sure! The best place to look at is some example of recently merged interpreters - i.e #520 Passing CI, docs and update to the LICENCE file with new depe

[GitHub] incubator-zeppelin pull request: HBase Shell Interpreter

2016-01-11 Thread vrajat
Github user vrajat commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/278#issuecomment-170481220 Can you guide me more ? Where should I change and a good example? On Mon, 11 Jan 2016 at 14:50, Alexander wrote: > Great! > > I think a go

[GitHub] incubator-zeppelin pull request: HBase Shell Interpreter

2016-01-11 Thread bzz
Github user bzz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/278#issuecomment-170480597 Great! I think a good next step is to update the docs, adding the details about a new interpreter. --- If your project is set up for it, you can reply

[GitHub] incubator-zeppelin pull request: HBase Shell Interpreter

2016-01-10 Thread vrajat
Github user vrajat commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/278#issuecomment-170434324 Cool! What are the next steps ? On Mon, Jan 11, 2016 at 10:28 AM Felix Cheung wrote: > looks good to me. > > — > Reply to t

[GitHub] incubator-zeppelin pull request: HBase Shell Interpreter

2016-01-10 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/278#issuecomment-170431120 looks good to me. --- 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] incubator-zeppelin pull request: HBase Shell Interpreter

2016-01-09 Thread vrajat
Github user vrajat commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/278#issuecomment-170259954 @bzz Please check. I have replied to or addressed outstanding comments. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] incubator-zeppelin pull request: HBase Shell Interpreter

2016-01-05 Thread vrajat
Github user vrajat commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/278#issuecomment-168980797 I'll put some effort into clean up the PR. On Tue, 5 Jan 2016 at 15:34, Alexander wrote: > @vrajat do you plan to upda

[GitHub] incubator-zeppelin pull request: HBase Shell Interpreter

2016-01-05 Thread bzz
Github user bzz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/278#issuecomment-168959651 @vrajat do you plan to update this PR\add some docs for new interpreter? Or shall we consider closing it as it diverged from master and CI fails, so no way to

[GitHub] incubator-zeppelin pull request: HBase Shell Interpreter

2015-09-10 Thread vrajat
Github user vrajat commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/278#issuecomment-139368738 The failure is a false positive. I made only cosmetic changes. Looks like it failed in downloading npm stuff. --- If your project is set up for it, you can

[GitHub] incubator-zeppelin pull request: HBase Shell Interpreter

2015-09-07 Thread vrajat
Github user vrajat commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/278#issuecomment-138369320 @felixcheung I have addressed your comments. Please take a look --- If your project is set up for it, you can reply to this email and have your reply appear o

[GitHub] incubator-zeppelin pull request: HBase Shell Interpreter

2015-09-06 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/278#issuecomment-138099373 Could you elaborate on what this does? ``` hbase.irb.load: (Testing only) Default is true. Whether to load irb in the interpreter. ``` --- I

[GitHub] incubator-zeppelin pull request: HBase Shell Interpreter

2015-09-03 Thread vrajat
Github user vrajat commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/278#issuecomment-137601196 Unit tests passed. --- 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] incubator-zeppelin pull request: HBase Shell Interpreter

2015-09-03 Thread vrajat
GitHub user vrajat opened a pull request: https://github.com/apache/incubator-zeppelin/pull/278 HBase Shell Interpreter Support for Hbase Shell. All the commands documented here https://wiki.apache.org/hadoop/Hbase/Shell is supported. Requirements: Hbase Shell sh