Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Threaded View
HBase >> mail # dev >> thrift vs thrift2


Copy link to this message
-
Re: thrift vs thrift2
I agree with what Jake said.
I can also spend some time maintaining the new maven module.

On Mon, Dec 10, 2012 at 12:36 PM, Jake Farrell <[EMAIL PROTECTED]> wrote:

> -1 for removing all together
>
> If removing from core I agree that it should still be offered as a
> separate maven module. If there is need for a maintainer of that module I
> would be glad to help
>
> -Jake
>
>   Andrew Purtell <[EMAIL PROTECTED]>
>  December 8, 2012 9:15 PM
> Depends on what is your definition of "core".
>
> As a separate maven module in the distro, +1.
>
> As ejected to unmaintained status, -1.
>
> I have to imagine the number of users who use thrift to talk to HBase is
> nonzero given we've had this option forever. But let's tag everything
> Thrift as deprecated and see.
>
>
>
>   Jimmy Xiang <[EMAIL PROTECTED]>
>  December 8, 2012 11:59 AM
> +1
>
>   Luke Lu <[EMAIL PROTECTED]>
>  December 8, 2012 2:50 AM
> With rpc moving to protobuf, we should get rid of thrift from HBase core
> completely :)
>
>
>
>   Stack <[EMAIL PROTECTED]>
>  December 8, 2012 2:31 AM
>
>
> Here is what Karthik had to say a while back
>
> http://search-hadoop.com/m/GAFt6B9CRZ/joe+pallas+karthik&subj=Re+Thrift2+interface
>
> As I read it, we should purge the code as a failed approach unless someone
> wants to take on this effort.
>
> St.Ack
>
>   Ted Yu <[EMAIL PROTECTED]>
>  December 8, 2012 12:12 AM
> Looks like there were activities on HRegionThriftServer.java in 0.89-fb
> branch till recently:
>
> r1391221 | mbautin | 2012-09-27 14:07:52 -0700 (Thu, 27 Sep 2012) | 15
> lines
>
> [jira] [HBASE-6882] [89-fb] Thrift IOError should include exception class
>
> Author: mbautin
>
> Summary: Return exception class as part of IOError thrown from the Thrift
> proxy or the embedded Thrift server in the regionserver.
>
> Test Plan:
> Unit tests
> Test through C++ HBase client
>
> Reviewers: liyintang, kranganathan, aaiyer, chip
>
> Reviewed By: liyintang
>
> Differential Revision: https://reviews.facebook.net/D5679
> ------------------------------------------------------------------------
> r1388183 | mbautin | 2012-09-20 12:47:23 -0700 (Thu, 20 Sep 2012) | 15
> lines
>
> [HBASE-6836] [89-fb] Parallel deletes in HBase Thrift server
>
> Author: mbautin
>
> Summary: We need to expose server-side parallel batch deletes through the
> Thrift server.
>
> Test Plan:
> Unit tests
> C++ client test
>
> Reviewers: aaiyer, nzhang, kranganathan
>
> Reviewed By: aaiyer
>
> Differential Revision: https://reviews.facebook.net/D5541
> ------------------------------------------------------------------------
> r1387433 | mbautin | 2012-09-18 18:52:58 -0700 (Tue, 18 Sep 2012) | 15
> lines
>
> [jira] [HBASE-6808] [89-fb] Add the ability to query region flush time and
> current time to regionserver Thrift API
>
>
>