Home | About | Sematext search-lucene.com search-hadoop.com
NEW: Monitor These Apps!
elasticsearch, apache solr, apache hbase, hadoop, redis, casssandra, amazon cloudwatch, mysql, memcached, apache kafka, apache zookeeper, apache storm, ubuntu, centOS, red hat, debian, puppet labs, java, senseiDB
 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
>
>
>
NEW: Monitor These Apps!
elasticsearch, apache solr, apache hbase, hadoop, redis, casssandra, amazon cloudwatch, mysql, memcached, apache kafka, apache zookeeper, apache storm, ubuntu, centOS, red hat, debian, puppet labs, java, senseiDB