Re: thrift vs thrift2

[Jake Farrell]

-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 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 December 8, 2012 11:59 AM +1 Luke Lu December 8, 2012 2:50 AM With rpc moving to protobuf, we should get rid of thrift from HBase core completely :) Stack 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 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