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